Skip to content

Newsletters: add 98 (2020-05-20)#402

Merged
bitschmidty merged 5 commits intobitcoinops:masterfrom
harding:2020-05-20-newsletter
May 20, 2020
Merged

Newsletters: add 98 (2020-05-20)#402
bitschmidty merged 5 commits intobitcoinops:masterfrom
harding:2020-05-20-newsletter

Conversation

@harding
Copy link
Copy Markdown
Collaborator

@harding harding commented May 14, 2020

  • Lede, action items, news, releases/RCs, any additional notable merges @harding
  • Update the release/RC section Tuesday afternoon @harding
  • Changes to services and client software @bitschmidty
  • Bitcoin Core #18877 @jnewbery
  • C-Lightning #3614 @dongcarl

@harding harding marked this pull request as ready for review May 17, 2020 19:54
Copy link
Copy Markdown
Collaborator

@jonatack jonatack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good so far!

There's no need[^getheaders-details] to send more than one `getheaders`
request and Bitcoin Core doesn't send more than one block announcement
in an `inv` message, so this change should have no effect on Bitcoin
Core nodes who connect to Bitcoin Core peers; it's only expected
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestions:

  • s/who/that/
  • s/it's/its/ (or "the")


- [Eclair #1395][] updates the route pathfinding used by Eclair to
factor in channel balances and to use [Yen's algorithm][]. Testing
quoted in the PR description says, "The new algorithm consistently
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Testing" seems awkward as a subject.

elegance] the "elegance" of a simplified version of the protocol
that uses three transactions. Dmitry Petukhov [posted][petukhov
tla+] about a [specification][sas tla+ spec] he'd written for the
protocol in the [TLA<sup>+</sup> formal specification language][tla+
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perhaps add "(Temporal Logic of Actions)" between "TLA+" and "formal" to describe the acronym without the reader needing to leave the newsletter.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general I do think we should expand unfamiliar abbreviations. In this case, I don't think it's useful---either people are familiar with the language or not. Those who are familiar with it either know what the abbreviation stands for or don't care about it. Those who are not familiar with TLA+ probably want to know more about it than the expansion of its abbreviation, so they're going to click the link anyway.

There's no need[^getheaders-details] to send more than one `getheaders`
request and Bitcoin Core doesn't send more than one block announcement
in an `inv` message, so this change should have no effect when Bitcoin
Core nodes connect to Bitcoin Core peers; it's only expected
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/it's/its/ (as this is possessive and not a contraction)

Considering the sentence length, could also break into 2 sentences here with s/; it's/. The/


- [Eclair #1395][] updates the route pathfinding used by Eclair to
factor in channel balances and to use [Yen's algorithm][].
The PR description say that "the new algorithm consistently
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/say/says/ (or "According to the PR description: ")

Copy link
Copy Markdown
Contributor

@jnewbery jnewbery left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I just have a couple of nits.

I've pushed a commit for my write-up for Bitcoin Core n18877.

- **Evaluate proposed changes to BIP341 taproot transaction digest:** as
described in [last week's newsletter][news97 spk commit], there has
been a request for [taproot][topic taproot] signatures to make an
additional commitment to all the scriptPubKeys of the UTXOs being
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: s/all the scriptPubKeys of the UTXOs/the scriptPubKeys of all the UTXOs/ sounds slightly better to me (the first sounds to me a bit like a UTXO could include multiple scriptPubKeys).

that uses three transactions. Dmitry Petukhov [posted][petukhov
tla+] about a [specification][sas tla+ spec] he'd written for the
protocol in the [TLA<sup>+</sup> formal specification language][tla+
lang], helping to verify the correctness of the protocol (to a
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This parenthetical is a little awkward. I think it can be removed.

@harding
Copy link
Copy Markdown
Collaborator Author

harding commented May 18, 2020

Made suggested edits (thanks @jonatack and @jnewbery!) and made a few small edits to the merge summary written by @jnewbery (thanks!).

@jnewbery
Copy link
Copy Markdown
Contributor

ACK 99ea5a8. All looks good so far. Thanks for the quick turnaround, @harding.

Copy link
Copy Markdown
Contributor

@jnewbery jnewbery left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple of small comments in @bitschmidty's section.

Comment thread _posts/en/newsletters/2020-05-20-newsletter.md Outdated
Comment thread _posts/en/newsletters/2020-05-20-newsletter.md Outdated
@adamjonas
Copy link
Copy Markdown
Member

ACK f9a2f1e. Looks good!

@harding
Copy link
Copy Markdown
Collaborator Author

harding commented May 19, 2020

Thanks @bitschmidty for your section and @dongcarl for your merge summary! I made a few edits, most of which I think are self-explanatory, but two to Schmidty's section that might warrant additional explanation:

  • I added a mention about the protocol upgrade in the Blockstream Satellite section for two reasons: (1) so that it is clear to existing satellite users that they need to upgrade to get the ability to IBD over satellite and (2) because, from what I've heard, the new protocol is pretty nifty, so I wanted to mention it explicitly.

  • I moved the comma-delimited parenthetical in the Copay description to a separate sentence both because it was a bit awkward (and I never (ever) write awkward parentheticals) and because I wanted to make it clearer to readers that waiting four hours to fee bump is not a requirement of CPFP, it's a decision by the Copay authors.

As always, please feel free to revert or edit any changes. Thanks!

@bitschmidty
Copy link
Copy Markdown
Contributor

bitschmidty commented May 19, 2020

added a mention about the protocol upgrade in the Blockstream Satellite

SGTM

moved the comma-delimited parenthetical in the Copay description to a separate sentence

I parenthetical too much. Change looks good.

@jnewbery
Copy link
Copy Markdown
Contributor

All changes look good to me. Thanks Harding!

- **Evaluate proposed changes to BIP341 taproot transaction digest:** as
described in [last week's newsletter][news97 spk commit], there has
been a request for [taproot][topic taproot] signatures to make an
additional commitment to the scriptPubKeys of the all the UTXOs being
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/of the all/of all/

Copy link
Copy Markdown
Collaborator

@jonatack jonatack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting reporting on a variety of client software and services 👍

than existing protocols, it saves on transaction fees (both by using
less block space and potentially by requiring less urgency for its
settlement transactions), it only requires consensus-enforced
timelocks on one of the chains in a cross-chain swap, and that it
Copy link
Copy Markdown
Collaborator

@jonatack jonatack May 19, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perhaps remove "that" here, as the previous clauses don't use one

settlement transactions), it only requires consensus-enforced
timelocks on one of the chains in a cross-chain swap, and that it
doesn't depend on any new security assumptions or Bitcoin consensus
changes. If taproot is adopted, it can be used even more privately
Copy link
Copy Markdown
Collaborator

@jonatack jonatack May 19, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ambiguous "it"

perhaps "If taproot is adopted, this proposal (or protocol) would allow it to be used even more privately..."

- **Blockstream Satellite 2.0 supports initial block download:**
Blockstream [outlines version 2.0 upgrades][blockstream satellite v2 blog] to
their satellite service which include expanded Asia-Pacific coverage,
additional bandwidth, and an updated protocol that enabling a full node to complete an initial sync
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove "that", or s/enabling/enables/


- **Breez wallet enables spontaneous payments:**
[Version 0.9][breez 0.9] of Breez wallet adds the ability to send spontaneous
payments to Lightning nodes that support keysend.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe?: [keysend payments][topic spontaneous payments].

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only wanted to avoid because we used it a couple bullets earlier.

since the first release candidate.

- [LND 0.10.1-beta.rc1][] is the first release candidate for the next
maintenace release of LND.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maintenance


- [Bitcoin Core #18877][] is the first step towards support for serving
[compact block filters][topic compact block filters] on the P2P
network, as specified in [BIP157][]. Nodes that enabled the compact block filter index
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/enabled/enable/

`getdata` requests that specify an unknown type of data. The new
logic will also ignore requests for types of data that aren't expected
to be sent over the current connection, such as requests for
transactions on block-relay-only connections.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps describe or mention the change from previous behavior.

@harding
Copy link
Copy Markdown
Collaborator Author

harding commented May 19, 2020

@jonatack thanks for that extra review!

@bitschmidty bitschmidty force-pushed the 2020-05-20-newsletter branch from eae32cb to 37dcdce Compare May 20, 2020 10:59
@bitschmidty bitschmidty merged commit 125a1e2 into bitcoinops:master May 20, 2020
@bitschmidty
Copy link
Copy Markdown
Contributor

Squashed and merged!

👍 team! Thanks to @harding @jnewbery @dongcarl for authoring and @adamjonas @jonatack for reviewing.

hulatown added a commit to PrimitivesLane/bitcoinops.github.io that referenced this pull request May 1, 2026
Co-authored-by: hulatown(OpenClaw) <clawdius@openclaw.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants