🤔 glozow reviewed a pull request: "[WIP] Cluster mempool implementation"
(https://github.com/bitcoin/bitcoin/pull/28676#pullrequestreview-2277909745)
had a look at some tests that are disabled or have TODOs

💬 glozow commented on pull request "[WIP] Cluster mempool implementation":
(https://github.com/bitcoin/bitcoin/pull/28676#discussion_r1742339252)
9047d620b81
Suggestion for adding the deltas to chunk feerates instead of deleting the fields:
```
diff --git a/test/functional/mining_prioritisetransaction.py b/test/functional/mining_prioritisetransaction.py
index a8183dc8e14..6c3e02b8862 100755
--- a/test/functional/mining_prioritisetransaction.py
+++ b/test/functional/mining_prioritisetransaction.py
@@ -101,21 +101,19 @@ class PrioritiseTransactionTest(BitcoinTestFramework):
self.nodes[0].prioritisetransaction(txid=txid_c, f
...

💬 glozow commented on pull request "[WIP] Cluster mempool implementation":
(https://github.com/bitcoin/bitcoin/pull/28676#discussion_r1742343258)
9047d620b81 Just deleting the clusterid is sufficient
```suggestion
del last_entry["clusterid"]
```

💬 glozow commented on pull request "[WIP] Cluster mempool implementation":
(https://github.com/bitcoin/bitcoin/pull/28676#discussion_r1742376309)
d0576c134a5
I don't think it's possible to adapt this test so that the tx has 100 direct conflicts, given that it needs to be within 1000vB (the only transactions allowed to have sibling eviction are TRUC children). Having 100 inputs makes it at least 3000-something vB. This would be the case for any new rule that requires 100 inputs.

So this test could just be deleted until we allow sibling eviction for transactions that don't have this size restriction.

💬 glozow commented on pull request "[WIP] Cluster mempool implementation":
(https://github.com/bitcoin/bitcoin/pull/28676#discussion_r1742345016)
line should be deleted

💬 glozow commented on pull request "[WIP] Cluster mempool implementation":
(https://github.com/bitcoin/bitcoin/pull/28676#discussion_r1742343699)
9047d620b81 similarly
```suggestion
del new_entry["clusterid"]
```

💬 glozow commented on pull request "[WIP] Cluster mempool implementation":
(https://github.com/bitcoin/bitcoin/pull/28676#discussion_r1742419135)
d0576c134a5
I think we only need to adapt 1 of these 2 tests and it should be quite easy to do. Currently, `test_too_many_replacements` requires a high descendant limit because it's 1 parent with 100 children, but we can just add a `self.generate(node, 1)` in between and it can use node 1 (the default one). Then, we can just delete `test_too_many_replacements_with_default_mempool_params`

```
diff --git a/test/functional/feature_rbf.py b/test/functional/feature_rbf.py
index f1de04008c4..578
...

💬 maflcko commented on pull request "doc: Clarify libbitcoin_consensus in design/libraries.md":
(https://github.com/bitcoin/bitcoin/pull/30802#discussion_r1742437523)
Actually, reverted. Can be changed in the future, if there is need to.

👍 hebasto approved a pull request: "doc: Clarify libbitcoin_consensus in design/libraries.md"
(https://github.com/bitcoin/bitcoin/pull/30802#pullrequestreview-2278101828)
ACK fa78ed83be1f6831416a6f6632e2161f12d359e4.

💬 instagibbs commented on pull request "Ephemeral Dust":
(https://github.com/bitcoin/bitcoin/pull/30239#issuecomment-2327111683)
Reworked the PR to relax `IsStandardTx` to allow a single dust output, and hopefully simplified the remaining checks that enforces spend of dusty parent's dust. This should assure the same mempool invariants as before, but with more understandable and minimal changes to logic flow.

@petertodd

> It's more important that this feature can be used securely by L2 implementations, without having to worry about security problems from complex edge cases.

I can't speak for all wallets, but at l
...

💬 instagibbs commented on pull request "cluster mempool: optimized candidate search":
(https://github.com/bitcoin/bitcoin/pull/30286#issuecomment-2327148258)
reACK 100e1b9ecb29c76187112fda9fed1c01da192c99

via `git range-diff master 12760a57b3a6cd1aeb3b7532311f648de2b45aa4 100e1b9ecb29c76187112fda9fed1c01da192c99`

just some comment updates

💬 Sjors commented on pull request "Stratum v2 Template Provider (take 3)":
(https://github.com/bitcoin/bitcoin/pull/29432#issuecomment-2327172907)
1f720ffe25443b37624d4d52e040d29ecf93278a adds a temporary fix for a crash after finding a block. This was introduced by the interface changes. Will investigate later, as part of https://github.com/Sjors/bitcoin/pull/49.

New tag [sv2-tp-0.1.9](https://github.com/Sjors/bitcoin/releases/tag/sv2-tp-0.1.9)

📝 maflcko opened a pull request: "scripted-diff: fuzz: Rename fuzz_seed_corpus to fuzz_corpora"
(https://github.com/bitcoin/bitcoin/pull/30804)
Now that cmake was a breaking change for all fuzz scripts, it seems fine to bundle it with another breaking change to rename the fuzz corpora directory, as discussed and approved in https://github.com/bitcoin-core/qa-assets/issues/200:

* The word "seed" in the old name doesn't really apply. In reality it is a collection of fuzz input seeds, as well as fuzz inputs.
* The rename will also allow in the future (when there is a need and desire) to provide a minimal set of possibly hand-crafted or
...

💬 maflcko commented on pull request "test: fixing failing system_tests/run_command under some Locales":
(https://github.com/bitcoin/bitcoin/pull/30788#issuecomment-2327195870)
review ACK 8289373ebf089d0719a8edecb8bfbd52c244f4ee

Please adjust the pull request description, now that the approach has changed: https://github.com/bitcoin/bitcoin/pull/30788#issue-2500949706

💬 maflcko commented on pull request "Reintroduce external signer support for Windows":
(https://github.com/bitcoin/bitcoin/pull/29868#discussion_r1742523187)
Fixed in https://github.com/bitcoin/bitcoin/pull/30788

💬 achow101 commented on pull request "assumeutxo: Add dumptxoutset height param, remove shell scripts":
(https://github.com/bitcoin/bitcoin/pull/29553#issuecomment-2327237127)
ACK 46c8d75d24d3355ec468f7f608effe94636e16db

🚀 achow101 merged a pull request: "assumeutxo: Add dumptxoutset height param, remove shell scripts"
(https://github.com/bitcoin/bitcoin/pull/29553)

💬 achow101 commented on pull request "kernel: Use spans instead of vectors for passing block headers to validation functions":
(https://github.com/bitcoin/bitcoin/pull/30742#issuecomment-2327273911)
ACK a2955f09792b6232f3a45aa44a498b466279a8b7

🚀 achow101 merged a pull request: "kernel: Use spans instead of vectors for passing block headers to validation functions"
(https://github.com/bitcoin/bitcoin/pull/30742)

💬 achow101 commented on pull request "log: expand BCLog::LogFlags (categories) to 64 bits":
(https://github.com/bitcoin/bitcoin/pull/26619#issuecomment-2327313569)
ACK b31a0cd0378184b2b9eb8f4bd3120cbd32c62005