Skip to content

Conversation

@cce
Copy link
Contributor

@cce cce commented Oct 17, 2025

Summary

This provides benchmark code and algodump tooling for capturing network traffic and running compression on it with different window sizes, used during the development of #6351 and #6276.

Test Plan

Tools-only change.

algorandskiy
algorandskiy previously approved these changes Oct 17, 2025
jannotti
jannotti previously approved these changes Oct 21, 2025
Copy link
Contributor

@jannotti jannotti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess! That's a lot of code. I guess it benchmarks?

I don't see much reason not to approve a tooling addition.

@cce cce dismissed stale reviews from jannotti and algorandskiy via 4c9fd51 October 24, 2025 17:17
@cce
Copy link
Contributor Author

cce commented Oct 24, 2025

@jannotti removed/shrunk some things and it is now less code

@cce cce marked this pull request as draft October 24, 2025 17:20
@codecov
Copy link

codecov bot commented Oct 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.95%. Comparing base (1625da7) to head (4c9fd51).
⚠️ Report is 5 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6467      +/-   ##
==========================================
- Coverage   51.13%   46.95%   -4.19%     
==========================================
  Files         668      659       -9     
  Lines      112101    88652   -23449     
==========================================
- Hits        57327    41623   -15704     
+ Misses      51907    44276    -7631     
+ Partials     2867     2753     -114     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

jannotti
jannotti previously approved these changes Oct 24, 2025
@cce cce force-pushed the vpack-dynamic-bench branch from f0045e9 to 4eae592 Compare October 24, 2025 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants