Skip to content

Conversation

@Bitl
Copy link
Contributor

@Bitl Bitl commented Oct 14, 2025

Description

This PR restores the cut MvM Upgrade Station music with a clientside implementation. This implementation is enabled by default.

NOTE: This may conflict with custom maps that play the song through external entities. I am trying to figure out a solution for this, so at the moment this will be marked as a draft.

The tf_music_upgrade_machine.wav file must be edited to support looping. The tf_music_upgrade_machine.wav provided with this PR supports looping.

devenv_3XuWYG1mhW.mp4

@Dessensce
Copy link

i personally feel like it would be better as a map edit, where it plays as an ambient_generic in a radius, instead of just when the UI is open

@Bitl Bitl marked this pull request as draft October 14, 2025 14:19
@Bitl
Copy link
Contributor Author

Bitl commented Oct 14, 2025

i personally feel like it would be better as a map edit, where it plays as an ambient_generic in a radius, instead of just when the UI is open

This isn't really the point of it though. This implementation intends to directly integrate the upgrade machine music completely clientside, which helps mappers stay under the entity limit and prevents other users from hearing it when you're in the upgrade station zone. The user can also enable or disable the music if they wish, which they can't do in other map implementations.

@gidi30
Copy link

gidi30 commented Oct 14, 2025

I feel like the description of the CVar calling the music cut doesn't really feel right as if this gets merged the music would no longer be cut.
Something like "Play music while in the mvm upgrade menu." would be better.

@Bitl
Copy link
Contributor Author

Bitl commented Oct 14, 2025

That has been implemented

@Tkain
Copy link
Contributor

Tkain commented Oct 25, 2025

It's worth noting that Freaky Fair has ambient_generic entities playing music at its upgrade stations, so this PR's upgrade station music should probably be disabled by default on that map to prevent overlap. I'm not sure what the most sensible way to disable it there would be, though.

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.

4 participants