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.

Sign up for GitHub

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

Jump to bottom

feat: introduce reward_per_wrapped to cap annuity rewards based on th… #622

Merged
merged 3 commits into from
Jun 10, 2022
Merged

feat: introduce reward_per_wrapped to cap annuity rewards based on th… #622

merged 3 commits into from
Jun 10, 2022

Conversation

gregdhill
Copy link
Member

…e total wrapped issuance

Signed-off-by: Gregory Hill gregorydhill@outlook.com

@gregdhill
feat: introduce reward_per_wrapped to cap annuity rewards based on th…
c5d0a4f 
…e total wrapped issuance

Signed-off-by: Gregory Hill <gregorydhill@outlook.com>
@nud3l
Copy link
Member

nud3l commented Jun 7, 2022

@gregdhill could you do a quick summary of the logic this code change implements?

@gregdhill
Copy link
Member Author

gregdhill commented Jun 7, 2022 •
edited
Loading

@gregdhill could you do a quick summary of the logic this code change implements?

Sure, so if RewardPerWrapped is set in the annuity pallet we will distribute min(reward_per_block, reward_per_wrapped * TotalWrapped::get()) where reward_per_block is the total amount of possible rewards (as currently distributed for KINT for example) and TotalWrapped is the issuance of kBTC / iBTC.

gregdhill added 2 commits June 10, 2022 10:47
@gregdhill
refactor: annuity tests shouldn't use magic constants
2fd957d 
Signed-off-by: Gregory Hill <gregorydhill@outlook.com>
@gregdhill
test: integration of TotalWrapped via rewards_per_wrapped
87479ee 
Signed-off-by: Gregory Hill <gregorydhill@outlook.com>
@gregdhill gregdhill merged commit 3028edd into interlay:master Jun 10, 2022
@gregdhill gregdhill deleted the feat/reward-per-wrapped branch June 10, 2022 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sander2 sander2 sander2 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gregdhill @nud3l @sander2