-
Notifications
You must be signed in to change notification settings - Fork 14
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(singularity): add singularity to mint and evmstaking module #202
base: main
Are you sure you want to change the base?
Conversation
9a07d05
to
d28dc13
Compare
d28dc13
to
9cd54f4
Compare
client/x/mint/types/params.go
Outdated
@@ -26,7 +27,8 @@ func DefaultParams() Params { | |||
return Params{ | |||
MintDenom: sdk.DefaultBondDenom, | |||
InflationsPerYear: math.LegacyNewDec(24625000000000000.000000000000000000), | |||
BlocksPerYear: uint64(60 * 60 * 8766 / 5), // assuming 5 second block times | |||
BlocksPerYear: uint64(60 * 60 * 8766 / 5), // assuming 5 seconds block times | |||
SingularityHeight: 725760, // 42 days with assuming 5 seconds block time |
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.
Should we combine this with the other SingularityHeight in evmstaking to make it consistent?
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 possible, it would be better to combine into one, but I have no idea on combining this. Or one way for this is that we make a new keeper named singularity
and import the keeper to both mint
and evmstaking
keeper. I think there is no need to make a new keeper for this, thus make like this. Do you have any better idea on this?
a82de33
to
3af9f8c
Compare
3af9f8c
to
db61453
Compare
Added singularity to mint and evmstaking module.
Until singularity height, no reward is minted and no validator set is updated.
BTW, there is one thing to discuss. What if user make requests for unstake greater than
MaxEntries
? In CL side, when there are more unstake requests thanMaxEntries
, it is processed as processing failure. However, in EL, unstake is processed normally, so I think we need to handle this. What do you think? I would like to discuss this because as the Singularity period gets longer, this possibility will increase.issue: none