-
-
Notifications
You must be signed in to change notification settings - Fork 412
feat: update weak subjectivity calculation for electra #7735
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
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## unstable #7735 +/- ##
=========================================
Coverage 56.09% 56.09%
=========================================
Files 823 823
Lines 58036 58075 +39
Branches 4464 4467 +3
=========================================
+ Hits 32554 32580 +26
- Misses 25415 25428 +13
Partials 67 67 🚀 New features to boost your workflow:
|
Performance Report✔️ no performance regression detected Full benchmark results
|
nflaig
left a comment
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.
just for reference, this ethereum/consensus-specs#4179 is the actual PR
|
|
||
| export function computeWeakSubjectivityPeriodFromConstituentsElectra( | ||
| totalBalanceByIncrement: number, | ||
| balanceChurnLimit: number, // Note this is not the same as churnLimit in `computeWeakSubjectivityPeriodFromConstituentsPhase0` |
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.
the function signature is really not ideal, it would be better to get it closer to how the spec works but I can see why we do it like this as well
|
could merge this as is |
nflaig
left a comment
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
|
🎉 This PR is included in v1.31.0 🎉 |
Related spec
ethereum/consensus-specs#4243ethereum/consensus-specs#4179Closes #7645