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
dviola has quit [Ping timeout: 260 seconds]
sbddesign has joined #bitcoin-core-dev
jespada has quit [Ping timeout: 252 seconds]
<bitcoin-git> [bitcoincore.org] joetor5 opened pull request #1156: Add spanish translation for 28.2 release (master...es-trans) https://github.com/bitcoin-core/bitcoincore.org/pull/1156
jadi has quit [Quit: jadi]
sbddesign has quit [Ping timeout: 252 seconds]
jadi has joined #bitcoin-core-dev
jadi has quit [Client Quit]
jadi has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
jonatack has quit [Read error: Connection reset by peer]
jonatack has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 272 seconds]
adil has joined #bitcoin-core-dev
adil has quit [Client Quit]
sbddesign has joined #bitcoin-core-dev
PaperSword has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 260 seconds]
jonatack has quit [Ping timeout: 268 seconds]
tarotfied has quit [Quit: WeeChat 4.1.1]
tarotfied has joined #bitcoin-core-dev
Cory81 has quit [Quit: Client closed]
Cory81 has joined #bitcoin-core-dev
Cory81 has quit [Quit: Client closed]
Cory81 has joined #bitcoin-core-dev
Cory81 has quit [Quit: Client closed]
Cory81 has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
dviola has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 260 seconds]
jadi has quit [Ping timeout: 260 seconds]
jadi has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 276 seconds]
adil has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 276 seconds]
adil has quit [Ping timeout: 276 seconds]
adil has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
adil1 has joined #bitcoin-core-dev
adil1 has quit [Client Quit]
adil has quit [Ping timeout: 276 seconds]
adil has joined #bitcoin-core-dev
adil has quit [Client Quit]
sbddesign has quit [Ping timeout: 276 seconds]
sbddesign has joined #bitcoin-core-dev
Cory81 has quit [Quit: Client closed]
Cory81 has joined #bitcoin-core-dev
SpellChecker has quit [Remote host closed the connection]
SpellChecker has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
Guest78 has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 252 seconds]
Guest78 has quit [Ping timeout: 272 seconds]
sbddesign has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 265 seconds]
sbddesign has joined #bitcoin-core-dev
joetor5 has quit [Quit: joetor5]
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
joetor5 has joined #bitcoin-core-dev
joetor5 has quit [Client Quit]
sbddesign has quit [Ping timeout: 252 seconds]
sbddesign has joined #bitcoin-core-dev
abubakarsadiq has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 272 seconds]
joetor5 has joined #bitcoin-core-dev
joetor5 has quit [Quit: joetor5]
kevkevin has quit [Remote host closed the connection]
kevkevin has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 245 seconds]
sbddesign has quit [Ping timeout: 245 seconds]
joetor5 has joined #bitcoin-core-dev
joetor5 has quit [Quit: joetor5]
TheRec has quit [Ping timeout: 244 seconds]
kevkevin has joined #bitcoin-core-dev
mcey_ has joined #bitcoin-core-dev
TheRec_ has joined #bitcoin-core-dev
emcy__ has quit [Ping timeout: 244 seconds]
drew_ has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 244 seconds]
sbddesign has quit [Ping timeout: 260 seconds]
kevkevin has joined #bitcoin-core-dev
Guyver2 has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 248 seconds]
kevkevin has quit [Ping timeout: 272 seconds]
sliv3r__ has quit [Quit: ZNC 1.8.2+deb3.1+deb12u1 - https://znc.in]
sliv3r__ has joined #bitcoin-core-dev
michaelfolkson has quit [Ping timeout: 272 seconds]
michaelfolkson has joined #bitcoin-core-dev
Guyver2 has quit [Quit: Going offline, see ya! (www.adiirc.com)]
kevkevin has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 260 seconds]
Guyver2 has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 260 seconds]
kevkevin has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 248 seconds]
f321x has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 248 seconds]
sbddesign has quit [Ping timeout: 245 seconds]
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/192743235493...49d5f1f2c6d2
<bitcoin-git> bitcoin/master d6aaffc brunoerg: test: check P2SH sigop count for coinbase tx
<bitcoin-git> bitcoin/master 49d5f1f merge-script: Merge bitcoin/bitcoin#32850: test: check P2SH sigop count for coinbase tx
<bitcoin-git> [bitcoin] fanquake merged pull request #32850: test: check P2SH sigop count for coinbase tx (master...2025-07-test-p2shsigopcount-coinbase) https://github.com/bitcoin/bitcoin/pull/32850
sbddesign has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] Sjors opened pull request #32861: Have createwalletdescriptor auto-detect an unused(KEY) (master...2025/07/smart-createwalletdescriptor) https://github.com/bitcoin/bitcoin/pull/32861
sbddesign has quit [Ping timeout: 260 seconds]
kevkevin has joined #bitcoin-core-dev
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
f321x has quit [Quit: f321x]
f321x has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 12 commits to master: https://github.com/bitcoin/bitcoin/compare/49d5f1f2c6d2...625194944333
<bitcoin-git> bitcoin/master 6c364e0 Martin Zumsande: test: Enable various tests for usage with cli
<bitcoin-git> bitcoin/master 7d5352a Martin Zumsande: test: use -stdin for large rpc commands
<bitcoin-git> bitcoin/master 5b08885 Martin Zumsande: test: enable functional tests with large rpc args for cli
<bitcoin-git> [bitcoin] fanquake merged pull request #32290: test: allow all functional tests to be run or skipped with --usecli (master...202505_fix_usecli) https://github.com/bitcoin/bitcoin/pull/32290
l0rinc has joined #bitcoin-core-dev
l0rinc has quit [Client Quit]
sbddesign has joined #bitcoin-core-dev
SpellChecker_ has joined #bitcoin-core-dev
SpellChecker has quit [Ping timeout: 244 seconds]
sbddesign has quit [Ping timeout: 248 seconds]
f321x has quit [Remote host closed the connection]
f321x has joined #bitcoin-core-dev
szkl has joined #bitcoin-core-dev
SpellChecker_ has quit [Remote host closed the connection]
SpellChecker has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] fanquake pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/625194944333...c7fe8abb5f38
<bitcoin-git> bitcoin/master bb9157d Hennadii Stepanov: cmake, refactor: Switch to `Python3::Interpreter` imported target
<bitcoin-git> bitcoin/master 67dc752 Hennadii Stepanov: cmake, test: Disable tests instead of ignoring them
<bitcoin-git> bitcoin/master c7fe8ab merge-script: Merge bitcoin/bitcoin#31233: cmake: Improve Python robustness and test usa...
<bitcoin-git> [bitcoin] fanquake merged pull request #31233: cmake: Improve Python robustness and test usability (master...241106-python) https://github.com/bitcoin/bitcoin/pull/31233
PaperSword has quit [Quit: PaperSword]
f321x_ has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
f321x has quit [Ping timeout: 244 seconds]
sbddesign has quit [Ping timeout: 252 seconds]
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/c7fe8abb5f38...927e9b220f17
<bitcoin-git> bitcoin/master 4f10a57 Hennadii Stepanov: depends: Override host compilers for FreeBSD and OpenBSD
<bitcoin-git> bitcoin/master 927e9b2 merge-script: Merge bitcoin/bitcoin#32716: depends: Override host compilers for FreeBSD ...
<bitcoin-git> [bitcoin] fanquake merged pull request #32716: depends: Override host compilers for FreeBSD and OpenBSD (master...250610-depends-bsd) https://github.com/bitcoin/bitcoin/pull/32716
sbddesign has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 244 seconds]
sbddesign has quit [Ping timeout: 248 seconds]
sbddesign has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 276 seconds]
<sipa> phantomcircuit: yeah, i think that's what you want. But for it to be useful, you'd need the ability for net_processing to continue while ABC is running. However, you cannot process further messages from the peer that gave you the block, as that wouls violate the protocol (e.g. if they send a BLOCK followed by a PING, they only expect the PONG after the block is processed), so you need a way to mark
<sipa> a peer in net_processing as "paused" while ABC is...
<sipa> running for it, making net_processing skip the peer
f321x_ has quit [Quit: f321x_]
f321x has joined #bitcoin-core-dev
<_aj_> sipa: you probably want to defer processing (and requesting?) transactions from other peers as well? probably also pause any peers that announce the same block/header at you?
<_aj_> s/probably/possibly/g
PaperSword has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
<sipa> _aj_: yes, i think what we really want is validation to work as a queue of (tx, block, cmpctblk, ...) objects, each with a callback for reporting success/failure. net_processing just hands the received objects to the queue, pausing the peer that gave them. Upon completion, the callback unpauses them (and perhaps relays, punishes, ...); the nice thing is that the validation queue can then be made to
<sipa> reorder/prioritize its objects (e.g. prioritize blocks...
<sipa> over txn), or split up processing (e.g. do an early header check first, or do things quickly if they are cached already, or even start doing transactioj validation in multiple script threads, ...)
jonatack has quit [Ping timeout: 268 seconds]
<bitcoin-git> [bitcoin] fanquake pushed 9 commits to 29.x: https://github.com/bitcoin/bitcoin/compare/983b268768f0...d360a6ef565f
<bitcoin-git> bitcoin/29.x e37a70b will: build: add root dir to CMAKE_PREFIX_PATH
<bitcoin-git> bitcoin/29.x 5987c1b Sebastian Falbesoner: test: fix catchup loop in outbound eviction functional test
<bitcoin-git> bitcoin/29.x a990c10 Hennadii Stepanov: cmake: Use `HINTS` instead of `PATHS` in `find_*` commands
<bitcoin-git> [bitcoin] fanquake merged pull request #32810: [29.x] More backports (29.x...even_more_29_backports) https://github.com/bitcoin/bitcoin/pull/32810
jespada has joined #bitcoin-core-dev
l0rinc has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] pinheadmz opened pull request #32862: rpc: use CScheduler for HTTPRPCTimer and remove RPCTimer (master...http-scheduler-notimer) https://github.com/bitcoin/bitcoin/pull/32862
<bitcoin-git> [bitcoin] pinheadmz closed pull request #32796: rpc: use CScheduler for HTTPRPCTimer (master...http-scheduler) https://github.com/bitcoin/bitcoin/pull/32796
<bitcoin-git> [bitcoin] fanquake opened pull request #32863: [29.x] Backports (29.x...probably_final_29_backports) https://github.com/bitcoin/bitcoin/pull/32863
sbddesign has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 248 seconds]
sbddesign has quit [Ping timeout: 260 seconds]
<bitcoin-git> [bitcoin] kilavvy opened pull request #32864: fix typos and improve clarity in documentation (master...master) https://github.com/bitcoin/bitcoin/pull/32864
<bitcoin-git> [bitcoin] fanquake opened pull request #32865: cmake: Use `AUTHOR_WARNING` for warnings (master...use_author_warning) https://github.com/bitcoin/bitcoin/pull/32865
<bitcoin-git> [bitcoin] fanquake closed pull request #32864: fix typos and improve clarity in documentation (master...master) https://github.com/bitcoin/bitcoin/pull/32864
sbddesign has joined #bitcoin-core-dev
eugenesiegel has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 252 seconds]
Guest7 has joined #bitcoin-core-dev
Guest7 has quit [Client Quit]
Guyver2 has quit [Quit: Going offline, see ya! (www.adiirc.com)]
l0rinc has quit [Quit: l0rinc]
l0rinc has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] rkrux opened pull request #32866: doc: add note for watch-only wallet migration (master...watchonly-migration) https://github.com/bitcoin/bitcoin/pull/32866
sbddesign has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] rkrux opened pull request #32867: doc: mention key removal in rpc interface modification (master...rpc-deprecation) https://github.com/bitcoin/bitcoin/pull/32867
eugenesiegel has quit [Ping timeout: 272 seconds]
f321x has quit [Quit: f321x]
joetor5 has joined #bitcoin-core-dev
joetor5 has quit [Remote host closed the connection]
joetor5 has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] theStack opened pull request #32868: test: refactor: overhaul block hash determination for `CBlock{,Header}` objects (master...202506-test-remove_blockhash_caching) https://github.com/bitcoin/bitcoin/pull/32868
jespada has quit [Quit: My Mac has gone to sleep. ZZZzzz…]
jespada has joined #bitcoin-core-dev
Saturday7 has quit [Quit: ZNC 1.10.0 - https://znc.in]
Saturday7 has joined #bitcoin-core-dev
Guyver2 has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 252 seconds]
joetor5 has quit [Remote host closed the connection]
bugs_ has joined #bitcoin-core-dev
joetor5 has joined #bitcoin-core-dev
sliv3r__ has quit [Ping timeout: 276 seconds]
<bitcoin-git> [bitcoin] instagibbs opened pull request #32869: WIP p2p: Relax BlockRequestAllowed to respond to advertised blocks (master...2025-07-invalid-cb-stall) https://github.com/bitcoin/bitcoin/pull/32869
sliv3r__ has joined #bitcoin-core-dev
l0rinc has quit [Quit: l0rinc]
vasild has quit [Ping timeout: 244 seconds]
sbddesign has joined #bitcoin-core-dev
vasild has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 260 seconds]
<bitcoin-git> [qa-assets] dergoegge pushed 3 commits to main: https://github.com/bitcoin-core/qa-assets/compare/c93ae8c8e7e3...c6cb320c0e45
<bitcoin-git> qa-assets/main f9816b7 MarcoFalke: Add fuzz inputs
<bitcoin-git> qa-assets/main c6cb320 Niklas Gögge: Merge pull request #227 from maflcko/main
<bitcoin-git> qa-assets/main 7720cbb MarcoFalke: remove crashing rpc inputs for now
<bitcoin-git> [qa-assets] dergoegge merged pull request #227: Add fuzz inputs (main...main) https://github.com/bitcoin-core/qa-assets/pull/227
joetor5 has quit [Quit: joetor5]
jespada has quit [Ping timeout: 276 seconds]
jespada has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
bitdex has joined #bitcoin-core-dev
kevkevin_ has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 276 seconds]
<bitcoin-git> [bitcoin] maflcko closed pull request #32615: fee estimate test: fix #31944 by handling a legitimate scenario that … (master...fix-fee-estimation-test) https://github.com/bitcoin/bitcoin/pull/32615
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/927e9b220f17...51ccc71b1bf9
<bitcoin-git> bitcoin/master 0a1af44 Hennadii Stepanov: doc: Add workaround for vcpkg issue with paths with embedded spaces
<bitcoin-git> bitcoin/master 51ccc71 merge-script: Merge bitcoin/bitcoin#32858: doc: Add workaround for vcpkg issue with path...
<bitcoin-git> [bitcoin] fanquake merged pull request #32858: doc: Add workaround for vcpkg issue with paths with embedded spaces (master...250702-vcpkg-spaces) https://github.com/bitcoin/bitcoin/pull/32858
jonatack has joined #bitcoin-core-dev
enochazariah has joined #bitcoin-core-dev
Emc99 has joined #bitcoin-core-dev
<achow101> #startmeeting
<corebot> achow101: Meeting started at 2025-07-03T16:00+0000
<corebot> achow101: Current chairs: achow101
<corebot> achow101: Useful commands: #action #info #idea #link #topic #motion #vote #close #endmeeting
<corebot> achow101: Participants should now identify themselves with '#here' or with an alias like '#here FirstLast'
<achow101> #bitcoin-core-dev Meeting: abubakarsadiq achow101 _aj_ ajonas b10c brunoerg cfields darosior dergoegge fanquake fjahr furszy gleb glozow hebasto hodlinator 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
<hebasto> hi
<cfields> hi
<johnny9dev> Hi
<laanwj> hi
<abubakarsadiq> hi
<lightlike> hi
<sr_gi[m]1> hi
<pinheadmz> hi
bitdex has quit [Quit: = ""]
<stickies-v> hi
<achow101> There are no pre-proposed meeting topics this week. Any last minute ones to add?
<janb84> hi
<danielabrozzoni> hi
<brunoerg> hi
<achow101> #topic Erlay WG Update (sr_gi, gleb)
<vasild> hi
l0rinc has joined #bitcoin-core-dev
<sipa> hi
<dergoegge> hi
<instagibbs> hi
<Murch[m]> Hi
<sr_gi[m]1> It seems the propagation time issue I found last week may be what was making erlay times too good to be true. I've been re-running the simulations for small (50 nodes) and medium-size (200 nodes) networks. I'm currently re-working part of the warnet simulations so they can handle bigger networks (500 nodes is the most I've done, but the sims are a bit unreliable)
<pablomartin4btc> hi
<sr_gi[m]1> My goal is having the sims for all of them in the upcoming days so I can report back hopefully next week, and start moving forward
<sr_gi[m]1> That's it on my end
<achow101> #topic Fuzzing WG Update (dergoegge)
<dergoegge> Found 2 bugs on PRs with fuzzamoto in the last two weeks. One really shallow one in the erlay sketch deserialization code and one in the cluster mempool PR. The mempool bug specifically is a nice example for the complex inputs that fuzzamoto can generate.
<dergoegge> Also implemented a “netsplit” oracle, which already found a minor issue, fixed by instagibbs here: https://github.com/bitcoin/bitcoin/pull/32869 (would be nice to get some review on that)
<dergoegge> More updates (and links to the above): https://gist.github.com/dergoegge/0f732c440c19d02cf5eb6059ee90147c
<sipa> nice!
<dergoegge> (that's it on my end)
<achow101> #topic Cluster Mempool WG Update (sdaftuar, sipa)
Naiyoma has joined #bitcoin-core-dev
<sipa> hi
<sipa> i think both #31553 and #30605 are really close
<corebot> https://github.com/bitcoin/bitcoin/issues/31553 | cluster mempool: add TxGraph reorg functionality by sipa · Pull Request #31553 · bitcoin/bitcoin · GitHub
<corebot> https://github.com/bitcoin/bitcoin/issues/30605 | Cluster linearization: separate tests from tests-of-tests by sipa · Pull Request #30605 · bitcoin/bitcoin · GitHub
<sipa> the past week some extra benchmarks are tests were added, but i don't expect many further changes
<glozow> hi
<instagibbs> try not to rebase on master if possible :P
<sipa> after that, the txgraph API will be largely complete, and i think we can start focusing on getting parts of the full cluster mempool implementation (#28676) in
<corebot> https://github.com/bitcoin/bitcoin/issues/28676 | [WIP] Cluster mempool implementation by sdaftuar · Pull Request #28676 · bitcoin/bitcoin · GitHub
<glozow> sipa: would the plan be to do that PR all together?
<glozow> (all of 28676)
<sipa> unsure, i assume it contains some amount of preparatory work that can be split off
<sipa> but the bulk, adding the cluster tracking, and switching out the mempool tracking, will need to be one PR I think (or at least a sequence of PRs that are all expected to go into the same release)
<glozow> makes sense
<sipa> that's it for me, unless more questions
<achow101> #topic MuSig2 WG Update (achow101, rkrux)
<abubakarsadiq> I like the test one transaction forming huge cluster by spending them. and then in trim we don't just evict it alone.
<abubakarsadiq> we evict others, I think that should not be the case?
<corebot> https://github.com/bitcoin/bitcoin/issues/31553 | cluster mempool: add TxGraph reorg functionality by sipa · Pull Request #31553 · bitcoin/bitcoin · GitHub
enochazariah has quit [Quit: Client closed]
<sipa> abubakarsadiq: an optimal eviction algorithm could avoid that, but the trim algorithm is really just a best-effort of what we can accomplish in O(n log n)
enochazariah has joined #bitcoin-core-dev
<sipa> with proper linearization we could do a lot better - but the whole point of trim is that it can work with much larger numbers of transactions than what fit in a cluster
<achow101> #topic cluster mempool still
<abubakarsadiq> yep, could just throw txs in compact block reconstruction txs with a bound if thats the case.
<sipa> right
enochazariah has quit [Client Quit]
<abubakarsadiq> Also I think if we evict children first, we could avoid that but the current best effort algorithm is better in terms of incentive compatibility I think
<sipa> i'm not too worried about this, in practice i think the amount of oversized would-be clusters formed by reorged will be pretty small in non-attack scenarios
<instagibbs> most clusters are singletons after all
<abubakarsadiq> 👍
<sipa> and it costs creating a block to do something deliberately crazy
Guyver2 has left #bitcoin-core-dev [Closing Window]
enochazariah has joined #bitcoin-core-dev
<sipa> abubakarsadiq: we're not restricted to the current Trim algorithm too; we could experiment with better algorithms later, or even have multiple algorithms depending on the size (e.g. have a medium-size trim algorithm that works acceptably fast for say ~200 transactions, but not 5000, for which we can still fall back to the current Trim one)
<sipa> but those don't affect the API, so wouldn't block further cluster mempool progress
<abubakarsadiq> yes please
memset has quit [Remote host closed the connection]
<sipa> now back to your regularly scheduled MuSig2
memset has joined #bitcoin-core-dev
<achow101> #topic MuSig2 WG Update (achow101, rkrux)
<achow101> #31244 is probably rfm, otherwise, no updates
<corebot> https://github.com/bitcoin/bitcoin/issues/31244 | descriptors: MuSig2 by achow101 · Pull Request #31244 · bitcoin/bitcoin · GitHub
<achow101> #topic orphan resolution WG Update (glozow)
<glozow> I've been working on benchmarks for #31829, and have been chatting with sipa about potentially baking number of inputs into the dos score
<corebot> https://github.com/bitcoin/bitcoin/issues/31829 | p2p: improve TxOrphanage denial of service bounds by glozow · Pull Request #31829 · bitcoin/bitcoin · GitHub
<instagibbs> for the eraseforblock case?
<glozow> Worst case EraseForBlock times were pretty bad (if you have tons of transactions each spending hundreds of inputs all conflicting with the block)
<glozow> yeah, I'm getting 35ms on my machine
<sipa> EraseForBlock, but also EraseForPeer could trigger most (but not all) of the same work, which is potentially triggerable by non-miners
<instagibbs> due to outpoint map?
<glozow> Yes. EraseForBlock is slightly worse because it has the loop that builds the set of wtxids to erase (each of the 1750 wtxids is added ~180 times). And then it has to erase all of those entries from the outpoints map
enochazariah has quit [Quit: Client closed]
<instagibbs> 👍
<glozow> The current ideated formula is that "announcement count" becomes 1 + floor(vin.size() / 10) for each tx. So the first 9 inputs are "free" and this limit is hopefully not relevant to normal transactions
<sipa> the vin.size()/10 would be deduplicated across peers, while the 1 part isn't
<sipa> as each tx's inputs only appear once in the outpoint index
Talkless has joined #bitcoin-core-dev
<bitcoin-git> [packaging] fanquake pushed 2 commits to main: https://github.com/bitcoin-core/packaging/compare/aad8d4df0f4c...1c03d604e451
<bitcoin-git> packaging/main 577ccb0 MarcoFalke: ci: Avoid separate pull to work around snap issue
<bitcoin-git> packaging/main 1c03d60 merge-script: Merge bitcoin-core/packaging#284: ci: Avoid separate pull to work around s...
<bitcoin-git> [packaging] fanquake merged pull request #284: ci: Avoid separate pull to work around snap issue (main...2506-ci) https://github.com/bitcoin-core/packaging/pull/284
<glozow> right that part is similar to the memusage metric
<glozow> Since last week I've also added sipa's simulation fuzzer + a few small tweaks and tests
<glozow> that's all the updates
<achow101> #topic QML GUI WG Update (jarolrod, johnny9dev)
<johnny9dev> Not much this week on QML work. bitcoin-core/gui-qml#450 got merged and hebasto and pinheadmz are making good progress on bitcoin-core/gui-qml#472.
<johnny9dev> Thats all for now
<corebot> https://github.com/bitcoin-core/gui-qml/issues/450 | Add Multiple Recipients option to the Send form by johnny9 · Pull Request #450 · bitcoin-core/gui-qml · GitHub
<corebot> https://github.com/bitcoin-core/gui-qml/issues/472 | Rebase gui-qml on bitcoin/bitcoin, include qt6 and cmake by pinheadmz · Pull Request #472 · bitcoin-core/gui-qml · GitHub
<achow101> Any other topics to discuss?
<bitcoin-git> [packaging] fanquake pushed 2 commits to main: https://github.com/bitcoin-core/packaging/compare/1c03d604e451...9ac225b7e060
<bitcoin-git> packaging/main 2357600 Hennadii Stepanov: snap: Drop `desktop-launch`
<bitcoin-git> packaging/main 9ac225b merge-script: Merge bitcoin-core/packaging#276: snap: Drop `desktop-launch`
<bitcoin-git> [packaging] fanquake merged pull request #276: snap: Drop `desktop-launch` (main...250417-desktop-launch) https://github.com/bitcoin-core/packaging/pull/276
<l0rinc> I have a mempool related question regarding https://github.com/bitcoin/bitcoin/pull/32827#issue-3186436123
<l0rinc> do we need to keep the mempool stats up-to-date when the mempool is still empty? My understanding is that when it becomes empty we definitely need to keep the fee estimation state updated, but not sure if that's necessary during IBD. And if I try skipping `MempoolTransactionsRemovedForBlock` notification for the resulting empty vector of removals, I'm getting a test failure and need some help in deciding if the test shows a representative behavior
<l0rinc> or not - would be nice if we could avoid adding another callback into the validation queue...
<achow101> #topic mempool in ibd
Naiyoma has quit [Ping timeout: 252 seconds]
<glozow> Is the question whether the fee estimator needs to know about every `MempoolTransactionsRemovedForBlock`?
<l0rinc> yes, in case of no removals
<l0rinc> as is the case as far as I can tell during IBD before any mempool activity
<abubakarsadiq> l0rinc: is it the fee estimator test that is failing?
<l0rinc> it's feature_fee_estimation
<glozow> I think this is the only way the fee estimator updates `txHeight`
Guest5846 has joined #bitcoin-core-dev
<glozow> Is this slowing IBD down though? Doesn't this happen on a background thread?
<bitcoin-git> [bitcoin] fanquake pushed 6 commits to 28.x: https://github.com/bitcoin/bitcoin/compare/e44d72b6480a...3c56d36bec53
<bitcoin-git> bitcoin/28.x d235d5b zaidmstrr: test: Fix list index out of range error in feature_bip68_sequence.py
<bitcoin-git> bitcoin/28.x 907772d fanquake: doc: fix transifex 404s
<bitcoin-git> bitcoin/28.x d19a8e0 Sjors Provoost: doc: taproot became always active in v24.0
<bitcoin-git> [bitcoin] fanquake merged pull request #32811: [28.x] Backports (28.x...28_more_backporting) https://github.com/bitcoin/bitcoin/pull/32811
<abubakarsadiq> shouldn't it be no-op when their is no data on mempool, is their a measurable improvement from skipping that?
<jonatack> hi
sbddesign has quit [Ping timeout: 276 seconds]
<l0rinc> glozow: it's blocking the queue, as furzsy sais:
<l0rinc> > First quick glance it seems that if the mempool is empty; we could avoid calling removeForBlock entirely. Which will save us from adding another callback into the validation queue too, which will leave space for processing another block during ibd.
<l0rinc> > Yet, that was a fast check. Should check who is using the MempoolTransactionsRemovedForBlock event.
<l0rinc> We don't have to resolve it here, if anyone has insights, please add it to the PR - otherwise I will keep the MempoolTransactionsRemovedForBlock call for empty txs_removed_for_block as well
<abubakarsadiq> glozow: yes, we update `nHeight` after we are done with ibd and update the fee estimator stats for the first time it recorded data is marked as confirmed because of a new block.
<glozow> so the fee estimator doesn't do anything during IBD?
Guest5846 has quit [Ping timeout: 276 seconds]
<abubakarsadiq> yes it does not, it has to see transaction enter mempool first.
<glozow> I meant to ask if the fee estimator gated on IBD?
<glozow> if its functions are gated*
<glozow> nvm, can look at this later
<abubakarsadiq> the updates are redundant, we don't do anything and can be avoided I think.
<abubakarsadiq> I started looking at it in https://github.com/bitcoin/bitcoin/pull/32748
<l0rinc> thanks, that's it from me
<achow101> Any other topics?
<achow101> #endmeeting
<corebot> achow101: Meeting ended at 2025-07-03T16:46+0000
Emc99 has quit [Quit: Client closed]
Guest9863 has joined #bitcoin-core-dev
PaperSword has quit [Quit: PaperSword]
PaperSword has joined #bitcoin-core-dev
l0rinc has quit [Quit: l0rinc]
jonatack has quit [Ping timeout: 260 seconds]
Cory81 has quit [Quit: Client closed]
Cory81 has joined #bitcoin-core-dev
PaperSword has quit [Quit: PaperSword]
memset has quit [Ping timeout: 244 seconds]
memset has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] davidgumberg closed pull request #31665: build: Make config warnings fatal if -DWCONFIGURE_ERROR=ON (master...1-15-24-configure_warnings) https://github.com/bitcoin/bitcoin/pull/31665
sbddesign has quit [Ping timeout: 248 seconds]
Guest71 has joined #bitcoin-core-dev
Guest71 has quit [Client Quit]
eugenesiegel has joined #bitcoin-core-dev
<bitcoin-git> [qa-assets] murchandamus closed pull request #229: Add Murch’s inputs July 2025 (main...2025-07-murch-inputs) https://github.com/bitcoin-core/qa-assets/pull/229
sbddesign has joined #bitcoin-core-dev
sbddesign has quit [Ping timeout: 252 seconds]
PaperSword has joined #bitcoin-core-dev
sbddesign has joined #bitcoin-core-dev
mudsip has joined #bitcoin-core-dev
mudsip has quit [Client Quit]
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
<furszy> l0rinc: I just had a quick talk with abubakarsadiq and It seems we still need to send the signal just to update the best seen height inside the fee estimation class, but we can skip some of the calculations on the event processing side (all the stats objects are initialized but contain only the default values, so we could avoid going through them while computing some mod operations). Still, not sure how much performance gain we will g
<furszy> et from this. Matter of checking it.
<abubakarsadiq> yeah working on it 👍🏾
Guest9 has joined #bitcoin-core-dev
Cory81 has quit [Quit: Client closed]
Cory81 has joined #bitcoin-core-dev
Talkless has quit [Quit: Konversation terminated!]
eugenesiegel has quit [Ping timeout: 272 seconds]
kevkevin_ has quit [Remote host closed the connection]
kevkevin has joined #bitcoin-core-dev
Cory81 has quit [Quit: Client closed]
Cory81 has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 276 seconds]
kevkevin has joined #bitcoin-core-dev
Guest9 has quit [Quit: Client closed]
szkl has quit [Quit: Connection closed for inactivity]
aleggg has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] 1BitcoinBoWP1FZ4xwTNkq6XksKidmgYYw opened pull request #32871: New SVG, Icons, PNGs and X PixMaps (master...master) https://github.com/bitcoin/bitcoin/pull/32871
kevkevin has quit [Remote host closed the connection]
Guest9863 has quit [Quit: WeeChat 4.1.1]
wafflefrie has quit [Quit: Connection closed for inactivity]
joetor5 has joined #bitcoin-core-dev
eugenesiegel has joined #bitcoin-core-dev
jespada has quit [Quit: My Mac has gone to sleep. ZZZzzz…]
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
joetor5 has quit [Ping timeout: 245 seconds]
Cory81 has quit [Quit: Client closed]
Cory81 has joined #bitcoin-core-dev
joetor5 has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 244 seconds]
eugenesiegel has quit [Quit: Client closed]
bugs_ has quit [Quit: Leaving]
wafflefrie has joined #bitcoin-core-dev
joetor5 has quit [Ping timeout: 248 seconds]
BlueMatt has quit [Quit: Quit]
BlueMatt has joined #bitcoin-core-dev
joetor5 has joined #bitcoin-core-dev
stratospher[m] has quit [Ping timeout: 260 seconds]
b10c[m] has quit [Ping timeout: 245 seconds]
BlueMattMtrxBot has quit [Ping timeout: 272 seconds]
Murch[m] has quit [Ping timeout: 260 seconds]
bitcoin-git has quit [Ping timeout: 260 seconds]
BlueMatt has quit [Ping timeout: 248 seconds]
sr_gi[m]1 has quit [Ping timeout: 248 seconds]
BlueMatt has joined #bitcoin-core-dev
bitcoin-git has joined #bitcoin-core-dev
sr_gi[m]1 has joined #bitcoin-core-dev
Murch[m] has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] achow101 closed pull request #32788: descriptors: Allow `H` as a hardened indicator (master...desc-allow-H) https://github.com/bitcoin/bitcoin/pull/32788
<bitcoin-git> [bitcoin] achow101 pushed 5 commits to master: https://github.com/bitcoin/bitcoin/compare/51ccc71b1bf9...ea4285775e61
<bitcoin-git> bitcoin/master a69c409 Hodlinator: rpc: take ownership of the file by WriteUTXOSnapshot()
<bitcoin-git> bitcoin/master 4bb5dd7 Vasil Dimov: util: check that a file has been closed before ~AutoFile() is called
<bitcoin-git> bitcoin/master 8bb34f0 Vasil Dimov: Explicitly close all AutoFiles that have been written
<bitcoin-git> [bitcoin] achow101 merged pull request #29307: util: explicitly close all AutoFiles that have been written (master...AutoFile_error_check) https://github.com/bitcoin/bitcoin/pull/29307
BlueMattMtrxBot has joined #bitcoin-core-dev
stratospher[m] has joined #bitcoin-core-dev
memset has quit [Ping timeout: 244 seconds]
memset has joined #bitcoin-core-dev
jadi has quit [Ping timeout: 276 seconds]
joetor5 has quit [Ping timeout: 244 seconds]
jadi has joined #bitcoin-core-dev
Guest0 has joined #bitcoin-core-dev
Guest0 has quit [Client Quit]
joetor5 has joined #bitcoin-core-dev
joetor5 has quit [Ping timeout: 244 seconds]
memset has quit [Remote host closed the connection]
memset has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] achow101 pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/ea4285775e61...e3f416dbf763
<bitcoin-git> bitcoin/master 5c1236f ismaelsadeeq: test: fix incorrect subtest in `feature_fee_estimation.py`
<bitcoin-git> bitcoin/master 9b75cfd ismaelsadeeq: test: retain the intended behavior of `feature_fee_estimation.py` nodes
<bitcoin-git> bitcoin/master e3f416d Ava Chow: Merge bitcoin/bitcoin#32463: test: fix an incorrect `feature_fee_estimatio...
<bitcoin-git> [bitcoin] achow101 merged pull request #32463: test: fix an incorrect `feature_fee_estimation.py` subtest (master...05-2025-fix-incorrect-fee-estimator-test) https://github.com/bitcoin/bitcoin/pull/32463
joetor5 has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev