Skip to content

Update JukeboxSystem.cs#416

Draft
imatsoup wants to merge 2 commits intomasterfrom
quiet-the-jukebox
Draft

Update JukeboxSystem.cs#416
imatsoup wants to merge 2 commits intomasterfrom
quiet-the-jukebox

Conversation

@imatsoup
Copy link
Collaborator

@imatsoup imatsoup commented Oct 4, 2025

About the PR

Changed the audio params in JukeboxSystem to make it quieter.

Why / Balance

Given that audio prediction is broken with it right now, the jukebox is just way too loud for how far away you can hear it. We had discussed removing the jukebox until it was fixed, but this seems like a better solution after some thought.

Technical details

Media

Requirements

Breaking changes

Changelog

🆑

  • tweak: The Jukebox has been made quieter until audio prediction can be fixed with it.

@github-actions github-actions bot added S: Needs Review Status: This PR requires new reviews before it can be merged. S: Untriaged Status: Needs appropriate labels. size/XS labels Oct 4, 2025
@imatsoup
Copy link
Collaborator Author

imatsoup commented Oct 4, 2025

Converting this to a draft as we explore alternative solutions to avoid accidental merging.

@imatsoup imatsoup marked this pull request as draft October 4, 2025 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S: Needs Review Status: This PR requires new reviews before it can be merged. S: Untriaged Status: Needs appropriate labels. size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants