-
Notifications
You must be signed in to change notification settings - Fork 648
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
feat: fix chunk producer staking threshold #12660
Conversation
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.
LGTM, we just likely need to fix/remove tests depending on the old ratio.
// (reason of "approximately 6x": ./fix_stake_threshold.rs, but it's irrelevant here) | ||
// so test3's proposal will not be approved |
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.
Let's just say that 6 corresponds to the number of shards, especially if fix_stake_threshold is irrelevant.
c556314
to
34bf46e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #12660 +/- ##
==========================================
+ Coverage 70.49% 70.51% +0.02%
==========================================
Files 845 845
Lines 172247 172257 +10
Branches 172247 172257 +10
==========================================
+ Hits 121421 121465 +44
+ Misses 45727 45696 -31
+ Partials 5099 5096 -3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
In case of empty seats, the seat price of cp used to depend on the number of existing shards (approximately
block_producer_threshold / num_shard
), but it is no longer the case. Hence the fix.cc @Longarithm @wacban