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

HIP36 proposal: use blockheights instead of time #257

Merged
merged 6 commits into from
Aug 12, 2021
Merged

Conversation

Bx64
Copy link
Contributor

@Bx64 Bx64 commented Aug 7, 2021

At the time of writing, most (if not all) blockchain variables and implementations are activated manually by the Helium team, requiring active intervention from the team and have a time-based activation. The active intervention can, for the greatest part, be avoided by enabling auto-activation of variables at certain blockheights. This has been the standard for most blockchain projects. One example is the bi-yearly halving of HNT rewards agreed to in HIP20 which should be integrated with blockheights rather than be defined by moment in time for its activation - and in the future, hopefully any change of blockchain variables can be defined that way.

This HIP does not contain a complete schedule for HIP20, but does contain all variables needed to properly create the schedule (last halving at block 964732 and increments of 1051200 blocks per halving).

Rendered view: https://github.com/Bx64/HIP/blob/master/0036-blockheights-instead-of-time.md

@jamiew
Copy link
Contributor

jamiew commented Aug 10, 2021

Thanks for your submission, will review!

@jamiew jamiew changed the title HIP36 proposal: use blockheights instead of time HIPxx proposal: use blockheights instead of time Aug 10, 2021
@jamiew jamiew added the draft label Aug 10, 2021
@jamiew jamiew changed the title HIPxx proposal: use blockheights instead of time HIP36 proposal: use blockheights instead of time Aug 12, 2021
@jamiew
Copy link
Contributor

jamiew commented Aug 12, 2021

Numbered and merging. Please tag tracking issue #260 for future PRs. For anyone with questions or comments on this HIP, please also use the tracking issue #260

@jamiew jamiew merged commit 9433233 into helium:master Aug 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants