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

feat: InitChainer auto stakes uniformly to validators at genesis #26

Merged
merged 4 commits into from
Feb 26, 2024

Conversation

tbruyelle
Copy link
Collaborator

@tbruyelle tbruyelle commented Feb 26, 2024

Stake automatically 50% of balance for accounts that have more than 25 $GOVGEN at genesis initialization. The resulting stake distribution will provide approximately the same voting power to all genesis validators. Accounts will automatically stake to 5 validators if balance is less than 500 $GOVGEN, 10 validators if balance is less than 10000 $GOVGEN and 20 validators if more, uniformly.

Copy link
Collaborator

@giunatale giunatale left a comment

Choose a reason for hiding this comment

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

let's make sure we add documentation on readme and changelog, but otherwise looks good!

we already tested succesfully on a localnet manually, so as soon as docs are ready I'm gonna merge and release for the sake of time, but let's also work on proper testing to add to the code later

README.md Outdated
@@ -14,3 +14,4 @@ The following modifications have been made to the Cosmos Hub software to create
7. Removed ability for validators to vote on proposals with delegations, they can only use their own stake
8. Removed community spend proposal
9. Allowed setting different voting periods for different proposal types
10. Auto stake genesis accounts
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we expand here describing better what the algorithm does?

also le'ts add the same description on the PR text

@giunatale giunatale changed the title feat: InitChainer auto stakes genesis accounts feat: InitChainer auto stakes from genesis accounts uniformly to validators Feb 26, 2024
@giunatale giunatale changed the title feat: InitChainer auto stakes from genesis accounts uniformly to validators feat: InitChainer auto stakes uniformly to validators at genesis Feb 26, 2024
README.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@giunatale giunatale merged commit 3c40c31 into main Feb 26, 2024
4 of 5 checks passed
@giunatale
Copy link
Collaborator

We will need proper tests @tbruyelle

@tbruyelle
Copy link
Collaborator Author

We will need proper tests @tbruyelle

working on it!

@tbruyelle tbruyelle deleted the tbruyelle/feat/initial-staking-distr branch February 27, 2024 09:03
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