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

Разблокировка BPM музыкальных инструментов #1585

Closed
wants to merge 3 commits into from

Conversation

msw7007
Copy link

@msw7007 msw7007 commented Oct 9, 2024

Что этот PR делает

Переопределяет функции, отвечающие за настройку темпа при воспроизведении треков с музыкальных инстурментов.
В данный момент она расчитывается по формуле:
600 / X = BPM, где X - значение задержки, и имеет шаг в 0.5, что достаточно жестко лочит BPM.

Почему это хорошо для игры

Музыка (в идеале) будет звучать ближе к оригиналу midi с которого она взята.

Изображения изменений

Тестирование

Тестирование на локале

Changelog

🆑
tweak: Снятие шаговой блокировки BPM для музыкальных инструментов.
/:cl:

@github-actions github-actions bot added :feelsgood: Частичная модульность Не всегда получается всё впихнуть в модуль, увы. TGUI Кто-то решил потрогать интерфейсы. labels Oct 9, 2024
@larentoun
Copy link
Collaborator

Почему не на оффы?

@msw7007
Copy link
Author

msw7007 commented Oct 9, 2024

Почему не на оффы?

Попробовать в виде ТМа, да и самому чет страшненько туда что-либо отправлять

@github-actions github-actions bot added the Merge Conflict Ну блять... label Oct 13, 2024
Copy link

This pull request seems to be stale as there have been no changes in 14 days, please make changes within 7 days or the PR will be closed. If you believe this is a mistake, please inform a development team member on Discord.

@github-actions github-actions bot added the Stale ПР долго был не активен, и требует обновления. label Oct 28, 2024
Copy link

github-actions bot commented Nov 5, 2024

This pull request has not received any updates since being marked stale, and as such is now being automatically closed. Please feel free to re-open this pull request or open a new one once you have new updates.

@github-actions github-actions bot closed this Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:feelsgood: Частичная модульность Не всегда получается всё впихнуть в модуль, увы. Merge Conflict Ну блять... 📜 CL валиден Stale ПР долго был не активен, и требует обновления. TGUI Кто-то решил потрогать интерфейсы.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants