-
Notifications
You must be signed in to change notification settings - Fork 10
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
Model lp fees #140
Model lp fees #140
Conversation
Specced LRNA mint in Spec.ipynb
… decorator. Loosening acceptable error in hope this resolves issue.
…or LPs is approximately double
* changed withdrawal fee for stableswap to match our runtime 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.
The results in this notebook can be confirmed in this spreadsheet:
https://docs.google.com/spreadsheets/d/1Fh0StOPIDhazhQgwsXHVVvubJCqJpgfJTA8ZdN53bV0/edit?usp=sharing
This does validate the notebook. I think though that if we are going to share these results, it makes sense to do it with a simple spreadsheet. Can we think of extensions to this notebook to make it more realistic? Having non-zero LRNA fees is a simple one, it's not immediately obvious how to incorporate that into the spreadsheet.
… graph at the bottom which separately extrapolates to one year
# Conflicts: # hydradx/notebooks/Omnipool/LP_fees_analysis.ipynb
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.
LP_fees_analysis.ipynb notebook looks good, there are some changes needed in the rest of the PR but the analysis is ready.
modeling for LP fees without the latest LRNA spec update