📝 yancyribbens opened a pull request: "docs: add doc comment for SRD selection algorithm"
(https://github.com/bitcoin/bitcoin/pull/33622)
Other selection algorithms provide a description of workings and parameters. This PR adds such a comment for SRD.

👍 TheCharlatan approved a pull request: "multiprocess: Fix high overhead from message logging"
(https://github.com/bitcoin/bitcoin/pull/33517#pullrequestreview-3335516834)
ACK 0626b90f507db68610a69feec86deb712dd095a1

📝 willcl-ark opened a pull request: "doc: document capnproto and libmultiprocess deps in 29.x"
(https://github.com/bitcoin/bitcoin/pull/33623)
Closes: #33576

These dependencies are both currently undocumented, and libmultiprocess has a relatively special requirement in that the documented version is the **latest** that will ever work with this version of Bitcoin Core.

💬 vasild commented on pull request "net_processing: rename RelayTransaction to better describe what it does":
(https://github.com/bitcoin/bitcoin/pull/33565#discussion_r2429141020)
Change to `InitiateTxBroadcastToAll()`?

💬 sipa commented on pull request "doc: document capnproto and libmultiprocess deps in 29.x":
(https://github.com/bitcoin/bitcoin/pull/33623#issuecomment-3401795935)
Concept ACK

🤔 sipa reviewed a pull request: "multiprocess: Fix high overhead from message logging"
(https://github.com/bitcoin/bitcoin/pull/33517#pullrequestreview-3335624700)
utACK 0626b90f507db68610a69feec86deb712dd095a1

💬 sipa commented on pull request "test, refactor: Embedded ASmap selected preparatory work":
(https://github.com/bitcoin/bitcoin/pull/33026#discussion_r2429188245)
Sounds like we need to agree on what approach to take here before this can move forward?

💬 achow101 commented on issue "`generatetoaddress` 2-3x slower on v30 compared to v29":
(https://github.com/bitcoin/bitcoin/issues/33618#issuecomment-3401869814)
> It seems the commit that changed the behavior was [7bacabb](https://github.com/bitcoin/bitcoin/commit/7bacabb204b6c34f9545f0b37e2c66296ad2c0de)

That doesn't make any sense. That commit doesn't touch anything related to how addresses are generated.

✅ stickies-v closed a pull request: "rest/rpc: use more util::Join"
(https://github.com/bitcoin/bitcoin/pull/32942)

💬 stickies-v commented on pull request "rest/rpc: use more util::Join":
(https://github.com/bitcoin/bitcoin/pull/32942#issuecomment-3401889578)
Closing for lack of reviewer interest, it's not a critical change.

💬 sipa commented on issue "`generatetoaddress` 2-3x slower on v30 compared to v29":
(https://github.com/bitcoin/bitcoin/issues/33618#issuecomment-3401891333)
@achow101 But it does touch things related to new blocks being created. Maybe the overhead of wallet writing the best block to the wallet more frequently has an impact here?

✅ stickies-v closed a pull request: "cmake: fatal error when PIE not supported"
(https://github.com/bitcoin/bitcoin/pull/33282)

💬 stickies-v commented on pull request "cmake: fatal error when PIE not supported":
(https://github.com/bitcoin/bitcoin/pull/33282#issuecomment-3401891361)
Closing for lack of reviewer interest, it's not a critical change.

✅ stickies-v closed a pull request: "cmake: make missing Python interpreter behaviour more explicit"
(https://github.com/bitcoin/bitcoin/pull/33278)

💬 stickies-v commented on pull request "cmake: make missing Python interpreter behaviour more explicit":
(https://github.com/bitcoin/bitcoin/pull/33278#issuecomment-3401892689)
Closing for lack of reviewer interest, it's not a critical change.

💬 purpleKarrot commented on pull request "Update `minisketch` subtree and switch to its build script":
(https://github.com/bitcoin/bitcoin/pull/32856#discussion_r2429246951)
I am not suggesting to use `block()` here. What I am suggesting is to use `function()` in a way that can be replaced with `block()` as soon as the minimum required version can be increased. Since we already know the direction of development, why add more legacy code just for the sake of consistency?

💬 fjahr commented on pull request "test, refactor: Embedded ASmap selected preparatory work":
(https://github.com/bitcoin/bitcoin/pull/33026#discussion_r2429247548)
There has been some further discussion on this in #33386 and I wasn't able to address it yet. I will drop the commit from here and open two separate PRs because there doesn't seem to be a clear outcome of the discussion yet. One adding the commit here and improving docs and the other splitting asmap= into asmap=/asmapfile= which is certainly the cleaner alternative but changes behavior and it's a bit messy to redefine behavior of asmap= just slightly.

💬 furszy commented on issue "`generatetoaddress` 2-3x slower on v30 compared to v29":
(https://github.com/bitcoin/bitcoin/issues/33618#issuecomment-3401965782)
> But it does touch things related to new blocks being created. Maybe the overhead of wallet writing the best block to the wallet more frequently has an impact here?

we might want to revive #25297 in that case.

💬 fjahr commented on pull request "test, refactor: Embedded ASmap selected preparatory work":
(https://github.com/bitcoin/bitcoin/pull/33026#issuecomment-3401968500)
No changes except for dropping the contentious change to args behavior.

🤔 sipa reviewed a pull request: "test, refactor: Embedded ASmap selected preparatory work"
(https://github.com/bitcoin/bitcoin/pull/33026#pullrequestreview-3335793567)
ACK 5bb456c92bbd27ebbba1773832b051968f162b8a