✅ hebasto closed a pull request: "Create aws.yml"
(https://github.com/bitcoin/bitcoin/pull/32164)

📝 hebasto locked a pull request: "Create aws.yml"
(https://github.com/bitcoin/bitcoin/pull/32164)
<!--
UN commissioner of cryptocurrency IHL Humanitarian BeinExchange involvement since 2008 sashatoshi mr pool Mr forbes always Beinhauer born Shirey living one enternity.
*** Please remove the following help text before submitting: ***

Pull requests without a rationale and clear improvement may be closed
immediately.

GUI-related pull requests should be opened against
https://github.com/bitcoin-core/gui
first. See CONTRIBUTING.md
-->

<!--
Please provide clear motivation for your
...

💬 fjahr commented on pull request "Embed default ASMap as binary dump header file":
(https://github.com/bitcoin/bitcoin/pull/28792#issuecomment-2764271297)
Rebased to get all the latest cmake changes and swapped the included mapping data to the [latest asmap run](https://github.com/asmap/asmap-data/pull/23). This used the `--fill` feature in encoding which leads to additional space savings. This is why the embedded data has gone down significantly from ~1.7MB to ~1.4MB.

💬 warioishere commented on issue "assumeutxo: not syncing the snapshot chainstate":
(https://github.com/bitcoin/bitcoin/issues/30971#issuecomment-2764273554)
hello, I think it doesnt help much, but assume utxo worked for me, but a 12G big chainstate_snapshot directory is left. Can I delete it? I have no more background validation logs inside my debug log an it said some hours ago:

```
2025-03-29T13:04:42Z [background validation] UpdateTip: new best=0000000000000000000320283a032748cef8227873ff4872689bf23f1cda83a5 height=840000 version=0x2a5fe000 log2_work=94.873245 tx=991032194 date='2024-04-20T00:09:27Z' progress=0.832048 cache=253.6MiB(1789616txo)

...

💬 fjahr commented on issue "assumeutxo: not syncing the snapshot chainstate":
(https://github.com/bitcoin/bitcoin/issues/30971#issuecomment-2764286415)
@warioishere The `chainstate_snapshot` directory is automatically deleted at the next restart of your node. Have you done one already? If not, please restart your node and see if it was removed. Thanks you!

💬 jirijakes commented on pull request "doc: Fix and clarify description of ZMQ message format":
(https://github.com/bitcoin/bitcoin/pull/31862#discussion_r2020060211)
I overlooked that, fixed.

💬 jirijakes commented on pull request "doc: Fix and clarify description of ZMQ message format":
(https://github.com/bitcoin/bitcoin/pull/31862#discussion_r2020060259)
Used this suggestion.

💬 jirijakes commented on pull request "doc: Fix and clarify description of ZMQ message format":
(https://github.com/bitcoin/bitcoin/pull/31862#discussion_r2020060307)
This was a very useful suggestion. Thanks!

💬 jirijakes commented on pull request "doc: Fix and clarify description of ZMQ message format":
(https://github.com/bitcoin/bitcoin/pull/31862#issuecomment-2764377913)
Reflected @ryanofsky's comments (thanks!) and used his suggestions.

- format of messages is now first outlined in a single table
- explanation of each topic follows the table in dedicated subsections
- format of numbers is consistent
- sequence numbers are referred to as either _message sequence number_ or _mempool sequence number_ to stress the difference
- the last subsection of “Usage” section is now called “Implementing ZMQ client” to separate it from previous sections; additionally,
...

⚠️ SatoshiFans888 opened an issue: "Acccedence; Double Poisson Sum†; p 0.04999579029644097"
(https://github.com/bitcoin/bitcoin/issues/32165)
Dear Bitcoin,

Wouldn't it Is the Poisson Sum shown as the periodic end or the spearhead in Bitcoin? Because it is not at all applicable to either Blockchain or what we call Cryptocurrency. Either you can really implement what the Poisson Sum is or you just have Delusions that think you understand what Satoshi Nakamoto published.

Why is it hurt by the content of my message? Let's focus on the topic ^^.

We know that the Poisson Sum has an exponential value of rational repeating but one thing
...

✅ fanquake closed an issue: "Acccedence; Double Poisson Sum†; p 0.04999579029644097"
(https://github.com/bitcoin/bitcoin/issues/32165)

💬 warioishere commented on issue "assumeutxo: not syncing the snapshot chainstate":
(https://github.com/bitcoin/bitcoin/issues/30971#issuecomment-2764480408)
> [@warioishere](https://github.com/warioishere) The `chainstate_snapshot` directory is automatically deleted at the next restart of your node. Have you done one already? If not, please restart your node and see if it was removed. Thanks you!

thanks pal I was better to read

[](https://github.com/bitcoin/bitcoin/blob/master/doc/design/assumeutxo.md?utm_source=chatgpt.com)
first hehe

Yes after restart `chainstate_snapshot` got cleaned up

💬 SatoshiFans888 commented on issue "Acccedence; Double Poisson Sum†; p 0.04999579029644097":
(https://github.com/bitcoin/bitcoin/issues/32165#issuecomment-2764493899)
@fanquake Thanks a lot

💬 laanwj commented on issue "(removed)":
(https://github.com/bitcoin/bitcoin/issues/32165#issuecomment-2764496988)
This github ia intended for concrete issues related to functionality of the software or its development. Please do not post rambling (LLM-generated?) pseudo-mathemetical essays about what bitcoin ought to be here, it is a waste of time for all of us.

✅ jirijakes closed an issue: "test: No unit test covers BIP342 tapscript signatures"
(https://github.com/bitcoin/bitcoin/issues/32012)

💬 jirijakes commented on issue "test: No unit test covers BIP342 tapscript signatures":
(https://github.com/bitcoin/bitcoin/issues/32012#issuecomment-2764497762)
Thank everybody for checking.

Originally, I was running the test without `qa-assets`, only using `./build/bin/test_bitcoin --run_test=script_tests`. To my understanding, that uses only the attached `bip341_wallet_vectors.json`, which is the test vector from the BIP (it does not cover tapscript).

What I did not verify was running with `qa-assets`. As others have reported, using `qa-assets` correctly raises errors after the modification.

I believe the BIP's test vectors should also cover tapscr
...

💬 SatoshiFans888 commented on issue "(removed)":
(https://github.com/bitcoin/bitcoin/issues/32165#issuecomment-2764500880)
@laanwj Either I understand or I'm wrong, the essence of the message is the determination of z and p values in the Double Poisson Sum, I always calculate the risk before calculating the reward, I also had a headache with how it was applied to Mathematics but I could only tell the numbers were 1.96 and 0.04999579029644097 and some other criteria, maybe Bitcoin is still the Bitcoin that Satoshi asked for at the beginning of his speech, so proud.

📝 eval-exec opened a pull request: "torcontrol: Define tor reply code as cosnt to improve readability) torcontrol: Define tor replay code as const to improve our maintainability"
(https://github.com/bitcoin/bitcoin/pull/32166)
This PR want to replace tor repy code with const to improve out maintainability.

Friendly invite

💬 eval-exec commented on pull request "torcontrol: Add comment explaining Proxy credential randomization for Tor privacy":
(https://github.com/bitcoin/bitcoin/pull/31973#issuecomment-2764507260)
Friendly ping @fanquake , I think this PR is ready to be merged.

💬 maflcko commented on pull request "torcontrol: Define tor reply code as const to improve our maintainability":
(https://github.com/bitcoin/bitcoin/pull/32166#issuecomment-2764511611)
Could probably be cherry-picked into https://github.com/bitcoin/bitcoin/pull/31973, so that there is only a single pull with doc/refactor fixups of the same file?

✅ eval-exec closed a pull request: "torcontrol: Define tor reply code as const to improve our maintainability"
(https://github.com/bitcoin/bitcoin/pull/32166)