Skip to content

Fix Vision Prediction, add vision off sound and separate sound channel for vision sounds. Also fix ghost pickup sound not playing on its own special channel - #2041

Open
AdamTadeusz wants to merge 8 commits into
NeotokyoRebuild:masterfrom
AdamTadeusz:370_fixVisionPrediction

Conversation

@AdamTadeusz

@AdamTadeusz AdamTadeusz commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Description

-----> EPILEPSY WARNING <------

Makes toggling the vision mode far smoother at high ping.

Recommend testing through the following:

bind mwheeldown +vision
test how quickly the vision mode can be triggered on a listen server and pay attention to the sound
then
net_fakelag 200
and do the same

also notice how individual inputs at high ping also don't cause the screen to flicker multiple times

Toolchain

  • Windows MSVC VS2022

Linked Issues

@AdamTadeusz
AdamTadeusz requested a review from a team July 31, 2026 16:22
@AdamTadeusz AdamTadeusz changed the title Fix Vision Prediction Fix Vision Prediction, add vision off sound and separate sound channel for vision sounds Jul 31, 2026
@AdamTadeusz AdamTadeusz changed the title Fix Vision Prediction, add vision off sound and separate sound channel for vision sounds Fix Vision Prediction, add vision off sound and separate sound channel for vision sounds. Also fix ghost pickup sound not playing on its own special channel Jul 31, 2026
@AdamTadeusz AdamTadeusz added the Tournament Priority Issues to be prioritized for the upcoming NT;RE tournament label Jul 31, 2026
@AdamTadeusz

Copy link
Copy Markdown
Contributor Author

I added the tournament priority label since this also fixed the ghost pickup sound not playing on its own channel. If need be I can move this fix to a separate PR

@AdamTadeusz

Copy link
Copy Markdown
Contributor Author

also this stops C_NEO_Player::PreDataUpdate returning early before calling the base class that does some important things so thats also important for the tourney

@sunmachine sunmachine left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leaving these as comments for now. Reading more tonight. I feel I'm still really trying to get a handle on this ahead of asking y'all questions as a point of self-exercise. The static stuff was easy to figure out at least.

The other thing I'm trying to figure out, too, is whether this should be two PRs? I can see a world where we need to do a revert for one of the things changed in this, but we'd need to pull out perfectly fine refactoring (so far as I can tell 👀) as a result.

Comment thread src/public/soundflags.h
Comment thread src/game/shared/neo/neo_player_shared.cpp Outdated
Comment thread game/neo/scripts/game_sounds_player.txt
Comment thread game/neo/scripts/game_sounds_ui.txt
Comment thread src/public/SoundParametersInternal.cpp
sunmachine
sunmachine previously approved these changes Aug 3, 2026

@sunmachine sunmachine left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solid PR. Good refactor.

I have a broader question about supposedly dead imports my compiler is warning me about, but that's not only affecting this PR, and its for chat (e.g. how worried are we about transitive dependencies from unused imports)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Tournament Priority Issues to be prioritized for the upcoming NT;RE tournament

Projects

None yet

2 participants