< bitcoin-git> [bitcoin] Empact closed pull request #16038: Drop BanLevel in favor of Optional<BanReason> (master...drop-ban-level) https://github.com/bitcoin/bitcoin/pull/16038
< bitcoin-git> [bitcoin] fanquake closed pull request #16018: banman: BanLevel enum (master...ban-level-enum) https://github.com/bitcoin/bitcoin/pull/16018
< fanquake> Pretty quiet meeting last night. Assume everyone is recovering from a few days in NY ?
< bitcoin-git> [bitcoin] fanquake opened pull request #16039: docs: add release note for 14954 (master...release-notes-14954) https://github.com/bitcoin/bitcoin/pull/16039
< gmaxwell> getrawmempool verbose is astonishingly slow, a PR just went in to make it faster but it is still astonishgly slow.
< gmaxwell> I am reasonably confident that it was not anywhere near this slow last time the mempools were big.
< gmaxwell> This change in aug 2017 made all the univalue additions quadratic: https://github.com/bitcoin-core/univalue/commit/ceb1194137421afba01e8c1042bf75f3a2cdddc3 (which probably makes a number of our other RPCs slow too), but the changes works around it.
< gmaxwell> Anyone else have any WAGs at what might have slowed it down during the last two years?
< b10c> IIRC it was already slow in late 2017
< bitcoin-git> [bitcoin] merland closed pull request #15830: docs: Improvements to two READMEs in /test (master...patch-1) https://github.com/bitcoin/bitcoin/pull/15830
< fanquake> wumpus 5 utACKs on #15999 if you'd like to take a look
< gribble> https://github.com/bitcoin/bitcoin/issues/15999 | init: Remove dead code in LoadChainTip by MarcoFalke · Pull Request #15999 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/8f2f17f79a10...f3d27d126bbf
< bitcoin-git> bitcoin/master 593a8e8 practicalswift: wallet: Use chain.lock() instead of temporary chain.assumeLocked()
< bitcoin-git> bitcoin/master 9402ef0 practicalswift: Remove temporary method assumeLocked(). Remove LockingStateImpl. Remove re...
< bitcoin-git> bitcoin/master f3d27d1 MarcoFalke: Merge #16033: Hold cs_main when reading chainActive via getTipLocator(). R...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16033: Hold cs_main when reading chainActive via getTipLocator(). Remove assumeLocked(). (master...assumeLocked) https://github.com/bitcoin/bitcoin/pull/16033
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/f3d27d126bbf...a822a0e4f631
< bitcoin-git> bitcoin/master fa86c8a MarcoFalke: init: Remove dead code in LoadChainTip
< bitcoin-git> bitcoin/master a822a0e MarcoFalke: Merge #15999: init: Remove dead code in LoadChainTip
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #15999: init: Remove dead code in LoadChainTip (master...1905-initNoDead) https://github.com/bitcoin/bitcoin/pull/15999
< stevenroose> How are the manpages generated again?
< stevenroose> I did it before I think, but can't seem to find out how to do it anymore.
< stevenroose> Ah I think I found it, in contrib/devtools/. Nvm :)
< bitcoin-git> [bitcoin] dongcarl closed pull request #16029: netbase: Remove "tor" as a network specification (master...2019-05-remove-net-tor) https://github.com/bitcoin/bitcoin/pull/16029
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/a822a0e4f631...277abed604a8
< bitcoin-git> bitcoin/master faede74 MarcoFalke: doc: Explain how to pass in non-fundamental types into functions
< bitcoin-git> bitcoin/master 277abed MarcoFalke: Merge #15922: doc: Explain how to pass in non-fundamental types into funct...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #15922: doc: Explain how to pass in non-fundamental types into functions (master...1904-docPassFun) https://github.com/bitcoin/bitcoin/pull/15922
< bitcoin-git> [bitcoin] dongcarl opened pull request #16041: depends: Build secondary deps statically. (master...2019-05-depends-static-if-secondary-dep) https://github.com/bitcoin/bitcoin/pull/16041
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #16042: test: Bump MAX_NODES to 12 (master...1905-testMaxNodes) https://github.com/bitcoin/bitcoin/pull/16042
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/277abed604a8...c89a63412e85
< bitcoin-git> bitcoin/master feed98e Hennadii Stepanov: Ensure repos are up-to-date
< bitcoin-git> bitcoin/master c89a634 MarcoFalke: Merge #15863: scripts and tools: Ensure repos are up-to-date in gitian-bui...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #15863: scripts and tools: Ensure repos are up-to-date in gitian-build.py (master...20190421-update-repos) https://github.com/bitcoin/bitcoin/pull/15863
< jb55> achow101: why would {sh_,}wpkh descriptor multimports takes twice as long as pkh multiimports? I'm testing your batch write branch now.
< jb55> going to do some profiling, these import times are brutal...
< achow101> jb55: more scripts to import probably
< jb55> achow101: looks IO bound... iotop shows tons of disk activity when importing. does importing really need to do that much disk IO?
< achow101> Yeah, it's writing everything to the wallet file on disk
< jb55> achow101: yeah but I was seeing like 3MBps for a good 8 minutes for an 11MB wallet.dat ...
< achow101> Sounds possible
< gmaxwell> that just sounds like its not batching.