Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Add --warp-epoch and --enable-inflation to ledger-tool cap.#11107

Merged
ryoqun merged 5 commits intosolana-labs:masterfrom
ryoqun:inflation-forced-warped-capitalization
Jul 20, 2020
Merged

Add --warp-epoch and --enable-inflation to ledger-tool cap.#11107
ryoqun merged 5 commits intosolana-labs:masterfrom
ryoqun:inflation-forced-warped-capitalization

Conversation

@ryoqun
Copy link
Copy Markdown
Contributor

@ryoqun ryoqun commented Jul 17, 2020

Problem

It's hard/tedious to casually check what a realistic inflation on mainnet-beta will be as a proactive measure.

Summary of Changes

Introduce small toy functionality to do that practice run for inflation.

Complement with #10914.

@ryoqun ryoqun added the v1.2 label Jul 17, 2020
@ryoqun ryoqun requested a review from mvines July 17, 2020 05:50
@ryoqun
Copy link
Copy Markdown
Contributor Author

ryoqun commented Jul 17, 2020

Wow, it seems that #10914 might corrupt bank.capitalization() in some cases... I'm digging...

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 17, 2020

Codecov Report

Merging #11107 into master will decrease coverage by 0.0%.
The diff coverage is 25.6%.

@@            Coverage Diff            @@
##           master   #11107     +/-   ##
=========================================
- Coverage    82.1%    82.1%   -0.1%     
=========================================
  Files         319      319             
  Lines       73471    73504     +33     
=========================================
+ Hits        60371    60378      +7     
- Misses      13100    13126     +26     

mvines
mvines previously approved these changes Jul 17, 2020
Comment thread ledger-tool/src/main.rs Outdated
Comment thread ledger-tool/src/main.rs Outdated
Comment thread runtime/src/bank.rs Outdated
Comment thread runtime/src/bank.rs Outdated
Comment thread runtime/src/bank.rs Outdated
Comment thread runtime/src/bank.rs Outdated
Comment thread sdk/src/native_token.rs Outdated
Comment thread sdk/src/native_token.rs Outdated
Comment thread sdk/src/native_token.rs
@ryoqun ryoqun mentioned this pull request Jul 17, 2020
@mergify mergify Bot dismissed mvines’s stale review July 20, 2020 07:20

Pull request has been modified.

@ryoqun ryoqun force-pushed the inflation-forced-warped-capitalization branch from c1c21f5 to bc9ac35 Compare July 20, 2020 07:34
@ryoqun ryoqun merged commit 3db246f into solana-labs:master Jul 20, 2020
@ryoqun ryoqun changed the title Add --warp-epoch and --force-inflation to ledger-tool cap. Add --warp-epoch and --enable-inflation to ledger-tool cap. Jul 20, 2020
mergify Bot pushed a commit that referenced this pull request Jul 20, 2020
* Add --warp-epoch and --force-inflation to ledger-tool cap.

* Add more arguments

* Address review comments

* Fix message

* Fix various typos...

(cherry picked from commit 3db246f)

# Conflicts:
#	ledger-tool/src/main.rs
mergify Bot added a commit that referenced this pull request Jul 20, 2020
… (#11132)

* Add --warp-epoch and --force-inflation to ledger-tool cap. (#11107)

* Add --warp-epoch and --force-inflation to ledger-tool cap.

* Add more arguments

* Address review comments

* Fix message

* Fix various typos...

(cherry picked from commit 3db246f)

# Conflicts:
#	ledger-tool/src/main.rs

* Fix conflicts..

Co-authored-by: Ryo Onodera <ryoqun@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants