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

Integrate economic model into StakingRewardsProvider trait implementation #1747

Open
1 of 2 tasks
shannonwells opened this issue Oct 30, 2023 · 0 comments
Open
1 of 2 tasks

Comments

@shannonwells
Copy link
Collaborator

shannonwells commented Oct 30, 2023

Now that an economic model (EM) has more or less been decided, use the values and caps described in the EM doc to calculate reward pool size and individual staking rewards.

Acceptance criteria:

  1. update reward_pool_size and staking_reward_total to return correct values as described in the EM doc by integrating the values and caps in the document.
  2. Tests that prove this using the examples in the document as well as any other needed test cases that exercise the model's limits
  3. Change the trait and implementation of StakingRewardsProvider: remove 'from_eraparameter fromstaking_reward_total`, since we should always pay out the oldest staking rewards.
  4. Tests for same

Possibly: Update/create (it's currently in a branch and may be out of date enough to start over) the Design Doc for the staking rewards economic model

Related/includes:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants