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

Audio track volume control and SoundPlayer ducking #459

Merged

Conversation

avanwinkle
Copy link
Collaborator

Audio Track Volume Control

This PR extends the audio tracks (defined in sound_system: tracks:) to have discrete volume levels that can be controlled from the Service Audio Menu (missionpinball/mpf#1743). Each track can already have a separate volume level, but with this PR that level is adjustable via menu and automatically persisted as a machine var, to allow fine-tuning of relative track levels during gameplay.

SoundPlayer Ducking

This PR adds support for ducking arguments in SoundPlayer, so ducking can be defined at the player level instead of on each individual sound file separately. This is intended as a broad-strokes approach to provide basic ducking quickly, and is not nearly as accurate as defining ducking to each sound specifically. With this in mind, for any sound that has its own ducking: defined that sound's ducking will take priority over the SoundPlayer ducking.

SlidePlayer Logging

This PR reduces the log level of the SlidePlayer from INFO to DEBUG, because during a game lots of slides are played and the logging for slides is bulky. SlidePlayer logs to the machine log, so it's log level cannot be configured separately from the general machine. After this change, the SlidePlayer logs will still appear in verbose mode.

turn it up!

@avanwinkle
Copy link
Collaborator Author

Wheel builder is failing on this (and previous) PRs but I've run all the tests locally and they check out. Proceeding to merge.

@avanwinkle avanwinkle merged commit 7c7c058 into missionpinball:dev Feb 4, 2024
5 of 13 checks passed
@avanwinkle avanwinkle deleted the audio-track-volume-control branch February 4, 2024 23:16
@avanwinkle avanwinkle restored the audio-track-volume-control branch February 6, 2024 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant