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

Powerup Model #83

Merged
merged 1 commit into from
Feb 7, 2024
Merged

Powerup Model #83

merged 1 commit into from
Feb 7, 2024

Conversation

quocle108
Copy link
Contributor

@quocle108 quocle108 commented Oct 29, 2023

Change Description

This PR creates a new optional NET and CPU rental market which displaces (over time) the existing staking system. Under the old model, system token holders own NET and CPU and may choose to use it themselves, delegate it to others. Under this new model, the chain owns almost all NET and CPU resources and the only way to access these resources is
through the new powerup action. It channels fees to the eosio.saving to enable token holders to profit off the new market.

More information can be found here

Deployment Changes

  • Deployment Changes
    A new account eosio.reserv must be created before starting the new resource system (i.e. before calling the cfgpowerup action). The eosio.reserv account should not be used as the authorizer of any actions because increasing the CPU/NET usage of that account could interfere with the proper functioning of the cfgpowerup and powerup actions.

API Changes

  • API Changes
    New actions: cfgpowerup, powerupexec, powerup

Documentation Additions

  • Documentation Additions
    Documentation can be found here

@quocle108 quocle108 self-assigned this Oct 29, 2023
Copy link
Contributor

@porkchop porkchop left a comment

Choose a reason for hiding this comment

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

Yeah looks right @quocle108 . Just one nitpick for you. Good work

@porkchop porkchop self-requested a review November 1, 2023 00:38
porkchop
porkchop previously approved these changes Nov 1, 2023
Copy link
Contributor

@porkchop porkchop left a comment

Choose a reason for hiding this comment

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

LGTM!!

* tokenomics explainer
* adjust chain inflation
* burn excess inflation
Copy link
Contributor

@porkchop porkchop left a comment

Choose a reason for hiding this comment

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

DEPLOYED

@porkchop porkchop merged commit c97a9d9 into develop Feb 7, 2024
1 check 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.

2 participants