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

inflation of foundation pool #5786

Closed
rob-solana opened this issue Sep 4, 2019 · 21 comments
Closed

inflation of foundation pool #5786

rob-solana opened this issue Sep 4, 2019 · 21 comments
Assignees
Labels
locked issue stale [bot only] Added to stale content; results in auto-close after a week.
Milestone

Comments

@rob-solana
Copy link
Contributor

rob-solana commented Sep 4, 2019

Problem

inflation, as specified, is split among validator rewards, storage rewards and the foundation
foundation pool/pools doesn't/don't exist yet

Proposed Solution

possible approaches:

  • keep a list of the accounts to be inflated as "the foundation"
  • add a new account owner that allows the bank to find these accounts?
  • ?
@rob-solana
Copy link
Contributor Author

CC @aeyakovenko @ericlwilliams

@mvines mvines added this to the Cloud Nine v0.20.0 milestone Sep 9, 2019
@ericlwilliams
Copy link
Contributor

@aeyakovenko it's likely we'll have to manually transfer foundation rewards in to the CBC account (they restrict 3rd party deposits) or we keep them in a separate account and combine the two (CBC and foundation rewards) during disassociation. WDYT?

@rob-solana rob-solana changed the title inflation of foundation and other grant pools inflation of foundation pool Sep 19, 2019
@mvines mvines modified the milestones: Supertubes v0.22.0, v0.23.0 Nov 25, 2019
@rob-solana
Copy link
Contributor Author

@aeyakovenko @ericlwilliams any updated thinking on this?

@aeyakovenko
Copy link
Member

@rob-solana i think the plan is to handle it of chain. Basically only delegate to people who can follow directions and lockup rewards.

@ericlwilliams
Copy link
Contributor

@aeyakovenko how does that relate to the inflation % specified to the Foundation?

@aeyakovenko
Copy link
Member

@rob-solana @ericlwilliams

so the % reward fee that is given to the foundation from the total inflation, is basically like a fixed global commission on all inflation rewards.

@ericlwilliams
Copy link
Contributor

ericlwilliams commented Dec 18, 2019 via email

@rob-solana
Copy link
Contributor Author

rob-solana commented Dec 29, 2019

The question here is how inflation code will identify the Foundation accounts.

Under the current configuration, in the first epoch (~13seconds), the BS leader will be entitled to redeem:

500_000_000SOL        // capitalization
  * 0.15              // inflation at T=0
  * 0.85              // validator rewards portion of inflation
  * 32                // slots in first epoch, about 13 seconds worth
  / (365*24*3600/0.4) // slots per year

Which is approximately 26SOL. The 2nd epoch is double that, and so on, until other nodes come online and more points are available, which will lead to rewards points dilution.

The Foundation is supposed to see 1.5SOL of that first epoch's inflation.

@mvines mvines modified the milestones: Tofino v0.23.0, Rincon v0.24.0 Jan 26, 2020
@mvines mvines modified the milestones: Rincon v0.24.0, v0.25.0 Feb 20, 2020
@mvines mvines modified the milestones: v1.1.0, v1.2.0 Mar 16, 2020
@mvines mvines modified the milestones: v1.2.0, The Future! Apr 19, 2020
@ericlwilliams
Copy link
Contributor

ericlwilliams commented May 7, 2020 via email

@mvines
Copy link
Member

mvines commented May 7, 2020

Not sure if this issue is even relevant anymore actually.

@mvines
Copy link
Member

mvines commented May 17, 2020

My bad, this is relevant still. The foundation/foundation_term fields in sdk/src/inflation.rs aren't hooked up to anything

@ericlwilliams
Copy link
Contributor

Looks like we're lumping Grant and Foundation inflation issuances into the Foundation terms here, which I think is fine, but since we originally scoped 5% grant & 5% foundation, might bump foundation inflation default to 10% for now. I think 7 years default is also okay to keep for now. Will publish and circulate official numbers before go-live.

@ryoqun
Copy link
Member

ryoqun commented Jul 20, 2020

@ericlwilliams @mvines I think we need to address this before enabling inflation on mainnet-beta, which is nearing to realize. :)

@ericlwilliams
Copy link
Contributor

@aeyakovenko what do you think? I think 5% Foundation, 5% grant still sounds reasonable (so w/ 8% initial inflation, that would be 2M tokens for each first year, dropping thereafter).
@ryoqun what exactly do you need to move forward besides these numbers?

@ryoqun
Copy link
Member

ryoqun commented Jul 22, 2020

@ericlwilliams I need specific hard-coded accounts to transfer the rewards for foundation and grant to like these:

"9huDUZfxoJ7wGMTffUE7vh1xePqef7gyrLJu9NApncqA",
"GK2zqSsXLA2rwVZk347RYhh6jJpRsCA69FjLW93ZGi3B",
"CWeRmXme7LmbaUWTZWFLt6FMnpzLCHaQLuR2TdgFn4Lq",
"HCV5dGFJXRrJ3jhDYA4DCeb9TEDTwGGYXtT3wHksu2Zr",
"14FUT96s9swbmH7ZjpDvfEDywnAYy9zaNhv4xvezySGu",
"HbZ5FfmKWNHC7uwk6TF1hVi6TCs7dtYfdjEcuPGgzFAg",
"C7C8odR8oashR5Feyrq2tJKaXL18id1dSj2zbkDGL2C2",
"Eyr9P5XsjK2NUKNCnfu39eqpGoiLFgVAv1LSQgMZCwiQ",
"DE1bawNcRJB9rVm3buyMVfr8mBEoyyu73NBovf2oXJsJ",
"CakcnaRDHka2gXyfbEd2d3xsvkJkqsLw2akB3zsN1D2S",
"7Np41oeYqPefeNQEHSv1UDhYrehxin3NStELsSKCT4K2",
"GdnSyH3YtwcxFvQrVVJMm1JhTS4QVX7MFsX56uJLUfiZ",
"Mc5XB47H3DKJHym5RLa9mPzWv5snERsF3KNv5AauXK8",
"7cvkjYAkUYs4W8XcXsca7cBrEGFeSUjeZmKoNBvEwyri",
"AG3m2bAibcY8raMt4oXEGqRHwX4FWKPPJVjZxn1LySDX",
"5XdtyEDREHJXXW1CTtCsVjJRjBapAwK78ZquzvnNVRrV",
"6yKHERk8rsbmJxvMpPuwPs1ct3hRiP7xaJF2tvnGU6nK",
"CHmdL15akDcJgBkY6BP3hzs98Dqr6wbdDC5p8odvtSbq",
"FR84wZQy3Y3j2gWz6pgETUiUoJtreMEuWfbg6573UCj9",
"5q54XjQ7vDx4y6KphPeE97LUNiYGtP55spjvXAWPGBuf",
"3o6xgkJ9sTmDeQWyfj3sxwon18fXJB9PV5LDc8sfgR4a",
"GumSE5HsMV5HCwBTv2D2D81yy9x17aDkvobkqAfTRgmo",
"AzVV9ZZDxTgW4wWfJmsG6ytaHpQGSe1yz76Nyy84VbQF",

Maybe the accounts will be one of them, because the accounts should be regarded as non-circulating?

@ericlwilliams
Copy link
Contributor

@ryoqun I see! Let me sync with @danpaul000 on this today and try to get those for you

@danpaul000
Copy link
Contributor

@ryoqun will the foundation accounts receiving rewards be system accounts, or stake accounts? If system, does it matter if the account exists/is already funded or not?

@mvines
Copy link
Member

mvines commented Sep 22, 2020

It can be any account. The bank would just deposit the foundation rewards in that account. Also consider if we just want 1 account or multiple (and if multiple how do the rewards get split up between them)

@mvines
Copy link
Member

mvines commented Sep 25, 2020

Temporary plan for pico-inflation is to just set the foundation rate to 0% to bypass this entirely

@stale
Copy link

stale bot commented Jan 9, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale [bot only] Added to stale content; results in auto-close after a week. label Jan 9, 2022
@github-actions
Copy link
Contributor

This issue has been automatically locked since there has not been any activity in past 7 days after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked issue stale [bot only] Added to stale content; results in auto-close after a week.
Projects
None yet
Development

No branches or pull requests

7 participants