Networking

[vladbat00]

• Implement spawning players and objects
• Implement interpolation
• Fix syncing positions
• Ignore outdated updates (avoid crashing)
• Allow losing unacknowledged packets
• Implement disconnecting clients on the server side and despawning
• Properly close connections on forcible disconnect (blocked by Ability to manually close a connection smokku/bevy_networking_turbulence#17)
• Implement pausing clients if there are no updates from server (avoid crashing)
• Implement reconnecting on the client side
• Investigate panics:

  • Inserting for a frame {} would remove future history (start_frame: {}, limit: {}, len: {})

      3: mr_shared_lib::net::ConnectionState::acknowledge_incoming
                at .\libs\shared_lib\src\net.rs:223
      4: mr_client_lib::net::process_network_events
                at .\libs\client_lib\src\net.rs:174
    

  • Expected player (entity: {:?}) direction for frame {}

      2: mr_shared_lib::game::movement::player_movement::{{closure}}
                at .\libs\shared_lib\src\game\movement.rs:169
      3: core::option::Option<tuple<mr_shared_lib::wrapped_counter::WrappedCounter<u16>, glam::vec2::Vec2*>>::unwrap_or_else<tuple<mr_shared_lib::wrapped_counter::WrappedCounter<u16>, glam::vec2::Vec2*>,closure-2>
                at C:\Users\mvlabat\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\core\src\option.rs:427
      4: mr_shared_lib::game::movement::player_movement
                at .\libs\shared_lib\src\game\movement.rs:166
    

• Fix webrtc connectivity
• Explore reconciliation improvements
• Fix handshaking when reconnecting
• Remove link conditioner
• Fix counting absolute frames diffs
• Run tests in the CI

Known issues to fix later:

• Sometimes a player won't spawn
• Sometimes a player will receive a name of another player
• Positions can still desync
• Sometimes a player name isn't received
• Counting packet loss can be improved: it shouldn't depend on message order. Clients never resend updates to the server (they may only enable redundancy).