Fixed multiplayer visual glitch with detached parts "teleporting" around the vehicle #3055
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ohlidalp
added a commit
to ohlidalp/rigs-of-rods
that referenced
this pull request
Jun 16, 2023
This is a spinoff from RigsOfRods#3055 Requires updated rorserver. New features: - Added send timestamp to `RoRnet::Header` - all packets now track their time shift between remote client and local client. Note that the existing truck time shift tracking made by Ulteq is unchanged for now. - Added queue timestamp to `RoRnet::Header` - all packets now track the delay between creating the packet and transmitting it to server. - Added recv queue timestamp to `NetRecvPacket` - all packets now track local delay between receiving packet and processing it. - Added 3 new graphs to the MPClientList UI: 1. timeshift (should be stable), 2. local recv lag, 3. remote send lag. Motivation: - to smoothen out character movement the same way truck movement is smoothed. That requires tracking the timeshift, and Ulteq's truck smoothing code is very much tied to `Actor` and `ActorManager`. - To clean up network timers - there are 3 separate timers (`Character`, `Actor`, `ActorManager` - the pattern is not clear). - To better understand and tweak network traffic. - I like diagnostic views. Time spent: 6.5hours, not counting rorserver.
|
NOTE TO SELF: Needs a rornet bump: https://discord.com/channels/136544456244461568/189904947649708032/1119932306036097047 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #2783
I replaced the Ulteq-invented
short intcompression format with a standard half-precision IEEE floats (using https://half.sourceforge.net/). Both are 2-byte formats so network traffic will remain the same.