-
Notifications
You must be signed in to change notification settings - Fork 100
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
Bigz/perp lp jit #448
Merged
Merged
Bigz/perp lp jit #448
Changes from 31 commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
7f865ef
bigz/perp-lp-jit
0xbigz 752f56e
sdk: add updatePerpMarketTargetBaseAssetAmountPerLP
0xbigz b013675
init tests (wip)
0xbigz 3aa9a51
todo: start split_with_lps tests
0xbigz c41bd08
fix test
0xbigz 65e6deb
controller/orders.rs: split out to calculate_amm_jit_liquidity
0xbigz e53ab10
split out with split_with_lps param in calculate_jit_amount
0xbigz f24cd31
fix idl
0xbigz c8bf3be
Merge branch 'master' into bigz/perp-lp-jit
0xbigz cbe7eff
update idl
0xbigz 546bafa
fix greater/less than on target lp amount
0xbigz 9c3c12e
wip test
0xbigz aaace61
working basic tests
0xbigz c585830
wip, working typescript tests and functions
0xbigz a743abb
fix tests/perpLpJit.ts
0xbigz 18fe355
fix test
0xbigz fb56aab
merge master
0xbigz 3bc6dbd
add LP split OrderActionExplanation, linter
0xbigz 91fbec4
fix prettier
0xbigz a27eb14
address feedback p1
0xbigz fc073b1
add amm_lp_wants_to_jit_make functions
0xbigz a16c934
cleanup logs, add intuition comment
0xbigz dacf592
improve amm_lp_allowed_to_jit_make compute
0xbigz 791c6fb
cargo fmt --
0xbigz 3c44caa
revert split_with_lps fee_to_market
0xbigz 156e8b0
use AMMLiquiditySplit
0xbigz 84bf634
use protocol_owned_min_side_liquidity for amm_lp_allowed_to_jit_make
0xbigz 9a2d562
wip lp-owned fill
0xbigz 505ac06
cargo fmt --, fix test
0xbigz 9c6e842
data check inside update_lp_market_position
0xbigz c29e80e
update_lp_market_position: comb early return
0xbigz 668b7c4
incorp most of feedback-p2
0xbigz 0dcf0d2
working LPOwned fills
0xbigz eb7b054
fix comment
0xbigz 5a7dcb7
merge master
0xbigz 123b734
add some perp_market precisions
0xbigz 3060374
tests/perpLpJit.ts: add asserts
0xbigz 749dcaa
tests/perpLpJit.ts: add btc test
0xbigz 1900c8f
wip more asserts for tests/perpLpJit.ts
0xbigz 1f0a105
tests/perpLpJit.ts: expanded
0xbigz 04475a7
tests: remove assert from spotSwap test causing race conditions
crispheaney 4f838b6
sdk: update spl-token dependency to 0.3.7 (#465)
crispheaney 5aa89b8
sdk: tweaks to make driftClient.swap work
crispheaney 69a5fbe
sdk: add ability to pass in route to driftClient.swap
crispheaney 4342ebf
sdk: supporting changes for jupiterClient (#466)
wphan 84419ba
program: add swap reduce only (#468)
crispheaney 6c93e7e
init msol config (#467)
0xbigz 4a7ffbb
sdk: spotBalance.ts docs
wphan 42a9907
sdk: make ts BN for swaprecords
lowkeynicc d2057ac
v2.30.0
crispheaney 64015e6
Merge branch 'master' into bigz/perp-lp-jit
0xbigz b2993db
update changelog
0xbigz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
i feel like
override_base_asset_amount
,override_fill_price
andliquidity_split
are all related, maybe we can merge them somehow so it's clearer how they're relatedThere 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.
think adding params to
AMMLiquiditySplit
is a nice-to-have future task