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

Apply inflationary rewards to voting power #939

Closed
Tracked by #340 ...
brentstone opened this issue Dec 20, 2022 · 4 comments
Closed
Tracked by #340 ...

Apply inflationary rewards to voting power #939

brentstone opened this issue Dec 20, 2022 · 4 comments
Assignees
Labels
PoS post-mainnet Don't worry about this yet. prio:low spec

Comments

@brentstone
Copy link
Collaborator

Need to decide how/when to apply inflationary rewards to voting powers. Voting power for consensus-participating validators and for validators and delegators participating in governance must consider inflationary rewards.

Current idea is for rewards to contribute to voting power at pipeline_len epochs after the rewards are distributed via the rewards products, as this is consistent with the current epoched staking model. Having rewards contribute to voting power immediately can cause issues for fixing a validator set in advance in order to run a DKG.

@tzemanovic
Copy link
Member

I'm in favor of pipeline_len offset for "auto-bonding" to behave like a bonding tx, as these can also affect validator sets (even if they are small changes)

@cwgoes
Copy link
Contributor

cwgoes commented Jan 2, 2023

Agreed with rewards contributing to voting power after pipeline_len epochs.

@tzemanovic
Copy link
Member

we won't be auto-bonding rewards for now

@tzemanovic tzemanovic added the post-mainnet Don't worry about this yet. label Oct 23, 2023
@cwgoes
Copy link
Contributor

cwgoes commented Aug 29, 2024

This is no longer planned.

@cwgoes cwgoes closed this as not planned Won't fix, can't repro, duplicate, stale Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PoS post-mainnet Don't worry about this yet. prio:low spec
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants