Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Companion for #13983 #7140

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

gpestana
Copy link
Contributor

@gpestana gpestana added A3-in_progress Pull request is in progress. No review needed at this stage. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. C3-medium PR touches the given topic and has a medium impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. T1-runtime This PR/Issue is related to the topic “runtime”. labels Apr 27, 2023
@gpestana gpestana self-assigned this Apr 27, 2023
@gpestana gpestana marked this pull request as draft April 27, 2023 09:38
@paritytech-ci paritytech-ci requested review from a team April 27, 2023 09:38
@gpestana gpestana requested review from kianenigma and Ank4n and removed request for a team April 27, 2023 09:38
@gpestana gpestana marked this pull request as ready for review April 27, 2023 10:30
@paritytech-ci paritytech-ci requested review from a team April 27, 2023 10:31
@gpestana
Copy link
Contributor Author

bot rebase

@paritytech-processbot
Copy link

Rebased

@gpestana gpestana removed the C3-medium PR touches the given topic and has a medium impact on builders. label May 17, 2023
Copy link
Contributor

@kianenigma kianenigma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be good to post some example of the deposit about for a typical solution based on various queue sizes.

@paritytech-ci paritytech-ci requested a review from a team June 10, 2023 07:06
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2972157

@gpestana
Copy link
Contributor Author

gpestana commented Jun 30, 2023

In Kusama, the progression with 10% increase without changing the current signed fixed deposit is: (term == size of the queue)

Term 1: 1,333,333,332,000
Term 2: 1,333,333,332,000 * 1.10 = 1,466,666,665,200
Term 3: 1,333,333,332,000 * 1.10^2 = 1,613,333,331,200
Term 4: 1,333,333,332,000 * 1.10^3 = 1,774,666,664,320
Term 5: 1,333,333,332,000 * 1.10^4 = 1,952,133,330,752
Term 6: 1,333,333,332,000 * 1.10^5 = 2,147,346,663,827.20
Term 7: 1,333,333,332,000 * 1.10^6 = 2,362,081,330,210.92
Term 8: 1,333,333,332,000 * 1.10^7 = 2,598,289,463,231.01
Term 9: 1,333,333,332,000 * 1.10^8 = 2,858,118,409,554.11
Term 10: 1,333,333,332,000 * 1.10^9 = 3,143,930,250,509.52

Westend:

Term 1: 2,000,000,000,000
Term 2: 2,000,000,000,000 * 1.10 = 2,200,000,000,000
Term 3: 2,000,000,000,000 * 1.10^2 = 2,420,000,000,000
Term 4: 2,000,000,000,000 * 1.10^3 = 2,662,000,000,000
Term 5: 2,000,000,000,000 * 1.10^4 = 2,928,200,000,000
Term 6: 2,000,000,000,000 * 1.10^5 = 3,221,020,000,000
Term 7: 2,000,000,000,000 * 1.10^6 = 3,543,122,000,000
Term 8: 2,000,000,000,000 * 1.10^7 = 3,897,434,200,000
Term 9: 2,000,000,000,000 * 1.10^8 = 4,287,177,620,000
Term 10: 2,000,000,000,000 * 1.10^9 = 4,715,895,382,000

and in Polkadot, the deposit increase is disabled in the current state of the PR, as the increase factor is 0% -- so nothing changes from the current behaviour.

@jakoblell what do you think about this progression?

@paritytech-ci paritytech-ci requested a review from a team July 10, 2023 14:05
@jakoblell jakoblell added D1-audited 👍 PR contains changes to critical 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 Aug 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A3-in_progress Pull request is in progress. No review needed at this stage. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D1-audited 👍 PR contains changes to critical logic that has been properly reviewed and externally audited. T1-runtime This PR/Issue is related to the topic “runtime”.
Projects
Status: ✂️ In progress.
Development

Successfully merging this pull request may close these issues.

5 participants