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

Model stableswap il #147

Closed
wants to merge 94 commits into from
Closed

Model stableswap il #147

wants to merge 94 commits into from

Conversation

jepidoptera
Copy link
Contributor

No description provided.

jepidoptera and others added 27 commits July 3, 2023 13:13
# Conflicts:
#	hydradx/model/amm/stableswap_amm.py
#	hydradx/model/amm/trade_strategies.py
# Conflicts:
#	hydradx/model/amm/trade_strategies.py
@jepidoptera jepidoptera requested a review from poliwop July 27, 2023 19:40
jepidoptera and others added 28 commits August 18, 2023 17:20
…stableswap-IL

# Conflicts:
#	hydradx/model/amm/stableswap_amm.py
#	hydradx/model/amm/trade_strategies.py
# Conflicts:
#	hydradx/model/amm/global_state.py
#	hydradx/model/amm/stableswap_amm.py
Copy link
Collaborator

@poliwop poliwop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something went wrong with the recent merge, neither of these notebooks seem to work.

It looks like ImpermanentGainAttack.ipynb just wasn't updated to account for the refactor, but I don't see the function called in ImpermanentLossAnalysis.ipynb anywhere?

" for trade_size in trade_sizes:\n",
" lp_state, lp_agent = initial_state.copy(), initial_agent.copy()\n",
" for tkn in initial_agent.asset_list:\n",
" StableSwapPoolState.execute_add_liquidity(\n",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This notebook doesn't work with the refactor.

"\n",
" for event in events:\n",
" # agent withdraws assets in equal proportion\n",
" StableSwapPoolState.execute_remove_uniform(\n",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This function doesn't appear to exist in this branch?

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.

2 participants