Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TTS Optimization #2519

Merged
merged 2 commits into from
Feb 1, 2025
Merged

TTS Optimization #2519

merged 2 commits into from
Feb 1, 2025

Conversation

stalengd
Copy link
Collaborator

Описание PR

Есть довольно правдоподобная теория, что TTS у нас вызывает много лагов, этот PR является попыткой это исправить 2 путями:

  1. Использование аудиоформата OGG - он весит меньше (примерно раз в 10 по моим оценкам), что очень положительно скажется на выделяемую память и нагрузку на сеть.
  2. Оптимизация использования памяти на сервере при обработке TTS

Из-за второго пункта удачи читать этот код, она вам пригодится. Не стесняйтесь спрашивать по всему этому ужасу.
По поводу кэша: он поддерживается, но можно его и отключить (выставить максимальный размер в 0), при этом будет работать механизм, при котором одновременные запросы к TTS "объединяются".

Планируется потестить это дело на тестовом сервере перед мержем.

Медиа

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я ознакомился с наставлениями по работе с репозиторием и следовал им при создании PR'а.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑

  • fix: Оптимизирован TTS!

@github-actions github-actions bot added the Changes: C# Изменение затронуло файлы ".cs" label Jan 31, 2025
@stalengd stalengd marked this pull request as ready for review February 1, 2025 00:38
@stalengd
Copy link
Collaborator Author

stalengd commented Feb 1, 2025

На тестовом побегали, всё должно работать

@Werzet Werzet merged commit b775db5 into SerbiaStrong-220:master Feb 1, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants