Skip to content

Conversation

@eheimburg
Copy link
Contributor

@eheimburg eheimburg commented Mar 28, 2021

Integrates an AudioMixer into the project, and makes the existing volume sliders use it.

  • Adds a mixer, BossRoomMixer, with separate groups for music, combat-sfx and footsteps
  • Adds an AudioMixerConfigurator component which initializes the AudioMixer based on PlayerPrefs. (It is a singleton, and I attached it to the music-player's GameObject in the Startup scene, since that's a singleton too. Figured no reason to add yet another dont-destroy-on-load GameObject...)
  • Volume sliders now delegate actual volume-change to the AudioMixerConfigurator
  • Note that there are two more volumes exposed by the mixer, but aren't currently configurable in-game: Combat-SFX and Footstep volume. These can be easily added to the UI if desired. (Or perhaps just one slider that configures both of those volumes.)

@SamuelBellomo SamuelBellomo added 1-Needs Review PR needs attention from the assignee and reviewers 0-URGENT Blocker for a release and needs to be merged ASAP labels Mar 29, 2021
pdeschain
pdeschain previously approved these changes Mar 29, 2021
@pdeschain pdeschain added 2-One More Review One review in, one to go 2-Reviewed with Comments PR requires owner's attention and removed 1-Needs Review PR needs attention from the assignee and reviewers labels Mar 29, 2021
@fernando-cortez fernando-cortez removed the 2-Reviewed with Comments PR requires owner's attention label Mar 30, 2021
@SamuelBellomo SamuelBellomo merged commit 8a028b2 into release/v0.1.0-experimental Mar 30, 2021
@SamuelBellomo SamuelBellomo deleted the feature/audio_mixer branch March 30, 2021 18:48
@SamuelBellomo SamuelBellomo restored the feature/audio_mixer branch March 30, 2021 18:48
@SamuelBellomo SamuelBellomo deleted the feature/audio_mixer branch March 30, 2021 18:48
SamuelBellomo added a commit that referenced this pull request Mar 30, 2021
* release/v0.1.0-experimental: (97 commits)
  adding screenshots (#228)
  [GOMPS-352] Add audiomixer to project (#222)
  feature: transport picker improvements (#200)
  fixing issue where client disconnect boots everyone else (#229)
  removed GC allocs inside of AIBrain (#210)
  init client door with server's current value for IsOpen (#227)
  Photon Realtime Setup Guide for Boss Room (#225)
  Update README.md add link to doc (#226)
  Bug fix: hide icons for unimplemented attacks
  feature: v0.1.0 experimental updating to latest master mlapi (#220)
  fixing missing reference exceptions, and missing animation (#219)
  feat: replace with photon package (#218)
  Replaced LobbyPlayerArray with a NetworkList<LobbyPlayerState> (#217)
  ART:  Warning Message fixes for models, animations, and Shader...and new App icon (#213)
  Updating to latest MLAPI on master (#214)
  null guard added inside of target action for accessing a possibly destroyed reticule (#212)
  [GOMPS-270] bossroom should let users specify connection port# (#205)
  Fix useless log error (#203)
  Bugfix: properly disconnect on quit [GOMPS-324][GOMPS-327][GOMPS-385] (#196)
  Volume control [GOMPS-373] (#195)
  ...

# Conflicts:
#	README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0-URGENT Blocker for a release and needs to be merged ASAP 2-One More Review One review in, one to go

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants