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

Add position calculator for Renzo Protocol #41

Merged
merged 2 commits into from
Apr 25, 2024

Conversation

anshuman73
Copy link
Contributor

Computes address position in terms of EZETH in wei. Writes out csv when called with requested spec.

@0xroll
Copy link
Contributor

0xroll commented Apr 16, 2024

hi @anshuman73 thanks for the PR.

quick question : is this just getting the token balances that are sitting on wallet addresses? (EOA + contracts)

@vokkantthakkar
Copy link

vokkantthakkar commented Apr 16, 2024

hey yes sir- a user's TVL in this case is this ezEth holding. The document is written from a DeFi protocol perspective, as we are an LRT, we thought this would work.

@nitish-91
Copy link
Contributor

@vokkantthakkar the query doesnt seem to be using the block number we give as input. We need user tvl at the given block number. Can you please make the change or please let me know if i have misunderstood this.

@vokkantthakkar
Copy link

@vokkantthakkar the query doesnt seem to be using the block number we give as input. We need user tvl at the given block number. Can you please make the change or please let me know if i have misunderstood this.

@anshuman73 is on it, and will be pushing a change shortly

@anshuman73
Copy link
Contributor Author

@vokkantthakkar the query doesnt seem to be using the block number we give as input. We need user tvl at the given block number. Can you please make the change or please let me know if i have misunderstood this.

Hi, this has been resolved. Note the adapter can take some time to process, is there any timeout limit?

@nitish-91 nitish-91 merged commit 94442ce into delta-hq:main Apr 25, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants