-
Notifications
You must be signed in to change notification settings - Fork 592
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Only do one massive send for all gauge distribution #510
Conversation
Codecov Report
@@ Coverage Diff @@
## main #510 +/- ##
==========================================
+ Coverage 20.41% 20.44% +0.02%
==========================================
Files 163 163
Lines 23206 23185 -21
==========================================
+ Hits 4738 4740 +2
+ Misses 17694 17668 -26
- Partials 774 777 +3
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK, LGTM
Seems like a pretty straightforward improvement 👍
Awesome, ty! Will merge and test if its also mainnet compatible. |
* Only do one massive send for all gauge distribution * Fix all the tests
* Only do one massive send for all gauge distribution * Fix all the tests
Changes the distribution logic to only do one bank multi-send for all the gauges, which should significantly help in speed / reducing the number of events. This may be state compatible with mainnet. This needs to be tested.
This needs to update the tests, and add a changelog entry.