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

Add governance transaction for epoch length adjustment #312

Conversation

jordanschalm
Copy link
Member

@jordanschalm jordanschalm commented Aug 20, 2024

This service account transaction updates the epoch length to increase the block rate,
taking advantage of performance improvements. It also enables FLIP204 - Smart-Contract-Specified Epoch Switchover Timing on Mainnet.

Currently, Mainnet runs at ~0.8 blocks/second. After this change, it will run at 1.25 blocks/second.

Further details in the README in the diff.

@jordanschalm jordanschalm changed the title add epoch length updates for Mainnet25 - Sep 4 2024 Add governance transaction for epoch length adjustment Aug 20, 2024

The DKG phase length is currently 2000 views on Mainnet.
The consensus block rate is increasing by (1.25-0.8)/(0.8)=0.56 -> 56%.
In order to maintain a similar wall-clock time, we increase the DKG phase length by 50%, and set:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should it not increase by 56% instead of 50%?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There isn't a "correct" duration for the DKG, so we don't need to be exact here. We don't want the wallclock time to decrease by 50%, but it doesn't need to be exactly the same. That's why we pick round numbers near (2000)(1.56).

@bluesign
Copy link
Collaborator

Currently, Mainnet runs at ~0.8 blocks/second. After this change, it will run at 1.25 blocks/second.

why ? ( not that I disagree ) but I think README should explain why we want 1.25 instead of 0.8.

@jordanschalm jordanschalm changed the base branch from main to vishal/sept_4/josh/cadence-1.0 September 3, 2024 20:06
@jordanschalm jordanschalm changed the base branch from vishal/sept_4/josh/cadence-1.0 to main September 3, 2024 20:06
Co-authored-by: Alexander Hentschel <[email protected]>
@jordanschalm
Copy link
Member Author

Closing as the transactions have been copied to #318, which includes all governance transactions associated with Crescendo.

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.

5 participants