Skip to content

Conversation

@Lmh-java
Copy link
Contributor

@Lmh-java Lmh-java commented Jan 22, 2026

Description

Part of #3549

Because we are sending lots of duplicated visualization protos over the network at a very fast rate, the system network buffer is not happy with this. This PR will introduce a sliding window deduper that controls the rate of sending duplicates (with less-to-no additional overhead) to mitigate the problem.

image

Testing Done

Resolved Issues

Length Justification and Key Files to Review

Review Checklist

It is the reviewers responsibility to also make sure every item here has been covered

  • Function & Class comments: All function definitions (usually in the .h file) should have a javadoc style comment at the start of them. For examples, see the functions defined in thunderbots/software/geom. Similarly, all classes should have an associated Javadoc comment explaining the purpose of the class.
  • Remove all commented out code
  • Remove extra print statements: for example, those just used for testing
  • Resolve all TODO's: All TODO (or similar) statements should either be completed or associated with a github issue

@Lmh-java Lmh-java changed the title Visualization Proto Deduplicators Visualization Proto Deduplicator Jan 22, 2026
@Lmh-java Lmh-java force-pushed the lmh/proto-deduper branch 2 times, most recently from 779826e to d317824 Compare January 24, 2026 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants