< bitcoin-git> [bitcoin] fanquake pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/ec0453cd5773...e21b824386bf
< bitcoin-git> bitcoin/master c201d73 gzhao408: style and nits for fee-checking in BroadcastTransaction
< bitcoin-git> bitcoin/master 88197b0 gzhao408: [doc] release notes for max fee checking
< bitcoin-git> bitcoin/master e21b824 fanquake: Merge #20109: Release notes and followups from 19339
< bitcoin-git> [bitcoin] fanquake merged pull request #20109: Release notes and followups from 19339 (master...docs-absurdfee) https://github.com/bitcoin/bitcoin/pull/20109
< wumpus> luke-jr: isn't a batched request considered to be a single request from the viewpoint of the queue?
< wumpus> luke-jr: the commands in a batch are always executed in order anyway, there would be no point trying to queue them to different threads
< wumpus> (if anything it would break the gaurantee they'll finish in order)
< wumpus> the work queue is to distribute incoming work over worker threads nothing more
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #20146: net: Send post-verack handshake messages at most once (master...2010-netPostVerackHandshake) https://github.com/bitcoin/bitcoin/pull/20146
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/e21b824386bf...3750f664b349
< bitcoin-git> bitcoin/master 8e4d622 Vasil Dimov: tests: don't export in6addr_loopback
< bitcoin-git> bitcoin/master 3750f66 MarcoFalke: Merge #20129: tests: don't export in6addr_loopback
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #20129: tests: don't export in6addr_loopback (master...fix_export_of_in6addr_loopback) https://github.com/bitcoin/bitcoin/pull/20129
< jnewbery> hmmm, #19988 is no longer showing up in my notifications stream in github. I can still open the PR and see it in the list of open bitcoin/bitcoin PRs
< gribble> https://github.com/bitcoin/bitcoin/issues/19988 | Overhaul transaction request logic by sipa · Pull Request #19988 · bitcoin/bitcoin · GitHub
< jnewbery> I don't think I silenced notifications for that PR
< aj> jnewbery: still shows up for me
< jnewbery> it showed up in 'Done' category. I generally don't mark notification in github as Done, but maybe I accidentally clicked . Probably just user error
< fanquake> You'll be happy to know that GitHub is flat out rolling out new features while at the same time not addressing our concerns
< jnewbery> that's fantastic news. Can't wait to update my github story and add some filters to my avatar
< * aj> clicks like and subscribe just in case
< * fanquake> will continue to bring up ALL the issues
< aj> jnewbery: "e" is apparently the keyboard shortcut to mark as done
< jnewbery> aj: ah! Maybe that's it
< hebasto> #18750 has 3 ACKs now :)
< gribble> https://github.com/bitcoin/bitcoin/issues/18750 | build: optionally skip external warnings by vasild · Pull Request #18750 · bitcoin/bitcoin · GitHub
< dburkett> It's not immediately clear to me when you would choose one over the other.
< dburkett> Is there a simple explanation for the differences, or a clear use case for each that would illuminate the differences?
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/3750f664b349...9efa55c715c0
< bitcoin-git> bitcoin/master 135afa7 Ivan Metlushko: wallet: remove db mode string
< bitcoin-git> bitcoin/master 9efa55c Wladimir J. van der Laan: Merge #20130: Wallet: remove db mode string
< bitcoin-git> [bitcoin] laanwj merged pull request #20130: Wallet: remove db mode string (master...wallet_remove_mode_3) https://github.com/bitcoin/bitcoin/pull/20130
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/9efa55c715c0...99a1d572eabc
< bitcoin-git> bitcoin/master ba8950e Vasil Dimov: build: optionally skip external warnings
< bitcoin-git> bitcoin/master 99a1d57 Wladimir J. van der Laan: Merge #18750: build: optionally skip external warnings
< hebasto> wumpus: thanks
< bitcoin-git> [bitcoin] laanwj merged pull request #18750: build: optionally skip external warnings (master...suppress_external_warnings) https://github.com/bitcoin/bitcoin/pull/18750
< luke-jr> wumpus: is there a guarantee they will finish in order? anyway, someone was telling me batches are limited usefulness because the limit affects them..
< luke-jr> if it's not an intended property, maybe I will look into fixing it
< wumpus> luke-jr: I'm fairly sure there is a guarantee that they run (and finish) in order, that's the point of a batch
< wumpus> the work queue assigns HTTP *requests*, a batch is one HTTP request right?
< wumpus> I don't understand what could be the problem
< wumpus> luke-jr: the HTTP worker queue code is in httpserver.cpp, the batch handling code is in rpcserver.cpp JSONRPCExecBatch, it literally iterates over the array of commands, executing them one by one, pushing their result into the result array
< wumpus> there's no way they could interfere
< bitcoin-git> [bitcoin] laanwj pushed 10 commits to master: https://github.com/bitcoin/bitcoin/compare/99a1d572eabc...c2c4dbaebd95
< bitcoin-git> bitcoin/master da3b8fd Pieter Wuille: Add txrequest module
< bitcoin-git> bitcoin/master 3c7fe0e Pieter Wuille: Add txrequest unit tests
< bitcoin-git> bitcoin/master 5b03121 Pieter Wuille: Add txrequest fuzz tests
< bitcoin-git> [bitcoin] laanwj merged pull request #19988: Overhaul transaction request logic (master...202009_txrequest_rand_wtxid) https://github.com/bitcoin/bitcoin/pull/19988
< sdaftuar> \o/
< hebasto> \o/
< sipa> \o/
< luke-jr> btw would anyone object to libsecp256k1 commits coming here?
< jnewbery> \o/\o/\o/
< jonatack> congrats |o|
< ariard> yes! congrats
< jamesob> look at all that extra fuzz. nice!
< sipa> it's rather fuzzy
< wumpus> luke-jr: don't think secp256k1 notifications should be here, no, but isn't there a specific channel for commit notifications?
< michaelfolkson> A new channel for secp256k1 notifications or have notifications set up on the existing #secp256k1 channel?
< michaelfolkson> I guess a question for the #secp256k1 channel
< luke-jr> well, I guess there's #bitcoin-commits
< luke-jr> maybe I just need to make it more visible to myself
< wumpus> bitcoin-commits did have secp256k1 notifications as far as I know? but don't know if it's still the case since github removed their own IRC notification
< luke-jr> yes, it does
< wumpus> ok!
< wumpus> ah yes it's even the same bot
< michaelfolkson> Cool, joined it. Thanks
< bitcoin-git> [bitcoin] sipa opened pull request #20147: Update libsecp256k1 (endomorphism, test improvements) (master...202010_secp256k1) https://github.com/bitcoin/bitcoin/pull/20147
< instagibbs> argh no email notifications of PRs for me now
< instagibbs> ie \o/
< andytoshi> has anyone here implemented BIP322? i am working on an implementation and have some questions about the bip
< andytoshi> generic signed message format, by kalle, https://github.com/bitcoin/bips/blob/master/bip-0322.mediawiki
< sipa> it's undergoing heavy changes right now: https://github.com/bitcoin/bips/pull/1003
< andytoshi> thanks!
< yanmaani> if I have a COutPoint, is there a better way to find its script than
< yanmaani> `pool.mapTx.find(p.hash)->GetTx().vout[p.n].scriptPubKey`