You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
#3027 updates the /quote response to return enough data to quote with resting limit orders or JIT orders.
The next step is to use the new data in the quote verification logic to check that whatever the solver proposes would actually work onchain.
Details
This requires the ExternalTradeFinder to parse the new data (pre-interactions, JIT orders) from the JSON response and pass it to the simulation logic via the Trade struct. trade_verifier::encode_settlement() would also have to encode the new interactions and orders. AFAIK the underlying simulation logic would not have to be adjusted since it's using the actual settlement contract under the hood so it should automatically do the right thing with the new interactions and orders.
Acceptance criteria
e2e test verifying that a cow amm can be used to provide a quote.
This requires:
setting up the cow amm
using the mock solver to return a hardcoded solution with it (example here)
Background
#3027 updates the
/quote
response to return enough data to quote with resting limit orders or JIT orders.The next step is to use the new data in the quote verification logic to check that whatever the solver proposes would actually work onchain.
Details
This requires the
ExternalTradeFinder
to parse the new data (pre-interactions
,JIT orders
) from the JSON response and pass it to the simulation logic via theTrade
struct.trade_verifier::encode_settlement()
would also have to encode the new interactions and orders. AFAIK the underlying simulation logic would not have to be adjusted since it's using the actual settlement contract under the hood so it should automatically do the right thing with the new interactions and orders.Acceptance criteria
e2e test verifying that a cow amm can be used to provide a quote.
This requires:
auction_prices
in the DB before starting theautopilot
verified: true
The text was updated successfully, but these errors were encountered: