-
Notifications
You must be signed in to change notification settings - Fork 165
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-0065: Ignore built-in market balances in circulating supply (first draft) #726
Conversation
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.
Looks good to me.
AFAIK, this is the first time we've FIP-ped something that is mostly a declaration of intent for future changes. While I don't oppose such a FIP, it's something we should discuss as part of the deliberation process -- if this FIP passes with little contention, but there's opposition to the future FIP that actually introduces direct commitment of data into sectors (on the grounds that the centrality of the built-in market actor shouldn't be eroded), would we entertain that discussion? Or shut it down saying "sorry, this previous FIP already announced a decision on this"?
Regardless, that doesn't affect landing this PR.
Thanks for raising. Yes, I think we would. I don't think there's anything that reasonable people could do now that should prevent other reasonable people in the future from debating a change. With that in mind, I think I will rework this FIP to focus on the technical change, with clear pointers to the rationale being expectation of direct commitment of data. The content will mostly remain, just re-arranged. |
f3c9846
to
a6a1e0d
Compare
FIPS/fip-xxxx-deal-collateral.md
Outdated
Participants and developers will need to explicitly alter their arrangements should they wish to make | ||
arrangements that uses a lesser amount. | ||
|
||
## Implementation |
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.
Does this mean we are excluding it from both regular and vm circulating supply or only the VM one?
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.
I only know of one protocol-defined circulating supply, which is the value that is input to calculations of things like sector initial pledge. Are any other definitions of circulating supply relevant to consensus rules?
First editorial pass ✔️ with some nits. re: @arajasek's good flag: This FIP's motivation and impact depends on many hypothetical future protocol changes, that may or may not be accepted by the Filecoin community. The idea of this proposal is reasonable and concrete enough to be landed as a draft, however, when this FIP can be moved to Last Call/Acceptance is worth discussing. some quick options:
@kaitlin-beegle I think this is gap in the current governance process and would love to hear our feedback. |
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.
comments left per @jennijuju
FIPS/fip-xxxx-deal-collateral.md
Outdated
--- | ||
|
||
## Simple Summary | ||
Exclude the built-in market actor's locked balances from the network circulating supply calculation, |
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.
Recommend a slightly different phrasing than include/exclude
The built-in markets locked balances are currently included in the total locked amount, which is then subtracted from the total circulating supply. Another way to plausible way to say this is "the total Filecoin circulating supply EXCLUDES the balances locked in the built in actor".
I realize now the proposal is to exclude them from the calculation, but I read this several times as excluding them from the total circulating supply number itself, which I found very confusing because I know that's what already happens.
Maybe "remove the built-in market actors locked balanced from the pool of locked filecoin on the network" is a better way to say it?
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.
Thanks, it's tricky to find something concise (FIP title is supposed to be <80 chars, though I find this restriction silly). I changed it to "ignore the built-in market locked balance..."
FIPS/fip-xxxx-deal-collateral.md
Outdated
Removing them from the calculation will thus have negligible effect on downstream calculations | ||
such as the sector initial pledge requirement. | ||
|
||
Removing these balances from the calculation of network circulating supply will not actually unlock any tokens, |
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.
Even if there's no underlying change, I assume crypto exchanges report the number the chain reports. I wonder if a sudden bump in the number affects token price?
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.
I have some reason to believe they chose their own definition (see @jennijuju comment about about which definition I'm talking about), and I don't know which if any include tokens in deal collateral or payment escrow as "circulating".
(I’m happy to land this DRAFT when we find a venue to discuss the raised governance process questions) |
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 this looks great, thanks for putting it together @anorth 👍
My only suggestions are to change a couple of things to improve the readability and understandability (particularly for non-native speakers):
- I'd suggest massaging the language to be as concise as possible
- eg minimize repetition of the same point in different sections
- I'd suggest using bullet points instead of paragraphs where possible
- I'd suggest using graphics where possible (better for non-native speakers) eg a pie graph of locked balances vs total locked amount vs circulating supply
Thanks @dirkmc , someone should make you a FIP editor. The template structure makes it hard to avoid repetition, but I will attempt to compress and tabulate a bit. A pie chart isn't going to work well to show a 0.21% slice. |
I have addressed the editorial comments. @jennijuju please approve given your ✔️ so I can merge this draft for community consideration . We can discuss any governance issues in the the discussion #719. |
Summary
Declare that future protocol changes may commit non-zero data into sectors without requiring a corresponding deal in the built-in market actor.
Remove the built-in market actor's locked balances from the network circulating supply calculation.
Discussion: #719