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

mint #165

Merged
merged 27 commits into from
Jan 10, 2025
Merged

mint #165

merged 27 commits into from
Jan 10, 2025

Conversation

kimurayu45z
Copy link
Contributor

@kimurayu45z kimurayu45z commented Jan 4, 2025

Overview

Before

  • Only vRISE will be minted
  • Inflated vRISE will be sent to fee collector module account

After

  • RISE will be minted for validator rewards
  • vRISE will be minted for LP rewards
  • The ratio of RISE and vRISE to mint will be determined with a parameter of liquidityincentive module
  • Minted vRISE will be withdrew by liquidityincentive module from fee collector module account, before distribution module distributes all balances (including RISE) of fee collector module account. It means that vRISE will not be distributed for validator rewards

@kimurayu45z kimurayu45z linked an issue Jan 4, 2025 that may be closed by this pull request
kimurayu45z and others added 19 commits January 5, 2025 09:28
@Senna46
Copy link
Contributor

Senna46 commented Jan 8, 2025

Both RISE and vRISE are accumulated in the distribution account. (RISE < vRISE)
need to verify that this distribution is correct.

$ sunrised q bank balances sunrise1jv65s3grqf6v6jl3dp4t6c9t9rk99cd84j0dcv
balances:
- amount: "7105"
  denom: urise
- amount: "7118"
  denom: uvrise
pagination:
  total: "2"

In addition, vRISE is accumulated in protocolpool and RISE is accumulated in protocolpool_distr little by little.

$ sunrised q bank balances sunrise1purnwdrhg477r0evks8t5ah7c8thvrsssfmyus
balances:
- amount: "211"
  denom: uvrise
pagination:
  total: "1"
$ sunrised q bank balances sunrise1rjpt2lknns396yue9f72r0khalqj8rcy8wtqdy
balances:
- amount: "142"
  denom: urise
pagination:
  total: "1"

@Senna46
Copy link
Contributor

Senna46 commented Jan 10, 2025

I confirmed that there is no problem with inflation.go and mint.go.
Merge to v0.52.

@Senna46 Senna46 merged commit d899b31 into v0.52 Jan 10, 2025
@Senna46 Senna46 deleted the v0.52-inflation-v2 branch January 10, 2025 15:12
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.

custom inflation v2
2 participants