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

Update fork choice rule #571

Merged
merged 7 commits into from Feb 6, 2019

Conversation

Projects
None yet
4 participants
@vbuterin
Copy link
Contributor

vbuterin commented Feb 4, 2019

  • Use validator indices instead of validators
  • Weight each validator by int(balance) at the position of the start state instead of one weight unit regardless of balance (using int(balance) instead of balance to reduce the number of recalculation events implementers will have to deal with).

vbuterin and others added some commits Feb 4, 2019

Update fork choice rule
* Use validator indices instead of validators
* Weight each validator by `int(balance)` at the position of the start state instead of one weight unit regardless of balance (using `int(balance)` instead of balance to reduce the number of recalculation events implementers will have to deal with).
@djrtwo

djrtwo approved these changes Feb 6, 2019

Copy link
Contributor

djrtwo left a comment

Should use get_effective_balance. Otherwise looks good!

Show resolved Hide resolved specs/core/0_beacon-chain.md Outdated

djrtwo and others added some commits Feb 6, 2019

Update specs/core/0_beacon-chain.md
Co-Authored-By: vbuterin <v@buterin.com>

@JustinDrake JustinDrake merged commit 8f37c5c into dev Feb 6, 2019

@djrtwo djrtwo deleted the vbuterin-patch-3 branch Feb 8, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment