search

gnosis / dex-services

Off-chain services for the Gnosis Protocol v1.
33 stars 9 forks source link

Add mix of dynamic and static economic viability strategy #1481

Closed e00E closed 4 years ago

e00E commented 4 years ago

which uses the better of the dynamic and the static strategy.

This is useful because for ease of use we want to guarantee a minimum viable order size but still lower it when this is economically viable because of low gas price.

successor of https://github.com/gnosis/dex-services/pull/1479 (sorry for making you review again, but the implementation is different enough that I think it's a good idea)

fleupold commented 4 years ago

👍 Looks like a reasonable option to me (I don't really see the advantage of renaming the fixedComputer to a staticComputer but 🤷)