search

NS2CDT / NS2Plus

NS2+ mod for Natural Selection 2
5 stars 7 forks source link

You can switch to metabolize regardless of legacy option #5

Open GhoulofGSG9 opened 6 years ago

GhoulofGSG9 commented 6 years ago

Issue by keatsandyeats Saturday Apr 07, 2018 at 20:29 GMT Originally opened as https://github.com/sclark39/NS2Plus/issues/84

As of https://github.com/sclark39/NS2Plus/commit/25ce33dcda3ff16097061f0bff9219820600afe4: With the legacy weapon switch option disabled, pressing the key for second weapon slot (2 by default) will switch to metabolize briefly, then return to your previous weapon. (With the legacy option enabled, it seems to work ok.)

GhoulofGSG9 commented 6 years ago

Comment by GhoulofGSG9 Sunday Apr 08, 2018 at 10:20 GMT

You can't switch to metabolize. The client prediction just thinks you can do, so if you press 2 the HUD will show for a few frames that you are switching to metabolize while you are not really switching to it at all. To fully fix this visual glitch I still need to figure out a way to propagate the option's value to the predict world.

I tried a client -> server -> predict network message pipeline but for whatever reason the network message hook doesn't trigger in predict.

Of course we could read the options.xml directly while running the prediction but that wouldn't detect any change of the option's value without an unreasonable amount of file sampling.