-
Notifications
You must be signed in to change notification settings - Fork 210
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
[Merged by Bors] - tortoise: don't use active set to compute number of expected ballots #5002
Conversation
88e9006
to
60cab4b
Compare
Codecov Report
@@ Coverage Diff @@
## develop #5002 +/- ##
=======================================
Coverage 77.1% 77.2%
=======================================
Files 254 254
Lines 30385 30351 -34
=======================================
- Hits 23443 23432 -11
+ Misses 5422 5399 -23
Partials 1520 1520
|
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.
if you want, you can also just delete the hare active set grading all together and remove legacy layer.
probably not in this change, but will do if i will be touching that code |
bors merge |
Pull request successfully merged into develop. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
closes: #4841
with this change active set is used only to verify eligibilities in the current epoch.
tortoise doesn't need to load activeset into the state, which is reflected in the accepted type