Skip to content
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

Update knowlege in both directions #2

Open
renepickhardt opened this issue May 6, 2022 · 1 comment
Open

Update knowlege in both directions #2

renepickhardt opened this issue May 6, 2022 · 1 comment
Labels
bug Something isn't working enhancement New feature or request good first issue Good for newcomers

Comments

@renepickhardt
Copy link
Owner

if we learn something from a channel in direction A --> B we currently update the A-->B arc in the uncertainty network. However we also learn something about the B-->A arc which we completely neglect as of now. This should be fixed before the release

@renepickhardt renepickhardt added bug Something isn't working enhancement New feature or request good first issue Good for newcomers labels May 12, 2022
@renepickhardt renepickhardt added this to the alpha release: 0.1 milestone May 12, 2022
@Lagrang3
Copy link

Lagrang3 commented Aug 3, 2023

BTW. we have this in renepay plugin, here flow.h. We first realized that a bound in the liquidity [a,b), meaning that the liquidity satisfies a<=l0<b, implies that the liquidity in the opposite direction l1 is bounded by [a',b'), where c is the capacity of the channel, a'=c-b and b'=c-a.

When knowledge is updated in one direction, we update the [a,b) and we recompute [a',b').

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants