Skip to content

HIP36 proposal: use blockheights instead of time#257

Merged
jamiew merged 6 commits intohelium:masterfrom
Bx64:master
Aug 12, 2021
Merged

HIP36 proposal: use blockheights instead of time#257
jamiew merged 6 commits intohelium:masterfrom
Bx64:master

Conversation

@Bx64
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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