16:00:08 <glozow> #startmeeting 16:00:08 <corebot> glozow: Meeting started at 2025-03-27T16:00+0000 16:00:09 <corebot> glozow: Current chairs: glozow 16:00:10 <corebot> glozow: Useful commands: #action #info #idea #link #topic #motion #vote #close #endmeeting 16:00:11 <corebot> glozow: See also: https://hcoop-meetbot.readthedocs.io/en/stable/ 16:00:12 <corebot> glozow: Participants should now identify themselves with '#here' or with an alias like '#here FirstLast' 16:00:12 <Murch[m]> hi 16:00:22 <glozow> #bitcoin -core-dev Meeting: abubakarsadiq achow101 _aj_ ajonas b10c brunoerg cfields darosior dergoegge fanquake fjahr furszy gleb glozow hebasto instagibbs jarolrod jonatack josibake kanzure laanwj LarryRuane lightlike luke-jr maflcko marcofleon maxedw Murch pinheadmz provoostenator ryanofsky sdaftuar S3RK stickies-v sipa sr_gi tdb3 theStack TheCharlatan vasild willcl-ark 16:00:23 <laanwj> hi 16:00:27 <cfields> hi 16:00:30 <sr_gi[m]> hi 16:00:38 <stickies-v> hi 16:00:44 <sliv3r__> hi 16:00:49 <brunoerg> hi 16:00:49 <glozow> I see one topic for today. Anything to add? 16:01:06 <furszy> hi 16:01:19 <glozow> #topic Erlay WG Update (sr_gi, gleb, marcofleon) 16:01:49 <rkrux> hi 16:02:06 <sr_gi[m]> I've been updating the approach for the current Erlay PR. Pushed the changes yesterday (#30116) 16:02:08 <corebot> https://github.com/bitcoin/bitcoin/issues/30116 | p2p: Fill reconciliation sets (Erlay) attempt 2 by sr-gi · Pull Request #30116 · bitcoin/bitcoin · GitHub 16:02:12 <fjahr> hi 16:02:16 <theStack> hi 16:02:23 <sr_gi[m]> And I'm currently rebasing the full Erlay implementation on top of it #30277 16:02:25 <corebot> https://github.com/bitcoin/bitcoin/issues/30277 | [DO NOT MERGE] Erlay: bandwidth-efficient transaction relay protocol (Full implementation) by sr-gi · Pull Request #30277 · bitcoin/bitcoin · GitHub 16:02:46 <sr_gi[m]> The goal is making sure this runs as expected to start running Warnet simulations 16:03:09 <sr_gi[m]> So if anyone is interested on the newest approach, take a look at 30116 16:03:43 <sr_gi[m]> That's it for now, hope I can share some interesting results in the following weeks once we start running experiments with actual nodes 16:03:56 <glozow> #topic Kernel WG Update (TheCharlatan) 16:04:02 <TheCharlatan> Had some more discussions on the long-term maintainability of having a C header API for the bitcoinkernel library in #30595. 16:04:04 <corebot> https://github.com/bitcoin/bitcoin/issues/30595 | kernel: Introduce initial C header API by TheCharlatan · Pull Request #30595 · bitcoin/bitcoin · GitHub 16:04:23 <TheCharlatan> Debate is still if a c++ header might be the better choice after all 16:04:34 <TheCharlatan> Thanks to some quick reviews we now also have a simple functional test for bitcoin-chainstate, which could make it a bit easier to ensure the library is actually usable and to validate future changes to the API. 16:04:51 <TheCharlatan> I am still waiting for more reviews on #31382. It is useful, because it cleans up the shutdown logic a bit and gets rid of the last remaining warts in bitcoin-chainstate. 16:04:53 <corebot> https://github.com/bitcoin/bitcoin/issues/31382 | kernel: Flush in ChainstateManager destructor by TheCharlatan · Pull Request #31382 · bitcoin/bitcoin · GitHub 16:05:38 <laanwj> an C API is extremely useful for dynamic loading from non-C++ languages, but it could be done the other way around, wrapping the C++ one i guess 16:07:05 <TheCharlatan> laanwj can you add a comment about that to 30595? 16:07:30 <laanwj> sure 16:07:50 <glozow> #topic QML GUI WG Update (jarolrod, johnny9dev) 16:07:55 <johnny9dev> The first iteration of the Send form was merged this week. bitcoin-core/gui-qml#445 16:07:58 <corebot> https://github.com/bitcoin-core/gui-qml/issues/445 | Introduce Send pages for singlesig, single input/output send by johnny9 · Pull Request #445 · bitcoin-core/gui-qml · GitHub 16:08:05 <johnny9dev> The first iteration of the Coin Selection menu PR was opened. bitcoin-core/gui-qml#448 16:08:07 <corebot> https://github.com/bitcoin-core/gui-qml/issues/448 | Introduce Coin Selection page by johnny9 · Pull Request #448 · bitcoin-core/gui-qml · GitHub 16:08:19 <johnny9dev> Christoph has opened a design discussion on Coin Selection at https://github.com/BitcoinDesign/Bitcoin-Core-App/issues/153 and Michael has provided a nice updated prototype. I encourage anyone interested in Coin Control to take a look and leave a comment if there is anything you would like to see. 16:08:55 <johnny9dev> My plans for the next week are to open up a PR for the multiple recipients form as well as get bitcoin-core/gui-qml#441 and bitcoin-core/gui-qml#434 merged. 16:08:56 <corebot> https://github.com/bitcoin-core/gui-qml/issues/441 | Rename Block Clock to Block Status by johnny9 · Pull Request #441 · bitcoin-core/gui-qml · GitHub 16:08:57 <corebot> https://github.com/bitcoin-core/gui-qml/issues/434 | Move various state computations from the UI to the NodeModel by jarolrod · Pull Request #434 · bitcoin-core/gui-qml · GitHub 16:09:48 <glozow> #topic Script Validation WG Update (fjahr) 16:10:10 <fjahr> #31689 has been merged and I am currently working on adding tests for #29491 which has seen some discussion on the approach. Happy to get more eyes on it so that we know early enough if we want changes to the secp API. 16:10:13 <corebot> https://github.com/bitcoin/bitcoin/issues/31689 | Benchmark Chainstate::ConnectBlock duration by Eunovo · Pull Request #31689 · bitcoin/bitcoin · GitHub 16:10:15 <corebot> https://github.com/bitcoin/bitcoin/issues/29491 | [EXPERIMENTAL] Schnorr batch verification for blocks by fjahr · Pull Request #29491 · bitcoin/bitcoin · GitHub 16:10:20 <fjahr> That's it from me. 16:11:00 <glozow> furszy: did you have wallet updates? 16:11:10 <glozow> #topic Cluster Mempool WG Update (sdaftuar, sipa) 16:11:15 <sipa> hi 16:11:46 <sipa> the first txgraph PR, #31363 got merged, hurray 16:11:48 <corebot> https://github.com/bitcoin/bitcoin/issues/31363 | cluster mempool: introduce TxGraph by sipa · Pull Request #31363 · bitcoin/bitcoin · GitHub 16:11:52 <instagibbs> \o/ 16:11:54 <Sjors[m]> hi 16:11:56 <glozow> \o/ 16:12:23 <TheCharlatan> \o/ 16:12:31 <sipa> next one in line adds support for feerate diagram comparisons, block building, and eviction: #31444 16:12:33 <corebot> https://github.com/bitcoin/bitcoin/issues/31444 | cluster mempool: add txgraph diagrams/mining/eviction by sipa · Pull Request #31444 · bitcoin/bitcoin · GitHub 16:12:54 <sipa> i've also opened #32151 with a few small follow-ups 16:12:55 <corebot> https://github.com/bitcoin/bitcoin/issues/32151 | Follow-ups for txgraph #31363 by sipa · Pull Request #32151 · bitcoin/bitcoin · GitHub 16:14:09 <sipa> on the research front, i've implemented the min-cut based algorithm from the 1989 paper, which seems to work very well (10s of microseconds for all our "hard" clusters in the benchmark) 16:15:09 <sipa> if the paper, and my implementation of it, is correct, it has O(n^3) complexity, though the worst cases are still ~100x slower than our actual benchmarks, so worst case runtimes would be in the milliseconds range 16:15:22 <sipa> lots of open questions, but very promising 16:15:57 <sipa> that's it for me 16:16:00 <furszy> glozow: Only legacy wallet removal update is #32149 finding. Other than that, #31250 still needs a rebase. 16:16:01 <corebot> https://github.com/bitcoin/bitcoin/issues/32149 | wallet, migration: Fix empty wallet crash by pablomartin4btc · Pull Request #32149 · bitcoin/bitcoin · GitHub 16:16:04 <corebot> https://github.com/bitcoin/bitcoin/issues/31250 | wallet: Disable creating and loading legacy wallets by achow101 · Pull Request #31250 · bitcoin/bitcoin · GitHub 16:16:10 <glozow> furszy: thanks 16:16:15 <glozow> #topic Stratum v2 WG Update (sjors) 16:16:41 <Sjors[m]> Not much changed since last week, mostly hoping to see multiprocess stuff landing soon 16:17:04 <Sjors[m]> Particularly #31741 16:17:07 <corebot> https://github.com/bitcoin/bitcoin/issues/31741 | multiprocess: Add libmultiprocess git subtree by ryanofsky · Pull Request #31741 · bitcoin/bitcoin · GitHub 16:17:30 <TheCharlatan> are we blocked on #31992 there? 16:17:34 <corebot> https://github.com/bitcoin/bitcoin/issues/31992 | cmake: Avoid fuzzer "multiple definition of `main'" errors by ryanofsky · Pull Request #31992 · bitcoin/bitcoin · GitHub 16:18:20 <ryanofsky> Not really, I could drop it from 31741 if needed. 31992 is just needed to enable more checks in CI 16:19:38 <glozow> #topic 29.0 release 16:19:49 <glozow> Reminder to test rc2 16:20:05 <glozow> Anything else to discuss? 16:20:45 <sipa> i proposed a topic 16:21:00 <glozow> oh yes sorry, I even had it written here 16:21:21 <glozow> #topic Snap and bitcoin.org stuck on 27.x (sipa) 16:21:43 <sipa> hi, i noticed that for unrelated reasons, both our snap package and the bitcoin.org website are stuck on 27.x releases 16:22:17 <sipa> i see there is https://github.com/bitcoin-core/packaging/pull/248 now to update snap to 28.x, so it seems there is work on that front 16:22:33 <sipa> just wanted to bring this to people's attention 16:23:07 <sipa> there is https://github.com/bitcoin-dot-org/Bitcoin.org/pull/4349 to add 28.1 to bitcoin.org too, which no activity 16:23:19 <sipa> but not sure what can be done about it 16:24:29 <sipa> that's it, unless people have opinions/ideas 16:24:47 <laanwj> not sure, i revoked my access to bitcoin.org long ago, and no idea who's maintaining it nowadays 16:25:47 <glozow> last update was by cobra in january? 16:26:16 <sipa> yeah 16:27:57 <laanwj> could try contacting cobra about it, but they're always difficult 16:28:25 <glozow> maybe give it a shot? 16:28:52 <laanwj> sure 16:28:57 <glozow> laanwj: thank you! 16:29:45 <glozow> Anything else to discuss? 16:30:02 <stickies-v> i'll leave a review on https://github.com/bitcoin-dot-org/Bitcoin.org/pull/4349 for more visibility/activity 16:30:13 <sipa> it seems like 27.1, 27.2, and 28.0 were never even attempted to be PR'ed, but if these PR's don't get merged, there is little point 16:30:51 <sipa> laanwj: thanks! 16:31:01 <glozow> stickies-v: that seems like a good idea! 16:32:19 <glozow> #endmeeting