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

Better estimate staking rewards after Damask #596

Merged
merged 4 commits into from
Nov 9, 2023
Merged

Conversation

lukaw3d
Copy link
Member

@lukaw3d lukaw3d commented Nov 2, 2023

Current chart estimates https://docs.oasis.io/general/oasis-network/token-metrics-and-distribution/#staking-incentives we are at epoch 27138, but we are already at 27359.

This estimates epoch duration differently for epochs made since -1 year ago.
Note: this doesn't only affect epoch to date calculation. It also affects APY calculation: how many epochs fit in one year.

Before After

image

image

DEV: estimatedEpochsPerHour 1.003339695445333
DEV: diff between estimates and known dates
at beta (0): +14.1 hours
at mainnet (1170): +0 hours
at block4M (6664): +9 hours
at block5M (8328): +5.8 hours
at block6M (9992): -1.3 hours
at block7M (11656): +2 hours
at block8M (13320): +17.3 hours
at block9M (14987): +5.4 hours
at block10M (16653): -6 hours
at block11M (18320): -22.2 hours
at block12M (19987): -47.3 hours
at block13M (21653): -73 hours
at block14M (23320): -110.2 hours
at block15M (24987): -148.5 hours
at block16M (26653): -186 hours
DEV: estimatedEpochsPerHourPreDamask 1.003339695445333
DEV: estimatedEpochsPerHourAfterDamask 1.0252064282899003
DEV: diff between estimates and known dates
at beta (0): +14.1 hours
at mainnet (1170): +0 hours
at block4M (6664): +9 hours
at block5M (8328): +5.8 hours
at block6M (9992): -1.3 hours
at block7M (11656): +2 hours
at block8M (13320): +17.3 hours
at block9M (14987): +5.4 hours
at block10M (16653): -6 hours
at block11M (18320): -22.2 hours
at block12M (19987): +0 hours
at block13M (21653): +9.7 hours
at block14M (23320): +7.9 hours
at block15M (24987): +5 hours
at block16M (26653): +3 hours

@lukaw3d lukaw3d requested review from matevz, tjanez and amela November 2, 2023 21:29
Copy link

netlify bot commented Nov 2, 2023

Deploy Preview for trusting-archimedes-14c863 ready!

Name Link
🔨 Latest commit d92165a
🔍 Latest deploy log https://app.netlify.com/sites/trusting-archimedes-14c863/deploys/654cd10601237d000860a222
😎 Deploy Preview https://deploy-preview-596--trusting-archimedes-14c863.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@lukaw3d lukaw3d force-pushed the lw/post-damask-estimates branch from fcdcf9d to d92165a Compare November 9, 2023 12:31
@lukaw3d lukaw3d enabled auto-merge November 9, 2023 12:32
@lukaw3d lukaw3d merged commit c9e12d0 into main Nov 9, 2023
2 checks passed
@lukaw3d lukaw3d deleted the lw/post-damask-estimates branch November 9, 2023 12:33
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.

2 participants