-
Notifications
You must be signed in to change notification settings - Fork 952
Askrene single path solver #8299
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Askrene single path solver #8299
Conversation
0d0c36d
to
444524a
Compare
f84f5ec
to
6bc2ba4
Compare
I benchmarked the single path solver in comparison to the MPP default solver The success rate for MPP is 82% while single path is 77%. I wanted to test the routing to the first 1000 nodes but askrene crashed while trying to reach node 785, this bug can also be reproduced in the master branch. I am digging into it. |
6bc2ba4
to
6f73888
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor feedback only... I'm reworking the predecessor PR, so this will need a rebase too..
6a8669e
to
62d3592
Compare
Rebased to fix conflicts with previous changes on #8163 |
62d3592
to
db179e1
Compare
Refactor MCF solver: remove structs linear_network and residual_network. Prefer passing raw data to the helper functions. Changelog-None Signed-off-by: Lagrang3 <[email protected]>
The single path solver uses the same probability cost and fee cost estimation of minflow. Single path routes computed this way are suboptimal with respect to the MCF solution but still are optimal among any other single path. Computationally is way faster than MCF, therefore for some trivial payments it should be prefered. Changelog-None. Signed-off-by: Lagrang3 <[email protected]>
Changelog-Added: askrene: an optimal single-path solver has been added, it can be called using the developer option --dev_algorithm=single-path or by adding the layer "auto.no_mpp_support" Signed-off-by: Lagrang3 <[email protected]>
Changelog-None Signed-off-by: Lagrang3 <[email protected]>
db179e1
to
b0fa17c
Compare
Wow, this is weird. I didn't touch renepay code at all and this test decided to randomly fail.
UPD: this must have been a glitch in the CI. |
73f0331
into
ElementsProject:master
With this PR I want to include an internal API to askrene to compute optimally reliable
single path payments. It uses the same costs functions as the minimum cost flow solver
but with the additional constraint that only a single path solution will be considered.
In practice this is solved with the execution of Dijkstra best path algorithm therefore it is likely
to produce orders of magnitude (~x100?) faster run times.
This option can be activated by calling
getroutes
with the layerauto.no_mpp_support
or with the developer flag
--dev_algorithm=single-path
.We used to compute single path routes by first doing a MCF computation and then
discarding all but one path which is computationally expensive and not even optimal.
We may use this single path computation with small payment amounts or situations for which MPP
is not supported or if time to solution is a concern.
Depends on #8163 and #7932.
TODO: