-
Notifications
You must be signed in to change notification settings - Fork 170
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
Set pre-commit deposit independent of sector content - first draft #307
Conversation
Thanks @anorth this proposal generally looks good and considerations have been described in discussions. We might want to double-check how PCD is used/refunded when a sector is ProveCommitted during implementation in case of bugs. |
Notes from lotus team from implementation consideration:
|
Drop redundant deal loading for CommD during prove-commit. Note restoration of security buffers from SnapDeals.
4744751
to
27fd80c
Compare
Updated:
|
Proposal looks good to me. Increasing PCD shouldn't be an issue, since the capital is locked only for ~140 blocks. |
@@ -0,0 +1,166 @@ | |||
--- | |||
fip: <to be assigned> |
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.
FIP0034! I'll merge into repo, and update number for you, @anorth
The sector pre-commit deposit (PCD) is currently set equal to an estimate of the storage pledge for a sector.
The storage pledge is a 20-day projection of the expected reward to be earned by the sector.
This reward depends on power, which depends on the presence of verified deals hosted by the sector.
This dependency means the miner actor must consult the market actor, which intermediates verified data cap,
to determine the pre-commit deposit (and then again to compute initial pledge when the sector is proven).
This proposal sets the pre-commit deposit to a value independent of sector content:
the estimated storage pledge of a sector with space-time that is exactly filled by verified deals.
Discussion at #290
FYI @zixuanzh @nicola