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

Adds MaximumMultiplier as upper bound for dynamic fee multiplier #1853

Merged
merged 7 commits into from
Oct 10, 2022

Conversation

notlesh
Copy link
Contributor

@notlesh notlesh commented Oct 6, 2022

What does it do?

Cherry-picks paritytech/substrate#12282 into our moonbeam-polkadot-v0.9.29 Substrate branch and adds a MaximumMultiplier value to bound the multiplier in pallet transaction-payment.

I picked an initial value of 1000 based on analysis from @nbaztec and the desire to have a value that will be "painful" but not impossibly expensive. It is certainly up for debate. IMO it should be high enough that it doesn't interfere with the normal multiplier algorithm and acts as a safety net.

The value has been set to 100_000 after discussion. This value is very high, but would prevent txns from being "impossibly expensive", thus acting as a safety net rather than something that would frequently interfere with the fee mechanism.

@notlesh notlesh changed the title Notlesh cherry pick max fee multiplier Add MaximumMultiplier as upper bound for dynamic fee multiplier Oct 6, 2022
@notlesh notlesh added A0-pleasereview Pull request needs code review. B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D5-nicetohaveaudit⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Oct 6, 2022
@librelois librelois mentioned this pull request Oct 10, 2022
24 tasks
@notlesh notlesh merged commit 131f076 into master Oct 10, 2022
@notlesh notlesh deleted the notlesh-cherry-pick-max-fee-multiplier branch October 10, 2022 16:27
@crystalin crystalin changed the title Add MaximumMultiplier as upper bound for dynamic fee multiplier Adds MaximumMultiplier as upper bound for dynamic fee multiplier Oct 20, 2022
@notlesh notlesh added D1-audited👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited and removed D5-nicetohaveaudit⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Feb 22, 2023
imstar15 pushed a commit to AvaProtocol/moonbeam that referenced this pull request May 16, 2023
…onbeam-foundation#1853)

* Bump substrate

* Bump polkadot

* Oops, bump the right repo (fix substrate and polkadot repos)

* Bump polkadot

* Add MaximumMultiplier

* fmt

* Bump max fee to 100_000
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A0-pleasereview Pull request needs code review. B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D1-audited👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants