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

Polkadot: Make the current inflation formula adjustable #443

Merged

Conversation

kianenigma
Copy link
Contributor

@kianenigma kianenigma commented Aug 23, 2024

Counterpart of #364.

Similarly, it has no impact on the inflation as-is, and just exposes the params to the Root track.

UI Consideration

Similarly, it adds Inflation_experimental_inflation_info Runtime API, which can be used by UIs to fetch information about the inflation, without needing to rely on storage items.

Example

  • With current parameters:
Inflation ==> staking = 402244.054 DOT (4,022,440,540,915,948) / leftover = 4178.285 DOT (41,782,856,835,356)
  • With UseLegacyAuctions = false, which means we no longer set aside around 15% of tokens for auctions. This makes the difference between actual staked and ideal staking rate (staking inefficiency) grow, and therefore reduce the staking reward and increase the treasury income.
Inflation ==> staking = 340780.189 DOT (3,407,801,892,871,273) / leftover = 65642.150 DOT (656,421,504,843,446)

relay/polkadot/Cargo.toml Outdated Show resolved Hide resolved
relay/polkadot/Cargo.toml Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
relay/polkadot/src/lib.rs Outdated Show resolved Hide resolved
relay/polkadot/src/lib.rs Outdated Show resolved Hide resolved
kianenigma and others added 7 commits September 12, 2024 12:19
Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Co-authored-by: Gonçalo Pestana <g6pestana@gmail.com>
Co-authored-by: Gonçalo Pestana <g6pestana@gmail.com>
Co-authored-by: Ankan <10196091+Ank4n@users.noreply.github.com>
Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Co-authored-by: Gonçalo Pestana <g6pestana@gmail.com>
relay/polkadot/src/lib.rs Outdated Show resolved Hide resolved
relay/kusama/src/lib.rs Outdated Show resolved Hide resolved
@kianenigma
Copy link
Contributor Author

/merge

@fellowship-merge-bot
Copy link
Contributor

There was a problem running the action.

❌😵❌

Please find more information in the logs.

@fellowship-merge-bot fellowship-merge-bot bot enabled auto-merge (squash) October 1, 2024 11:27
@fellowship-merge-bot
Copy link
Contributor

Enabled auto-merge in Pull Request

Available commands
  • /merge: Enables auto-merge for Pull Request
  • /merge cancel: Cancels auto-merge for Pull Request
  • /merge help: Shows this menu

For more information see the documentation

@ggwpez
Copy link
Member

ggwpez commented Oct 1, 2024

@Bullrich ^

@Bullrich
Copy link
Contributor

Bullrich commented Oct 1, 2024

/merge help

@fellowship-merge-bot
Copy link
Contributor

Auto-Merge-Bot

User @Bullrich is not the author of the PR and does not publicly belong to the org polkadot-fellows.

Only author or public org members can trigger the bot.

@Bullrich
Copy link
Contributor

Bullrich commented Oct 1, 2024

@ggwpez it seems it was a one time problem, as the method worked with my comment:
image

@fellowship-merge-bot fellowship-merge-bot bot merged commit 646031d into polkadot-fellows:main Oct 1, 2024
38 of 48 checks passed
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.

6 participants