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

chore: add comments to eip-1559 code #6818

Merged
merged 5 commits into from
Nov 3, 2023

Conversation

PaddyMc
Copy link
Collaborator

@PaddyMc PaddyMc commented Nov 3, 2023

Closes: #XXX

What is the purpose of the change

  • Adding some comments and removing unused files

@github-actions github-actions bot added C:docs Improvements or additions to documentation C:x/txfees labels Nov 3, 2023
Copy link
Member

@p0mvn p0mvn left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

Comment on lines 30 to 37
- DefaultBaseFee: Default base fee, initialized to 0.0025.
- MinBaseFee: Minimum base fee, initialized to 0.0025.
- MaxBaseFee: Maximum base fee, initialized to 10.
- TargetGas: Gas wanted per block, initialized to 60,000,000.
- MaxBlockChangeRate: The maximum block change rate, initialized to 1/16.
- ResetInterval: The interval at which eipState is reset, initialized to 1000 blocks.
- BackupFile: File for backup, set to "eip1559state.json".
- RecheckFeeConstant: A constant value for rechecking fees, initialized to 4.
Copy link
Member

Choose a reason for hiding this comment

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

Should any of these be exposed via config?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not sure maybe @ValarDragon could input here?

x/txfees/keeper/mempool-1559/code.go Outdated Show resolved Hide resolved
x/txfees/keeper/mempool-1559/code.go Show resolved Hide resolved
x/txfees/keeper/mempool-1559/code.go Outdated Show resolved Hide resolved
x/txfees/keeper/mempool-1559/code.go Outdated Show resolved Hide resolved
@p0mvn
Copy link
Member

p0mvn commented Nov 3, 2023

Going to merge this to start setting up a testnet from this commit on Dev's branch.

Adam is working on the configs in the meantime

@p0mvn p0mvn merged commit 83c9b13 into dev/mempool_1559 Nov 3, 2023
1 check passed
@p0mvn p0mvn deleted the paddy/add-comments-to-eip-1559-code branch November 3, 2023 16:21
ValarDragon added a commit that referenced this pull request Nov 3, 2023
* initial commit

* More skeleton code

* Is this good for testing?

* sync

* better comment

* fix

* Update

* add cli

* fix code

* ensure it doesn't go below min

* export eip struct

* Update smth

* update

* moire prints

* change target to 40M to see what happens

* reparam + config

* Nicolas/1559 persist (#6812)

* persisting to disk

* better prints after testing

* add max base fee

* cloning everywhere to be safe

* chore: add unit tests to eip code

* Add a recheck bound

* Cfg + adjust constants

* Make a GlobalMempool

* Ok testing

* remove log

* lint

* Apply @Pipello 's code suggestions

* chore: add comments to eip-1559 code (#6818)

* chore: add comments to eip-1559 code

* Update x/txfees/keeper/mempool-1559/code.go

Co-authored-by: Roman <[email protected]>

* Update x/txfees/keeper/mempool-1559/code.go

Co-authored-by: Roman <[email protected]>

* chore: add logger, use const instead of var

* chore: remove sleep in test

---------

Co-authored-by: Roman <[email protected]>

* add app.toml options for all vars

* Revert "add app.toml options for all vars"

This reverts commit 6fa54ba.

---------

Co-authored-by: Adam Tucker <[email protected]>
Co-authored-by: Nicolas Lara <[email protected]>
Co-authored-by: PaddyMc <[email protected]>
Co-authored-by: Roman <[email protected]>
ValarDragon added a commit that referenced this pull request Nov 10, 2023
* initial commit

* More skeleton code

* Is this good for testing?

* sync

* better comment

* fix

* Update

* add cli

* fix code

* ensure it doesn't go below min

* export eip struct

* Update smth

* update

* moire prints

* change target to 40M to see what happens

* reparam + config

* Nicolas/1559 persist (#6812)

* persisting to disk

* better prints after testing

* add max base fee

* cloning everywhere to be safe

* chore: add unit tests to eip code

* Add a recheck bound

* Cfg + adjust constants

* Make a GlobalMempool

* Ok testing

* remove log

* lint

* Apply @Pipello 's code suggestions

* chore: add comments to eip-1559 code (#6818)

* chore: add comments to eip-1559 code

* Update x/txfees/keeper/mempool-1559/code.go

Co-authored-by: Roman <[email protected]>

* Update x/txfees/keeper/mempool-1559/code.go

Co-authored-by: Roman <[email protected]>

* chore: add logger, use const instead of var

* chore: remove sleep in test

---------

Co-authored-by: Roman <[email protected]>

* add app.toml options for all vars

* Revert "add app.toml options for all vars"

This reverts commit 6fa54ba.

---------

Co-authored-by: Adam Tucker <[email protected]>
Co-authored-by: Nicolas Lara <[email protected]>
Co-authored-by: PaddyMc <[email protected]>
Co-authored-by: Roman <[email protected]>
ValarDragon added a commit that referenced this pull request Nov 10, 2023
* initial commit

* More skeleton code

* Is this good for testing?

* sync

* better comment

* fix

* Update

* add cli

* fix code

* ensure it doesn't go below min

* export eip struct

* Update smth

* update

* moire prints

* change target to 40M to see what happens

* reparam + config

* Nicolas/1559 persist (#6812)

* persisting to disk

* better prints after testing

* add max base fee

* cloning everywhere to be safe

* chore: add unit tests to eip code

* Add a recheck bound

* Cfg + adjust constants

* Make a GlobalMempool

* Ok testing

* remove log

* lint

* Apply @Pipello 's code suggestions

* chore: add comments to eip-1559 code (#6818)

* chore: add comments to eip-1559 code

* Update x/txfees/keeper/mempool-1559/code.go

Co-authored-by: Roman <[email protected]>

* Update x/txfees/keeper/mempool-1559/code.go

Co-authored-by: Roman <[email protected]>

* chore: add logger, use const instead of var

* chore: remove sleep in test

---------

Co-authored-by: Roman <[email protected]>

* add app.toml options for all vars

* Revert "add app.toml options for all vars"

This reverts commit 6fa54ba.

---------

Co-authored-by: Adam Tucker <[email protected]>
Co-authored-by: Nicolas Lara <[email protected]>
Co-authored-by: PaddyMc <[email protected]>
Co-authored-by: Roman <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:docs Improvements or additions to documentation C:x/txfees
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants