Skip to content

Conversation

@xezon
Copy link

@xezon xezon commented Sep 29, 2025

This change is a follow up for #1528 and fixes the issue where tint color (and probably other visuals) did not work in Network matches at 30 render fps.

It happened because the Client update happens before Network update, and therefore the isStalling function always returned true (at 30 fps) and the client update skipped certain updates because it would think that the Network was stalling when it was not.

To fix this, we now detect the stalling state in the Network update function and save it for each frame.

TODO

  • Replicate in Generals

@xezon xezon added Bug Something is not working right, typically is user facing Minor Severity: Minor < Major < Critical < Blocker Network Anything related to network, servers ThisProject The issue was introduced by this project, or this task is specific to this project labels Sep 29, 2025
@xezon xezon added this to the Decouple logic and rendering milestone Sep 29, 2025
@xezon xezon force-pushed the xezon/rendering-1-network-stalling branch from 8b01517 to 12fd38a Compare September 30, 2025 17:09
@xezon
Copy link
Author

xezon commented Sep 30, 2025

Replicated in Generals without conflicts.

@xezon xezon merged commit e13efd4 into TheSuperHackers:main Sep 30, 2025
18 checks passed
@xezon xezon deleted the xezon/rendering-1-network-stalling branch September 30, 2025 17:24
fbraz3 pushed a commit to fbraz3/GeneralsX that referenced this pull request Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something is not working right, typically is user facing Minor Severity: Minor < Major < Critical < Blocker Network Anything related to network, servers ThisProject The issue was introduced by this project, or this task is specific to this project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants