-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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(fip0081): initialPledge calculation & ramp, plus migration tests #12526
Conversation
3832644
to
72cdd3f
Compare
@ZenGround0 I'm having a bit of trouble testing pledge in here. I can observe the change in pledge value if I log values of internal calculation during the ramp period: lotus/itests/migration_test.go Lines 974 to 986 in 72cdd3f
But the value I get is capped by the Consistently the cap ends up being ~66% of the uncapped pledge. Got any suggestions for reducing this? I'm guessing it's to do with baseline power, but I'm actually not sure. How can we budge the state-sourced pledge inputs in an itest? |
btw |
a4eb091
to
0bba2ea
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the PR title to match https://github.com/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions
PR title now matches the required format.
Thanks for the nice analysis I'll take a look at the code. If you wanted to think more about avoiding the cap resetting thing the levers you can play with are
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall LGTM I'll do a final review once all dependencies are landed before approving
Removed my commit with the custom actors build and added the tagged build from https://github.com/filecoin-project/builtin-actors/releases/tag/v15.0.0-rc1; also updated go-state-types to |
Initial draft, need to add some tests.
Depends on:
Also has #12525 because it was in my path to this.