dex(router): define ordering on `Path<S>` #2763

erwanor · 2023-06-29T20:48:46Z

Closes #2758

crates/core/component/dex/src/component/router/path.rs

erwanor added 3 commits June 29, 2023 15:47

dex(router): add path_search_unique test

c74bd3e

dex(router): impl Ord for Path<S>

dbd1be9

dex(router): Utilize Path<S> compare operator in update

dec672b

erwanor temporarily deployed to smoke-test June 29, 2023 20:48 — with GitHub Actions Inactive

erwanor requested a review from hdevalence June 29, 2023 20:50

erwanor marked this pull request as ready for review June 29, 2023 20:50

hdevalence reviewed Jun 30, 2023

View reviewed changes

crates/core/component/dex/src/component/router/path.rs Outdated Show resolved Hide resolved

dex: use Ord combinators

8c14279

erwanor temporarily deployed to smoke-test June 30, 2023 12:17 — with GitHub Actions Inactive

erwanor had a problem deploying to smoke-test June 30, 2023 12:48 — with GitHub Actions Error

erwanor force-pushed the path_search_tie_breaker branch from e6f63b8 to dfd68e9 Compare June 30, 2023 12:50

erwanor had a problem deploying to smoke-test June 30, 2023 12:50 — with GitHub Actions Error

dex(router): basic tests for path comparison

196ad5c

erwanor force-pushed the path_search_tie_breaker branch from dfd68e9 to 196ad5c Compare June 30, 2023 12:55

erwanor temporarily deployed to smoke-test June 30, 2023 12:55 — with GitHub Actions Inactive

dex(router): consider the path start in compare

be193e2

erwanor temporarily deployed to smoke-test June 30, 2023 14:05 — with GitHub Actions Inactive

erwanor mentioned this pull request Jun 30, 2023

rpc: SimulateTrade is slow and contains duplicates for very large inputs #2751

Closed

erwanor merged commit 00c7bdf into main Jun 30, 2023

erwanor deleted the path_search_tie_breaker branch June 30, 2023 20:01

cratelyn added the A-dex Area: Relates to the dex label Mar 5, 2024

Provide feedback