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

Run protocol accounting for contract-dependent values #3792

Closed
Chris-Hibbert opened this issue Sep 3, 2021 · 11 comments
Closed

Run protocol accounting for contract-dependent values #3792

Chris-Hibbert opened this issue Sep 3, 2021 · 11 comments
Assignees
Labels
AMM Core Economy OBSOLETE in favor of INTER-protocol documentation Improvements or additions to documentation Inter-protocol Overarching Inter Protocol Vaults VaultFactor (née Treasury)
Milestone

Comments

@Chris-Hibbert
Copy link
Contributor

What is the Problem Being Solved?

We need a list of statistics to be collected across the treasury and the AMM for

  • managing debt (liquidations and shortfalls)
  • tracking issued RUN and BLD
  • tracking the use of RUN in the economy (e.g. execution fees)
  • tracking collateral in Vaults

Description of the Design

This is about collecting documentation. Design will follow

Security Considerations

It's about tracking flows and holdings so we can monitor the economy.

@Chris-Hibbert Chris-Hibbert added documentation Improvements or additions to documentation Beta Core Economy OBSOLETE in favor of INTER-protocol Inter-protocol Overarching Inter Protocol AMM labels Sep 3, 2021
@Chris-Hibbert Chris-Hibbert added this to the Beta Phase 4: Governance milestone Sep 3, 2021
@Chris-Hibbert Chris-Hibbert self-assigned this Sep 3, 2021
@rowgraus rowgraus added the MN-1 label Jan 19, 2022
@Tartuffo Tartuffo added MN-1 restival to be done before RUN Protocol Purple Team festival and removed MN-1 restival to be done before RUN Protocol Purple Team festival labels Jan 21, 2022
@Tartuffo
Copy link
Contributor

Tartuffo commented Feb 2, 2022

@Chris-Hibbert Please review with @rowgraus and @btulloh

@Tartuffo
Copy link
Contributor

Tartuffo commented Feb 4, 2022

This might be handoff-able to @turadg and/or @dckc . We gave a rough time-box estimate of 5.

@Tartuffo Tartuffo removed the MN-1 label Feb 7, 2022
@Tartuffo Tartuffo removed this from the Beta Phase 4: Governance milestone Feb 8, 2022
@Tartuffo
Copy link
Contributor

Superseded by #4639.

@btulloh
Copy link

btulloh commented Feb 25, 2022

I don't think this issue should be closed, at least it is not superseded by #4639. This refers to the internal accounting for RUN debt that the protocol incurs when it mints RUN. This is needed to track bad debt that needs to be covered by liquidation and reserves.

@dckc dckc removed the duplicate label Feb 25, 2022
@dckc dckc reopened this Feb 25, 2022
@Tartuffo
Copy link
Contributor

@dckc @Chris-Hibbert Who should be the assignee on this?

@Chris-Hibbert
Copy link
Contributor Author

Chris-Hibbert commented Feb 25, 2022

Let's make it me. I'll add some accounting in #4188, and after Restival spend some time writing down (and making issues) for what needs to be done in AMM and Vaults.

@Chris-Hibbert Chris-Hibbert added the Vaults VaultFactor (née Treasury) label Mar 10, 2022
@Tartuffo Tartuffo added this to the Mainnet 1 milestone Mar 23, 2022
@rowgraus
Copy link

@dtribble @btulloh @Chris-Hibbert

We need RUN Protocol accounting to happen somewhere in Mainnet 1. However, it's plausible that these don't need to be written in contracts and could instead be done with third party tools working off of eventing. Which would mean this isn't blocking and could be built post launch or as a bounty.

The question is: aside from debt limits, which I believe we are handling elsewhere, is there any reason I'm missing for why we need this built into contracts?

@btulloh
Copy link

btulloh commented Mar 24, 2022

I'm trying to distinguish between two different things.

  1. An external view via third-party tools. This could be done as a bounty, but we need it at launch of the RUN protocol, not post-launch. This is needed for external monitoring of what is happening. I don't like the idea of flying blind.
  2. An internal view of the overall solvency, which might trigger onchain actions. Perhaps what we currently have is sufficient, but it is not clear to me where everything is being tracked. For example, how does the protocol know that its assets are worth less than its liabilities, and therefore it needs to issue BLD to bring it back to solvency. The liabilities are the RUN outstanding. The protocol has multiple ways of issuing and burning RUN. Its assets are the dollar values of the PCV held by the protocol, plus the dollar values of the UCV held in vaults. My question is does the protocol need a way to sum across these different pools of assets and liabilities, and if so, do we have currently have a way to do that.

@turadg turadg changed the title Treasury Accounting Run protocol accounting for contract-dependent values Mar 30, 2022
@turadg
Copy link
Member

turadg commented Mar 30, 2022

Per Run Proto meeting today we've scoped this ticket to accounting that's necessary for the logic of the contracts. #4482 is for what can be tracked off-chain.

@Tartuffo
Copy link
Contributor

@rowgraus has this been handled, and can we close this issue?

@Chris-Hibbert
Copy link
Contributor Author

duplicate with #4639, #5300, #4648, #4649, #5352

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AMM Core Economy OBSOLETE in favor of INTER-protocol documentation Improvements or additions to documentation Inter-protocol Overarching Inter Protocol Vaults VaultFactor (née Treasury)
Projects
None yet
Development

No branches or pull requests

6 participants