-
Notifications
You must be signed in to change notification settings - Fork 137
Newsletters: add 71 (2019-11-06) #254
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
Conversation
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.
tACK
ACK a826e8e . Looks great! |
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.
couple nits, but otherwise looks good!
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.
Really good newsletter @harding! A couple of comments below, in addition to those by @adamjonas.
|
||
- [LND #3558][] synthesizes a unified policy for any case where two | ||
particular nodes have multiple channels open between them and then | ||
uses this synthetic policy when considering routing through any of |
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.
nit: maybe just me, but "synthetic policy" here could be read as fake/ersatz, perhaps "unified policy" or "policy"?
|
||
- [LND #3556][] adds a new `queryprob` RPC that returns the expected | ||
probability that a payment would succeed given a particular | ||
destination node and amount. This replaces functionality previously |
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.
IIUC the queryprob rpc takes source node, destination node, and amount as args.
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.
ACK e207061 |
e207061
to
08ec4a3
Compare
LN anchor outputs talks about remote delay Please help test the latest Bitcoin Core RC Thanks to @harding for another hit today |
No description provided.