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
I want to have pathFinder check n-length paths, instead of hard-coded three-asset paths as it does now.
Right now it does this path structure:
XLM ➡️ USD ➡️ EUR ➡️ XLM
With this feature it could also do, for example:
XLM ➡️ USD ➡️ EUR ➡️ CNY ➡️ XLM
Describe alternatives you've considered
This would be an adaptation of the n-path code that pathRequester already has; pathRequester itself was the alternative to pathFinder's current behavior but the rest of pathRequester didn't perform well.
Requirements
Change pathFinder's PaymentPath so it uses a list of pairs
Make MakePathPayment behave like makePathPaymentredux, but for rockfish's native structs instead of Horizon's json return
Update pathFinder's configuration to accommodate the changes
The text was updated successfully, but these errors were encountered:
But since Lumen allows N-length payment transactions, this could make sense to insta-capture additional margin...
My bots on waves dex were exposed to a great risk in 4+ steps transactions, because most of the time I ended up stuck in some low-liqudity coin, this may not be the case with Lumen.
Describe the solution you'd like
I want to have
pathFinder
check n-length paths, instead of hard-coded three-asset paths as it does now.Right now it does this path structure:
XLM ➡️ USD ➡️ EUR ➡️ XLM
With this feature it could also do, for example:
XLM ➡️ USD ➡️ EUR ➡️ CNY ➡️ XLM
Describe alternatives you've considered
This would be an adaptation of the n-path code that
pathRequester
already has; pathRequester itself was the alternative to pathFinder's current behavior but the rest of pathRequester didn't perform well.Requirements
PaymentPath
so it uses a list of pairsMakePathPayment
behave likemakePathPaymentredux
, but for rockfish's native structs instead of Horizon's json returnThe text was updated successfully, but these errors were encountered: