Skip to content
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

Fix slashing parameter #137

Merged
merged 1 commit into from
Feb 25, 2023
Merged

Conversation

yuta0x89
Copy link
Contributor

Altair upgrade states MIN_SLASHING_PENALTY_QUOTIENT_ALTAIR is 2**6 (= 64).
In Ethereum Economic Model, the value of MIN_SLASHING_PENALTY_QUOTIENT in Mathematical_specification.md is 32 and 64 in the other files.

@yuta0x89
Copy link
Contributor Author

yuta0x89 commented Jan 20, 2023

The current Ethereum Economic Model is based on Altair (Blue Loop / v1.1.0-alpha.7 release), so if we wanted to establish the model on Bellatrix, we would need to change the value of penalty parameters such as MIN_SLASHING_PENALTY_QUOTIENT and PROPORTIONAL_SLASHING_MULTIPLIER.

Name Value in Altair Value in Bellatrix
MIN_SLASHING_PENALTY_QUOTIENT uint64(2**6) (= 64) uint64(2**5) (= 32)
PROPORTIONAL_SLASHING_MULTIPLIER uint64(2) uint64(3)

@BenSchZA
Copy link
Collaborator

Thanks for the contribution @uta0x89! I'll merge this PR, and I'm happy for you to open a new PR for the upgrade to Bellatrix - were there any other significant updates apart from the parameters above?

@BenSchZA BenSchZA merged commit dade179 into CADLabs:main Feb 25, 2023
@yuta0x89
Copy link
Contributor Author

Thank you for the merge!

I'm happy for you to open a new PR for the upgrade to Bellatrix

Of course, I will create a new PR for the upgrade to Bellatrix!

were there any other significant updates apart from the parameters above?

No, there weren’t. The reasons are as follows.


As described here, the Bellatrix upgrade added transaction execution and updated penalty parameters.

Since the Ethereum Economic Model models only the behavior of the consensus layer, not the execution layer, we can ignore transaction execution.
The updated penalty parameters are INACTIVITY_PENALTY_QUOTIENT, MIN_SLASHING_PENALTY_QUOTIENT, and PROPORTIONAL_SLASHING_MULTIPLIER. The current Ethereum Economic Model does not model the inactivity leak mechanism, so we can ignore INACTIVITY_PENALTY_QUOTIENT.

Therefore, we only need to update MIN_SLASHING_PENALTY_QUOTIENT and PROPORTIONAL_SLASHING_MULTIPLIER to upgrade the Ethereum Economic Model from Altair to Bellatrix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants