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

feat(slot-reservations): Add expansionRate parameter #179

Conversation

emizzle
Copy link
Collaborator

@emizzle emizzle commented Sep 20, 2024

Closes #115

This is part 2 of a longer list of tasks for slot reservations to prevent PRs from being too large.

Add expansionRate parameter, that indicates the percentage of total time before expiry that all addresses are eligible to reserve a slot. Total time is the duration between request creation and expiry. Higher number means faster expansion. Valid range is [0, 100], where 100 means all addresses will be eligible at request creation, and 0 indicates that all addresses will be eligible at the time of expiry.

This in preparation for a linear curve that describes the expanding window rate of expansion, as opposed to using two parameter families to describe a non-linear curve.

Add "expansion" parameter, that indicates the percentage of addresses eligible to reserve a slot halfway between request creation and when all network addresses are eligible to participate. Valid range = [1-100].
Add a parameter, `saturation` to the network-level configuration, under `SlotReservationConfig`. This parameter represents the percentage of total time before expiry that all addresses are eligible to reserve a slot. Total time is the duration between request creation and expiry. Valid range is [0, 99].

Note, this parameter is not yet used anywhere.
Remove saturation parameter, delta, and instead use expansionRate as part of the storage request, in preparation for a linear curve of the expanding window definition.
@emizzle emizzle changed the title feat(slot-reservations): Add expansionRate parameter feat(slot-reservations): Add expansionRate parameter Sep 22, 2024
@emizzle emizzle added the Marketplace See https://miro.com/app/board/uXjVNZ03E-c=/ for details label Sep 23, 2024
@emizzle emizzle assigned emizzle and unassigned emizzle Sep 23, 2024
@emizzle emizzle removed the Marketplace See https://miro.com/app/board/uXjVNZ03E-c=/ for details label Sep 23, 2024
@emizzle emizzle linked an issue Sep 23, 2024 that may be closed by this pull request
@emizzle
Copy link
Collaborator Author

emizzle commented Sep 24, 2024

Closing this for now as the expanding window is not essential for mainnet. Can resurrect this PR once the expanding window work is planned.

@emizzle emizzle closed this Sep 24, 2024
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.

Add disperal parameter
1 participant