stellar-deprecated / kelp

Kelp is a free and open-source trading bot for the Stellar DEX and 100+ centralized exchanges
https://kelpbot.io
Other
1.1k stars 263 forks source link

[3] buytwap and selltwap strategies should support specifying filters by quote #633

Open nikhilsaraf opened 3 years ago

nikhilsaraf commented 3 years ago

currently the twap based strategies are running in restricted mode where they only support filters based on the quote asset.

the factory method for the sell twap level provider explicitly checks that the filter is on the base asset otherwise does not allow it.

we need to update the log lines and variable names into a format that allows the quote asset as the filter. this will probably be the primary change in this issue.

the filter has automated tests against the quote asset.

testing should also be done manually once completed for sanity checking.

not high priority unless someone requests it explicitly

related to #522