Reapply #2095 and fix reconnection error cause by it#2134
Reapply #2095 and fix reconnection error cause by it#2134NetsuNegi wants to merge 3 commits intoPhobos-developers:developfrom
Conversation
|
To Chinese users:
|
|
Nightly build for this pull request:
This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build. |
|
didn't desync in our test so far, and the previous issues are solved. More testing is appreciated tho |
|
I think we should find out exactly what processing was missing before that caused the desync. |
TaranDahl
left a comment
There was a problem hiding this comment.
The code lgtm. But the issue above should be addressed.
We can't even figure out what exactly caused the desync yet, but just take reference of other event codes and add missing checks as much as possible (e.g. planning node, tether etc). So sadly there's no clear document as of now |
The most important code to resolve this desync seems like |
No description provided.