💬 sipa commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#issuecomment-3563746342)
@instagibbs After discussing this offline with @sdaftuar, I think the complexity here is due to the combination of a number of requirements:
* We want to bound the latency of a single `TxGraph::DoWork()` invocation (to something in the order of milliseconds).
* We want to bound what percentage of CPU time in total is spent by `TxGraph`'s linearization efforts (inside `DoWork()`, or elsewhere), over long periods of time.
* In practice get everything optimal.

The current code, which sets `PO
...

🤔 ismaelsadeeq reviewed a pull request: "mining: add getMemoryLoad() and track template non-mempool memory footprint"
(https://github.com/bitcoin/bitcoin/pull/33922#pullrequestreview-3493499544)
Concept ACK

I think it would be better if we have internal memory management for the mining interface IPC, since we hold on to the block templates.

I would suggest the following approach:

- Add memory budget for the mining interface.

- Introduce a tracking list of recently built block templates and total memory usahe.

- Add templates to the list and increment the memory usage after every `createnewblock` or `waitnext` return.

- Whenever the memory budget is exhausted, we should release tem
...

💬 l0rinc commented on pull request "miniscript refactor: Remove unique_ptr-indirection (#30866 follow-up)":
(https://github.com/bitcoin/bitcoin/pull/31713#issuecomment-3563782644)
reACK d782bffc5461f7361ae410963466ae1577ef63df

The only [difference](https://github.com/bitcoin/bitcoin/compare/c6ac239124b3e66f2d5ff52edc6ff422926abead..d782bffc5461f7361ae410963466ae1577ef63df) since last ACK was [to omit adding empty leaves](https://github.com/bitcoin/bitcoin/pull/31713#discussion_r2541499665)

💬 Sjors commented on pull request "mining: add getMemoryLoad() and track template non-mempool memory footprint":
(https://github.com/bitcoin/bitcoin/pull/33922#issuecomment-3563799563)
> In my opinion, we should not rely on the IPC client to manage our memory.

> Whenever the memory budget is exhausted, we should release templates in FIFO order

It seems counter intuitive, but from a memory management perspective IPC clients are treated no different than our own code. And if we started FIFO deleting templates that are used by our own code, we'd crash.

So I think FIFO deletion should be a last resort (not implemented here).

There's another reason why we should give cl
...

💬 fjahr commented on pull request "CI: silent merge check":
(https://github.com/bitcoin/bitcoin/pull/33145#issuecomment-3563843760)
Concept ACK

💬 maflcko commented on pull request "CI: silent merge check":
(https://github.com/bitcoin/bitcoin/pull/33145#issuecomment-3563863948)
I think what could work (hacky, untested):

* Setup a separate "silent-merge-check" repo with the desired ci config
* Push the selected ci runs to it on the desired schedule
* Get the result, and on failure, pass it back to the upstream repo

💬 fanquake commented on pull request "ci: Add IWYU job":
(https://github.com/bitcoin/bitcoin/pull/33810#discussion_r2550408178)
I'm wondering, give we are building IWYU from source, can we just patch it, so the output is correct (uses <>); avoiding the need for this fixup.

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550482102)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550482202)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550482323)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550482467)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550482573)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550483077)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550483175)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550483287)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550483371)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550483458)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550483537)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550483605)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09

💬 kevkevinpal commented on pull request "wallet/refactor: change PSBTError to PSBTResult and remove std::optional<common::PSBTResult> and return common::PSBTResult":
(https://github.com/bitcoin/bitcoin/pull/32958#discussion_r2550483677)
Thanks, updated in 4e038c4b7a6fbfaafe1f2570592e2bb751f9be09