achow101 changed the topic of #bitcoin-core-dev to: Bitcoin Core development discussion and commit log | Feel free to watch, but please take commentary and usage questions to #bitcoin | Channel logs: http://www.erisian.com.au/bitcoin-core-dev/, http://gnusha.org/bitcoin-core-dev/ | Weekly Meeting Thursday @ 16:00 UTC | Meeting topics http://gnusha.org/bitcoin-core-dev/proposedmeetingtopics.txt
salvatoshi__ has quit [Ping timeout: 248 seconds]
kevkevin has joined #bitcoin-core-dev
flag has quit [Ping timeout: 244 seconds]
flag has joined #bitcoin-core-dev
cold has quit [Quit: Quitting...]
cold has joined #bitcoin-core-dev
zeropoint has quit [Quit: leaving]
pablomartin has joined #bitcoin-core-dev
abubakarsadiq has quit [Quit: Connection closed for inactivity]
pablomartin has quit [Ping timeout: 244 seconds]
pablomartin has joined #bitcoin-core-dev
robszarka has joined #bitcoin-core-dev
szarka has quit [Ping timeout: 268 seconds]
pablomartin has quit [Remote host closed the connection]
nanotube has quit [Quit: *poof*]
<laanwj> lightlike: i'm not sure, maybe if the warnings are clear enough? wouldn't want "just don't produce the binary" leading to accidentally pass the tests in the CI instead of flag a problem
<laanwj> acting based on what should be there is not bad behavior for tests
asini_ has joined #bitcoin-core-dev
aLeSD has quit [Ping timeout: 244 seconds]
asini__ has joined #bitcoin-core-dev
asini_ has quit [Ping timeout: 260 seconds]
joetor5 has joined #bitcoin-core-dev
flag has quit [Ping timeout: 252 seconds]
flag has joined #bitcoin-core-dev
joetor5 has quit [Quit: joetor5]
pablomartin has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] pablomartin4btc opened pull request #32149: wallet, migration: Fix empty wallet crash (master...wallet-migration-empty-wallet-crash-fix) https://github.com/bitcoin/bitcoin/pull/32149
* Artea is away, afk since 03:07:10pm [26-03-2025 UTC+0] (for 11hrs 24mins 58secs)
<bitcoin-git> [bitcoin] murchandamus opened pull request #32150: refactor: Optimize BnB exploration (master...2025-03-rewrite-BnB) https://github.com/bitcoin/bitcoin/pull/32150
* Artea is away, afk since 03:07:10pm [26-03-2025 UTC+0] (for 11hrs 26mins 38secs)
Artea has quit [Killed (ozone (No Spam))]
javi404 has joined #bitcoin-core-dev
robszarka has quit [Quit: Leaving]
szarka has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] sipa opened pull request #32151: Follow-ups for txgraph #31363 (master...202403_txgraph_followups) https://github.com/bitcoin/bitcoin/pull/32151
pablomartin has quit [Ping timeout: 244 seconds]
nanotube has joined #bitcoin-core-dev
mcey_ has joined #bitcoin-core-dev
mcey has quit [Ping timeout: 260 seconds]
abubakarsadiq has joined #bitcoin-core-dev
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
MrHAPPY has quit []
Artea has joined #bitcoin-core-dev
bitdex has quit [Read error: Connection reset by peer]
bitdex has joined #bitcoin-core-dev
sliv3r__ has quit [Quit: ZNC 1.8.2+deb3.1+deb12u1 - https://znc.in]
sliv3r__ has joined #bitcoin-core-dev
vasild has quit [Remote host closed the connection]
SpellChecker has quit [Read error: Connection reset by peer]
vasild has joined #bitcoin-core-dev
SpellChecker has joined #bitcoin-core-dev
abubakarsadiq has quit [Quit: Connection closed for inactivity]
asini__ has quit [Ping timeout: 252 seconds]
bitdex has quit [Remote host closed the connection]
bitdex has joined #bitcoin-core-dev
asini__ has joined #bitcoin-core-dev
Christoph_ has joined #bitcoin-core-dev
asini__ has quit [Quit: Leaving]
aLeSD has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/f1d129d96340...6971d3a0f52e
<bitcoin-git> bitcoin/master e3ce2bd dennsikl: Remove needless borrow to fix Clippy warning
<bitcoin-git> bitcoin/master 6971d3a merge-script: Merge bitcoin/bitcoin#32144: lint: Remove needless borrow to fix Clippy wa...
<bitcoin-git> [bitcoin] fanquake merged pull request #32144: lint: Remove needless borrow to fix Clippy warning (master...master) https://github.com/bitcoin/bitcoin/pull/32144
andytoshi has quit [Ping timeout: 260 seconds]
andytoshi has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/6971d3a0f52e...84bbb40558f4
<bitcoin-git> bitcoin/master fb2b05b Hennadii Stepanov: build: Remove bitness suffix from Windows installer
<bitcoin-git> bitcoin/master 84bbb40 merge-script: Merge bitcoin/bitcoin#32132: build: Remove bitness suffix from Windows ins...
<bitcoin-git> [bitcoin] fanquake merged pull request #32132: build: Remove bitness suffix from Windows installer (master...250324-installer) https://github.com/bitcoin/bitcoin/pull/32132
<bitcoin-git> [bitcoin] fanquake pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/84bbb40558f4...603fcc07d595
<bitcoin-git> bitcoin/master fa29842 MarcoFalke: refactor: Remove IsArgSet guard when fallback value is provided
<bitcoin-git> bitcoin/master fa2b529 MarcoFalke: refactor: Remove redundant call to IsArgSet
<bitcoin-git> bitcoin/master 0000fb3 MarcoFalke: doc: Remove outdated and stale todo comment
<bitcoin-git> [bitcoin] fanquake merged pull request #31896: refactor: Remove redundant and confusing calls to IsArgSet (master...2502-less-is-arg-set) https://github.com/bitcoin/bitcoin/pull/31896
salvatoshi__ has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/603fcc07d595...e563cb5c6062
<bitcoin-git> bitcoin/master 9633550 fanquake: depends: set CMAKE_*_COMPILER_TARGET in toolchain
<bitcoin-git> bitcoin/master e563cb5 merge-script: Merge bitcoin/bitcoin#31849: depends: set `CMAKE_*_COMPILER_TARGET` in too...
<bitcoin-git> [bitcoin] fanquake merged pull request #31849: depends: set `CMAKE_*_COMPILER_TARGET` in toolchain (master...clang_cross_toolchain) https://github.com/bitcoin/bitcoin/pull/31849
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/e563cb5c6062...b413b088ae89
<bitcoin-git> bitcoin/master b1de59e Lőrinc: fuzz: extract unsequenced operations with side-effects
<bitcoin-git> bitcoin/master b413b08 merge-script: Merge bitcoin/bitcoin#32141: fuzz: extract unsequenced operations with sid...
<bitcoin-git> [bitcoin] fanquake merged pull request #32141: fuzz: extract unsequenced operations with side-effects (master...l0rinc/psbt-fuzz-fix) https://github.com/bitcoin/bitcoin/pull/32141
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/b413b088ae89...5dd6ebc7e139
<bitcoin-git> bitcoin/master 9f35d4d Martin Zumsande: test: fix intermittent timeout in p2p_ibd_stalling.py
<bitcoin-git> bitcoin/master 5dd6ebc merge-script: Merge bitcoin/bitcoin#32148: test: fix intermittent timeout in p2p_ibd_sta...
<bitcoin-git> [bitcoin] fanquake merged pull request #32148: test: fix intermittent timeout in p2p_ibd_stalling.py (master...202503_fix_ibd_stalling) https://github.com/bitcoin/bitcoin/pull/32148
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/5dd6ebc7e139...140f0d89bf68
<bitcoin-git> bitcoin/master 52ac177 Hennadii Stepanov: cmake: Add `NO_CACHE_IF_FAILED` option for checking linker flags
<bitcoin-git> bitcoin/master 140f0d8 merge-script: Merge bitcoin/bitcoin#32027: cmake: Add `NO_CACHE_IF_FAILED` option for ch...
<bitcoin-git> [bitcoin] fanquake merged pull request #32027: cmake: Add `NO_CACHE_IF_FAILED` option for checking linker flags (master...250310-nocache) https://github.com/bitcoin/bitcoin/pull/32027
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/140f0d89bf68...8cc601196b9d
<bitcoin-git> bitcoin/master 52ede28 Anthony Towns: doc: Update comments for AreInputsStandard to match code
<bitcoin-git> bitcoin/master 8cc6011 merge-script: Merge bitcoin/bitcoin#32129: doc: Update comments for AreInputsStandard to...
<bitcoin-git> [bitcoin] fanquake merged pull request #32129: doc: Update comments for AreInputsStandard to match code (master...202503-areinputstd-comment) https://github.com/bitcoin/bitcoin/pull/32129
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/8cc601196b9d...b131e1bfc072
<bitcoin-git> bitcoin/master 248fdd8 willcl-ark: test: accept unordered tracepoints in...
<bitcoin-git> bitcoin/master b131e1b merge-script: Merge bitcoin/bitcoin#32101: Accept unordered tracepoints in interface_usd...
<bitcoin-git> [bitcoin] fanquake merged pull request #32101: Accept unordered tracepoints in interface_usdt_utxocache.py (master...usdt-order-independent) https://github.com/bitcoin/bitcoin/pull/32101
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/b131e1bfc072...a54baa869814
<bitcoin-git> bitcoin/master 329a0dc ismaelsadeeq: doc: clarify the documentation of `Assume`
<bitcoin-git> bitcoin/master a54baa8 merge-script: Merge bitcoin/bitcoin#32100: doc: clarify the documentation of `Assume` as...
<bitcoin-git> [bitcoin] fanquake merged pull request #32100: doc: clarify the documentation of `Assume` assertion (master...03-2025-clarify-assume-doc) https://github.com/bitcoin/bitcoin/pull/32100
salvatoshi__ has quit [Ping timeout: 276 seconds]
salvatoshi__ has joined #bitcoin-core-dev
aLeSD has quit [Remote host closed the connection]
jurraca has quit [Quit: leaving]
Guest28 has joined #bitcoin-core-dev
cold has quit [Ping timeout: 246 seconds]
midnight has quit [Ping timeout: 260 seconds]
Guest28 has quit [Quit: Client closed]
midnight has joined #bitcoin-core-dev
cold has joined #bitcoin-core-dev
Guyver2 has joined #bitcoin-core-dev
SpellChecker has quit [Remote host closed the connection]
SpellChecker has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] musaHaruna opened pull request #32152: test: add p2pk support to wallet_implicity_segwit.py (master...add-p2pk-support-to-wallet-implicity-segwit-test) https://github.com/bitcoin/bitcoin/pull/32152
yuvic has joined #bitcoin-core-dev
yuvic has joined #bitcoin-core-dev
yuvic has quit [Remote host closed the connection]
<bitcoin-git> [bitcoin] rkrux opened pull request #32153: wallet: remove redundant `Assert` call when block is disconnected (master...wallet-block) https://github.com/bitcoin/bitcoin/pull/32153
yuvic has joined #bitcoin-core-dev
yuvic has joined #bitcoin-core-dev
yuvic has quit [Client Quit]
aLeSD has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] maflcko opened pull request #32154: fuzz: Avoid integer sanitizer warnings in policy_estimator target (master...2503-fuzz-isan) https://github.com/bitcoin/bitcoin/pull/32154
Christoph_ has quit [Quit: Christoph_]
Christoph_ has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] maflcko closed pull request #32152: test: add p2pk support to wallet_implicity_segwit.py (master...add-p2pk-support-to-wallet-implicity-segwit-test) https://github.com/bitcoin/bitcoin/pull/32152
ubbabeck has quit [Ping timeout: 245 seconds]
Saturday7 has quit [Ping timeout: 248 seconds]
Christoph_ has quit [Ping timeout: 260 seconds]
Christoph_ has joined #bitcoin-core-dev
Saturday7 has joined #bitcoin-core-dev
aLeSD has quit [Remote host closed the connection]
dermoth has quit [Ping timeout: 260 seconds]
abubakarsadiq has joined #bitcoin-core-dev
dermoth has joined #bitcoin-core-dev
pablomartin has joined #bitcoin-core-dev
pablomartin has quit [Remote host closed the connection]
jespada has joined #bitcoin-core-dev
pablomartin has joined #bitcoin-core-dev
pablomartin4btc_ has joined #bitcoin-core-dev
pablomartin4btc has quit [Ping timeout: 244 seconds]
Christoph_ has quit [Quit: Christoph_]
Christoph_ has joined #bitcoin-core-dev
andytosh1 has joined #bitcoin-core-dev
andytoshi has quit [Ping timeout: 245 seconds]
OGU has joined #bitcoin-core-dev
OGU has quit [Changing host]
OGU has joined #bitcoin-core-dev
pablomartin4btc has joined #bitcoin-core-dev
pyth has quit [Quit: Leaving]
bitdex has quit [Quit: = ""]
pablomartin4btc_ has quit [Ping timeout: 245 seconds]
Cory21 has quit [Quit: Client closed]
Cory21 has joined #bitcoin-core-dev
codingp110 has joined #bitcoin-core-dev
jespada has quit [Quit: My Mac has gone to sleep. ZZZzzz…]
codingp110 has quit [Client Quit]
jespada has joined #bitcoin-core-dev
sliv3r__ has quit [Quit: ZNC 1.8.2+deb3.1+deb12u1 - https://znc.in]
sliv3r__ has joined #bitcoin-core-dev
MrHAPPY has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] ryanofsky pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/a54baa869814...bcb316bd888e
<bitcoin-git> bitcoin/master 346a099 Sebastian Falbesoner: test: avoid unneeded hash -> uint256 -> hash roundtrips
<bitcoin-git> bitcoin/master a82829f Sebastian Falbesoner: test: simplify (w)txid checks by avoiding .calc_sha256 calls
<bitcoin-git> bitcoin/master bcb316b Ryan Ofsky: Merge bitcoin/bitcoin#32050: test: avoid treating hash results as integers
<bitcoin-git> [bitcoin] ryanofsky merged pull request #32050: test: avoid treating hash results as integers (master...202503-test-avoid_hash_as_int) https://github.com/bitcoin/bitcoin/pull/32050
MrHAPPY has quit [Ping timeout: 246 seconds]
MrHAPPY has joined #bitcoin-core-dev
jespada has quit [Ping timeout: 260 seconds]
jespada has joined #bitcoin-core-dev
MrHAPPY has quit [Ping timeout: 252 seconds]
Christoph_ has quit [Quit: Christoph_]
Christoph_ has joined #bitcoin-core-dev
MrHAPPY has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] ryanofsky pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/bcb316bd888e...74c23f80abbe
<bitcoin-git> bitcoin/master 3f9c716 TheCharlatan: test: Fix docstring for cmake migration
<bitcoin-git> bitcoin/master ca55613 TheCharlatan: test: Add functional test for bitcoin-chainstate
<bitcoin-git> bitcoin/master 74c23f8 Ryan Ofsky: Merge bitcoin/bitcoin#32145: test: Add functional test for bitcoin-chainst...
<bitcoin-git> [bitcoin] ryanofsky merged pull request #32145: test: Add functional test for bitcoin-chainstate (master...BitcoinChainstateFunctionalTest) https://github.com/bitcoin/bitcoin/pull/32145
pablomartin4btc has quit [Remote host closed the connection]
<bitcoin-git> [qa-assets] dergoegge merged pull request #221: Delete nonreduced fuzz inputs (main...main) https://github.com/bitcoin-core/qa-assets/pull/221
<bitcoin-git> [qa-assets] dergoegge pushed 5 commits to main: https://github.com/bitcoin-core/qa-assets/compare/c62dc520e0c7...12d26e3fb87f
<bitcoin-git> qa-assets/main 20e827c delete_nonreduced_inputs script: Delete fuzz inputs
<bitcoin-git> qa-assets/main e3b4a14 delete_nonreduced_inputs script: Reduced inputs for afl-cmin
<bitcoin-git> qa-assets/main 518a3d8 delete_nonreduced_inputs script: Reduced inputs for fuzzer
andytosh1 is now known as andytoshi
bugs_ has joined #bitcoin-core-dev
<bitcoin-git> [qa-assets] maflcko opened pull request #222: add back temporarily removed fuzz inputs (main...main) https://github.com/bitcoin-core/qa-assets/pull/222
jespada has quit [Quit: My Mac has gone to sleep. ZZZzzz…]
zeropoint has joined #bitcoin-core-dev
Guyver2 has left #bitcoin-core-dev [Closing Window]
Emc99 has joined #bitcoin-core-dev
<glozow> hi
<johnny9dev> hi
<sipa> hi
<lightlike> Hi
<TheCharlatan> hi
<glozow> #startmeeting
<corebot> glozow: Meeting started at 2025-03-27T16:00+0000
<corebot> glozow: Current chairs: glozow
<corebot> glozow: Useful commands: #action #info #idea #link #topic #motion #vote #close #endmeeting
<corebot> glozow: Participants should now identify themselves with '#here' or with an alias like '#here FirstLast'
<Murch[m]> hi
ubbabeck has joined #bitcoin-core-dev
brunoerg_ has quit []
<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
<laanwj> hi
brunoerg has joined #bitcoin-core-dev
<cfields> hi
<sr_gi[m]> hi
<stickies-v> hi
<sliv3r__> hi
<brunoerg> hi
<glozow> I see one topic for today. Anything to add?
<furszy> hi
<glozow> #topic Erlay WG Update (sr_gi, gleb, marcofleon)
rkrux has joined #bitcoin-core-dev
<rkrux> hi
<sr_gi[m]> I've been updating the approach for the current Erlay PR. Pushed the changes yesterday (#30116)
<corebot> https://github.com/bitcoin/bitcoin/issues/30116 | p2p: Fill reconciliation sets (Erlay) attempt 2 by sr-gi · Pull Request #30116 · bitcoin/bitcoin · GitHub
<fjahr> hi
<theStack> hi
<sr_gi[m]> And I'm currently rebasing the full Erlay implementation on top of it #30277
<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
<sr_gi[m]> The goal is making sure this runs as expected to start running Warnet simulations
<sr_gi[m]> So if anyone is interested on the newest approach, take a look at 30116
<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
<glozow> #topic Kernel WG Update (TheCharlatan)
<TheCharlatan> Had some more discussions on the long-term maintainability of having a C header API for the bitcoinkernel library in #30595.
<corebot> https://github.com/bitcoin/bitcoin/issues/30595 | kernel: Introduce initial C header API by TheCharlatan · Pull Request #30595 · bitcoin/bitcoin · GitHub
<TheCharlatan> Debate is still if a c++ header might be the better choice after all
<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.
<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.
<corebot> https://github.com/bitcoin/bitcoin/issues/31382 | kernel: Flush in ChainstateManager destructor by TheCharlatan · Pull Request #31382 · bitcoin/bitcoin · GitHub
<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
rkrux has quit [Ping timeout: 240 seconds]
<TheCharlatan> laanwj can you add a comment about that to 30595?
<laanwj> sure
<glozow> #topic QML GUI WG Update (jarolrod, johnny9dev)
<johnny9dev> The first iteration of the Send form was merged this week. bitcoin-core/gui-qml#445
<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
<johnny9dev> The first iteration of the Coin Selection menu PR was opened. bitcoin-core/gui-qml#448
<corebot> https://github.com/bitcoin-core/gui-qml/issues/448 | Introduce Coin Selection page by johnny9 · Pull Request #448 · bitcoin-core/gui-qml · GitHub
<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.
<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.
rkrux has joined #bitcoin-core-dev
<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
<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
<glozow> #topic Script Validation WG Update (fjahr)
<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.
<corebot> https://github.com/bitcoin/bitcoin/issues/31689 | Benchmark Chainstate::ConnectBlock duration by Eunovo · Pull Request #31689 · bitcoin/bitcoin · GitHub
<corebot> https://github.com/bitcoin/bitcoin/issues/29491 | [EXPERIMENTAL] Schnorr batch verification for blocks by fjahr · Pull Request #29491 · bitcoin/bitcoin · GitHub
<fjahr> That's it from me.
<glozow> furszy: did you have wallet updates?
<glozow> #topic Cluster Mempool WG Update (sdaftuar, sipa)
<sipa> hi
<sipa> the first txgraph PR, #31363 got merged, hurray
<corebot> https://github.com/bitcoin/bitcoin/issues/31363 | cluster mempool: introduce TxGraph by sipa · Pull Request #31363 · bitcoin/bitcoin · GitHub
<instagibbs> \o/
<Sjors[m]> hi
<glozow> \o/
<TheCharlatan> \o/
<sipa> next one in line adds support for feerate diagram comparisons, block building, and eviction: #31444
<corebot> https://github.com/bitcoin/bitcoin/issues/31444 | cluster mempool: add txgraph diagrams/mining/eviction by sipa · Pull Request #31444 · bitcoin/bitcoin · GitHub
<sipa> i've also opened #32151 with a few small follow-ups
<corebot> https://github.com/bitcoin/bitcoin/issues/32151 | Follow-ups for txgraph #31363 by sipa · Pull Request #32151 · bitcoin/bitcoin · GitHub
<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)
mentalbarcode has joined #bitcoin-core-dev
<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
<sipa> lots of open questions, but very promising
<sipa> that's it for me
<furszy> glozow: Only legacy wallet removal update is #32149 finding. Other than that, #31250 still needs a rebase.
<corebot> https://github.com/bitcoin/bitcoin/issues/32149 | wallet, migration: Fix empty wallet crash by pablomartin4btc · Pull Request #32149 · bitcoin/bitcoin · GitHub
<corebot> https://github.com/bitcoin/bitcoin/issues/31250 | wallet: Disable creating and loading legacy wallets by achow101 · Pull Request #31250 · bitcoin/bitcoin · GitHub
<glozow> furszy: thanks
<glozow> #topic Stratum v2 WG Update (sjors)
<Sjors[m]> Not much changed since last week, mostly hoping to see multiprocess stuff landing soon
<Sjors[m]> Particularly #31741
<corebot> https://github.com/bitcoin/bitcoin/issues/31741 | multiprocess: Add libmultiprocess git subtree by ryanofsky · Pull Request #31741 · bitcoin/bitcoin · GitHub
<TheCharlatan> are we blocked on #31992 there?
<corebot> https://github.com/bitcoin/bitcoin/issues/31992 | cmake: Avoid fuzzer "multiple definition of `main'" errors by ryanofsky · Pull Request #31992 · bitcoin/bitcoin · GitHub
<ryanofsky> Not really, I could drop it from 31741 if needed. 31992 is just needed to enable more checks in CI
<glozow> #topic 29.0 release
<glozow> Reminder to test rc2
<glozow> Anything else to discuss?
<sipa> i proposed a topic
<glozow> oh yes sorry, I even had it written here
<glozow> #topic Snap and bitcoin.org stuck on 27.x (sipa)
<sipa> hi, i noticed that for unrelated reasons, both our snap package and the bitcoin.org website are stuck on 27.x releases
<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
<sipa> just wanted to bring this to people's attention
<sipa> there is https://github.com/bitcoin-dot-org/Bitcoin.org/pull/4349 to add 28.1 to bitcoin.org too, which no activity
<sipa> but not sure what can be done about it
<sipa> that's it, unless people have opinions/ideas
<laanwj> not sure, i revoked my access to bitcoin.org long ago, and no idea who's maintaining it nowadays
<glozow> last update was by cobra in january?
<sipa> yeah
pablomartin has joined #bitcoin-core-dev
pablomartin is now known as pablomartin4btc
<laanwj> could try contacting cobra about it, but they're always difficult
<glozow> maybe give it a shot?
<laanwj> sure
<glozow> laanwj: thank you!
<glozow> Anything else to discuss?
<stickies-v> i'll leave a review on https://github.com/bitcoin-dot-org/Bitcoin.org/pull/4349 for more visibility/activity
<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
<sipa> laanwj: thanks!
<glozow> stickies-v: that seems like a good idea!
<glozow> #endmeeting
<corebot> glozow: Meeting ended at 2025-03-27T16:32+0000
rkrux has quit [Quit: Client closed]
Emc99 has quit [Quit: Client closed]
Christoph_ has quit [Quit: Christoph_]
mentalbarcode has quit [Ping timeout: 244 seconds]
abubakarsadiq has quit [Quit: Connection closed for inactivity]
OGU has quit [Ping timeout: 265 seconds]
bugs_ has quit [Quit: Leaving]
jespada has joined #bitcoin-core-dev
<laanwj> PR adding 27.1, 27.2 and 28.0: https://github.com/bitcoin-dot-org/Bitcoin.org/pull/4386
<sipa> laanwj: great
<darosior> laanwj: thanks
pablomartin4btc has quit [Ping timeout: 248 seconds]
Talkless has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] instagibbs reopened pull request #26398: Replace MIN_STANDARD_TX_NONWITNESS_SIZE to preclude 64 non-witness bytes only (master...relax_too_small_tx_equality) https://github.com/bitcoin/bitcoin/pull/26398
jespada has quit [Quit: My Mac has gone to sleep. ZZZzzz…]
jespada has joined #bitcoin-core-dev
OGU has joined #bitcoin-core-dev
OGU has quit [Changing host]
OGU has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] darosior opened pull request #32155: miner: timelock the coinbase to the mined block's height (master...2502_height_in_cb_locktime) https://github.com/bitcoin/bitcoin/pull/32155
jespada has quit [Quit: My Mac has gone to sleep. ZZZzzz…]
Talkless has quit [Quit: Konversation terminated!]
<bitcoin-git> [bitcoin] darosior closed pull request #28216: fuzz: a new target for the coins database (master...fuzz_coins_db) https://github.com/bitcoin/bitcoin/pull/28216
<bitcoin-git> [bitcoin] ryanofsky pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/74c23f80abbe...930b237f163d
<bitcoin-git> bitcoin/master 35d17cd Hodlinator: qa wallet: Actually make use of expressions
<bitcoin-git> bitcoin/master 930b237 Ryan Ofsky: Merge bitcoin/bitcoin#31874: qa wallet: Activate dormant checks in wallet_...
<bitcoin-git> [bitcoin] ryanofsky merged pull request #31874: qa wallet: Activate dormant checks in wallet_multisig_descriptor_psbt.py (master...2025/02/wallet_dormant_checks) https://github.com/bitcoin/bitcoin/pull/31874
<bitcoin-git> [bitcoin] darosior closed pull request #26573: Wallet: don't underestimate the fees when spending a Taproot output (master...taproot_over_dont_under_estimate) https://github.com/bitcoin/bitcoin/pull/26573
Guest86 has joined #bitcoin-core-dev
Guest86 has quit [Client Quit]
OGU has quit [Ping timeout: 272 seconds]
andytoshi has quit [Ping timeout: 265 seconds]
andytoshi has joined #bitcoin-core-dev
Earnestly has quit [Ping timeout: 260 seconds]
Earnestly has joined #bitcoin-core-dev
sliv3r__ has quit [Ping timeout: 244 seconds]
Earnestly has quit [Ping timeout: 268 seconds]
conman has quit [Ping timeout: 248 seconds]
sliv3r__ has joined #bitcoin-core-dev
conman has joined #bitcoin-core-dev
conman has quit [Ping timeout: 252 seconds]
Earnestly has joined #bitcoin-core-dev