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

[FIP Review]: FIP-0065 ignore built-in market balances from circulating supply #75

Open
anorth opened this issue Jul 23, 2023 · 0 comments
Labels
FIP Review Official (or soon to be) FIPs that require analysis

Comments

@anorth
Copy link

anorth commented Jul 23, 2023

Request Summary

Please review FIP-0065.

The built-in storage market actor's minimum deal collateral functions as a mandatory data assurance mechanism. The market's collateral and deal payment balances are subtracted from total token supply when calculating the network circulating supply. This proposal is to ignore the built-in market's locked balance in the network circulating supply definition. This is logical support for subsequent protocol changes that would allow storage providers to commit data into sectors without a built-in market deal or the associated collateral. With such change, the built-in market would not be a privileged actor and does not warrant special treatment in the circulating supply calculations.

Audience / Consumer for this Review

Core Developers

Timeframe

Needed in 1 month

Rationale

Very minor impacts on circulating supply. More significant impacts on implied protocol direction.

This is a logical prerequisite to direct data onboarding (Direct FIL+).

I don't anticipate any significant findings from review, this is mainly a courtesy to ensure you have time to process it before governance progresses.

Desired Deliverables

Comment on the FIP discussion indicating any significant impacts not already discussed.

Relevant Links (Including FIPs & FIP discussions)

https://github.com/filecoin-project/FIPs/blob/master/FIPS/fip-0065.md

Contact Details (if Private Response Requested)

No response

@anorth anorth added the FIP Review Official (or soon to be) FIPs that require analysis label Jul 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FIP Review Official (or soon to be) FIPs that require analysis
Projects
None yet
Development

No branches or pull requests

1 participant