-
Notifications
You must be signed in to change notification settings - Fork 944
fix: Refactor APR calculations to APY for Rezerve's lstRZR #2001
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
Conversation
Updated the calculation functions to use APR instead of APY for better accuracy. Introduced a new function to convert APR to APY. This change enhances the clarity and precision of the financial metrics in the rezerve-money adaptor.
The rezerve-money adapter exports pools: Test Suites: 1 passed, 1 total
|
The rezerve-money adapter exports pools: Test Suites: 1 passed, 1 total
|
@drecronye does the vault autocompound? yield seems off compared to ur UI |
The rezerve-money adapter exports pools: Test Suites: 1 passed, 1 total
|
Hey @slasher125 our UI was actually wrong as well. And we got that fixed. It now matches close to what the adapter returns and also what Pendle/Spectra also reports PFA the UI https://rezerve.money/liquid-staking ![]() |
The rezerve-money adapter exports pools: Test Suites: 1 passed, 1 total
|
We have been tracking the APR and not the APY for the vaults 🙈. This PR fixes that.
Historic APY charts