Skip to content
This repository has been archived by the owner on Jan 5, 2024. It is now read-only.

FMOD and sound improvements - busses and effects and properties #553

Merged
merged 8 commits into from
Nov 4, 2023

Conversation

pawnishoovy
Copy link

@pawnishoovy pawnishoovy commented Nov 1, 2023

Gets rid of all the confusing initial busses and replaces them with this hierarchy:

Master
-SFX
-UI
-Music

Sound volume controls SFX and UI, Music volume controls Music. We might want a UI slider or we might not, whatever, it's not in here.

Master has a safety volume limiter added.
SFX has a cool, heavy-handed compression effect added. Can be changed later in a later PR to add proper ducking and sidechaining.

SoundContainers route to SFX by default, but can be changed using BusRouting property.

PanningStrengthMultiplier added, does what it says on the tin, can eventually be used for some ambient effects (or whatever other funky stuff.)

Since the weird Immobile sound bus was removed, the 3D effect processing code now checks for FMOD_2D mode and skips updating positional stuff in that case. May theoretically be the tiniest bit slower, but I doubt it'll ever matter.

@pawnishoovy pawnishoovy added enhancement New feature or request refactor Code needs to be refactored labels Nov 1, 2023
@pawnishoovy pawnishoovy added this to the PreRelease v6.0 milestone Nov 1, 2023
@pawnishoovy pawnishoovy requested a review from Causeless November 1, 2023 11:50
@pawnishoovy pawnishoovy self-assigned this Nov 1, 2023
Managers/AudioMan.cpp Outdated Show resolved Hide resolved
@Causeless Causeless added this pull request to the merge queue Nov 4, 2023
Merged via the queue into development with commit e07cd9f Nov 4, 2023
4 checks passed
@Causeless Causeless deleted the fmod-improvements branch November 4, 2023 20:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request refactor Code needs to be refactored
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants