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

distributions: implement pool budget #5025

Closed
Tracked by #5010
erwanor opened this issue Jan 29, 2025 · 0 comments
Closed
Tracked by #5010

distributions: implement pool budget #5025

erwanor opened this issue Jan 29, 2025 · 0 comments
Assignees

Comments

@erwanor
Copy link
Member

erwanor commented Jan 29, 2025

The distribution component defines an EpochHandler extension trait, with an end_epoch method. This handler is triggered at the end of an epoch and executes the following logic:

  1. Compute the size of the tournament incentive pool for the ending epoch:
let epoch_duration = epoch.end - epoch.start;
let lqt_block_reward_rate = self.get_lqt_block_reward_rate();
let total_pool_size = lqt_block_reward_rate * epoch_duration;

The pool size expands or contract based on how long the epoch lasts. Since epochs are at most ~2 days, this gives a predictable upper bound on the total issued rewards. On the other hand, since the pool size contracts with epoch length, this means that there’s no upside to trying to make the chain churn epochs.

@erwanor erwanor mentioned this issue Jan 29, 2025
28 tasks
TalDerei added a commit that referenced this issue Jan 30, 2025
## Describe your changes

Implements the LQT rewards comprising the total reward pool size for an
epoch.

## Issue ticket number and link

references #5025

## Checklist before requesting a review

- [x] I have added guiding text to explain how a reviewer should test
these changes.

- [x] If this code contains consensus-breaking changes, I have added the
"consensus-breaking" label. Otherwise, I declare my belief that there
are not consensus-breaking changes, for the following reason:

  > adds new logic for computing LQT issuance
conorsch pushed a commit that referenced this issue Jan 31, 2025
## Describe your changes

Implements the LQT rewards comprising the total reward pool size for an
epoch.

## Issue ticket number and link

references #5025

## Checklist before requesting a review

- [x] I have added guiding text to explain how a reviewer should test
these changes.

- [x] If this code contains consensus-breaking changes, I have added the
"consensus-breaking" label. Otherwise, I declare my belief that there
are not consensus-breaking changes, for the following reason:

  > adds new logic for computing LQT issuance
conorsch pushed a commit that referenced this issue Feb 4, 2025
## Describe your changes

Implements the LQT rewards comprising the total reward pool size for an
epoch.

## Issue ticket number and link

references #5025

## Checklist before requesting a review

- [x] I have added guiding text to explain how a reviewer should test
these changes.

- [x] If this code contains consensus-breaking changes, I have added the
"consensus-breaking" label. Otherwise, I declare my belief that there
are not consensus-breaking changes, for the following reason:

  > adds new logic for computing LQT issuance
conorsch pushed a commit that referenced this issue Feb 5, 2025
## Describe your changes

Implements the LQT rewards comprising the total reward pool size for an
epoch.

## Issue ticket number and link

references #5025

## Checklist before requesting a review

- [x] I have added guiding text to explain how a reviewer should test
these changes.

- [x] If this code contains consensus-breaking changes, I have added the
"consensus-breaking" label. Otherwise, I declare my belief that there
are not consensus-breaking changes, for the following reason:

  > adds new logic for computing LQT issuance
conorsch pushed a commit that referenced this issue Feb 5, 2025
## Describe your changes

Implements the LQT rewards comprising the total reward pool size for an
epoch.

## Issue ticket number and link

references #5025

## Checklist before requesting a review

- [x] I have added guiding text to explain how a reviewer should test
these changes.

- [x] If this code contains consensus-breaking changes, I have added the
"consensus-breaking" label. Otherwise, I declare my belief that there
are not consensus-breaking changes, for the following reason:

  > adds new logic for computing LQT issuance
conorsch pushed a commit that referenced this issue Feb 14, 2025
## Describe your changes

Implements the LQT rewards comprising the total reward pool size for an
epoch.

## Issue ticket number and link

references #5025

## Checklist before requesting a review

- [x] I have added guiding text to explain how a reviewer should test
these changes.

- [x] If this code contains consensus-breaking changes, I have added the
"consensus-breaking" label. Otherwise, I declare my belief that there
are not consensus-breaking changes, for the following reason:

  > adds new logic for computing LQT issuance
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

No branches or pull requests

2 participants