< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b4b057a3e071...5c63d665e51e
< bitcoin-git> bitcoin/master 010d68e Pieter Wuille: Broadcast address every day, not 9 hours
< bitcoin-git> bitcoin/master 5c63d66 Wladimir J. van der Laan: Merge #10460: Broadcast address every day, not 9 hours...
< bitcoin-git> [bitcoin] laanwj closed pull request #10460: Broadcast address every day, not 9 hours (master...hour_is_not_24_minutes) https://github.com/bitcoin/bitcoin/pull/10460
< bitcoin-git> [bitcoin] cg31 opened pull request #10477: Use C++ initializer to initialze map and implement map comparator as const (master...master) https://github.com/bitcoin/bitcoin/pull/10477
< gmaxwell> more changes to checkpoint code, great. :P
< bitcoin-git> [bitcoin] laanwj opened pull request #10478: rpc: Add listen address to incoming connections in `getpeerinfo` (master...2017_05_peer_listenaddr) https://github.com/bitcoin/bitcoin/pull/10478
< bitcoin-git> [bitcoin] jnewbery opened pull request #10479: [trivial] Fix comment for ForceSetArg() (master...trivial_forcesetarg_comment) https://github.com/bitcoin/bitcoin/pull/10479
< morcos> wumpus: I think #9208 can be merged
< gribble> https://github.com/bitcoin/bitcoin/issues/9208 | Improve DisconnectTip performance by sdaftuar · Pull Request #9208 · bitcoin/bitcoin · GitHub
< morcos> instagibbs: can you tell me what command you ran and what result you got?
< instagibbs> https://0bin.net/paste/BEjwwDlgkUuAUIGS#o7sEulMGdPZVuVaE+gNjWzdtuIPph4TvlXInqBhe0v1
< morcos> you should be able to get an estimate for a target of X after recording 2*X blocks worth of fee data. The recording basically starts after the first block received after your node is caught up (its own defintiion of caught up)
< instagibbs> huh ok something is definitely wrong then
< morcos> no
< morcos> you're using estimaterawfee
< morcos> no reason to use that
< instagibbs> wow mega brainfart
< instagibbs> omg, sorry
< morcos> use "estimatesmartfee target" or "estimatesmartfee target false"
< instagibbs> yes I'm getting estimates just fine
< instagibbs> using raw
< morcos> estimaterawfee defaults to the medium time horizon which only tracks up to 48 confirms
< instagibbs> err smart
< morcos> so you could do what you want with "estimaterawfee 49 0.85 2" which would select the long time horizon and use an 85% threshold
< morcos> but smart will combine several calculations more intelligently
< instagibbs> right. Totally forgot about the 3rd param
< morcos> don't let sipa read that, he wanted me to get rid of that param :)
< instagibbs> lol
< sipa> morcos: yes, i still do ;)
< bitcoin-git> [bitcoin] luke-jr closed pull request #10476: Update luke-jr's PGP key (master...2017_pgp_update) https://github.com/bitcoin/bitcoin/pull/10476
< wumpus> morcos: thanks, agree
< bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/5c63d665e51e...acd9957b72a2
< bitcoin-git> bitcoin/master 9decd64 Suhas Daftuar: [qa] Relax assumptions on mempool behavior during reorg...
< bitcoin-git> bitcoin/master 71f1903 Suhas Daftuar: Store disconnected block transactions outside mempool during reorg...
< bitcoin-git> bitcoin/master c1235e3 Russell Yanofsky: Add RecursiveDynamicUsage overload for std::shared_ptr...
< bitcoin-git> [bitcoin] laanwj closed pull request #9208: Improve DisconnectTip performance (master...faster-disconnect-rebased) https://github.com/bitcoin/bitcoin/pull/9208
< sdaftuar> wumpus: thanks!
< morcos> just in time for the Fork Wars!
< timothy> 2MB forks or not 2MB forks, that is the question
< sipa> there is no spoon
< timothy> like in matrix?
< jtimon> BlueMatt: can you also take a look at https://github.com/bitcoin/bitcoin/pull/10427 ? it's almost equal to what you are doing in https://github.com/bitcoin/bitcoin/pull/10192 with some of the nits I had in, but without the moveonly to versionbits.o I was suggesting
< bitcoin-git> [bitcoin] jtimon closed pull request #7829: Globals: TODO: Experiment: Kill "Params()" (master...0.12.99-todo-globals-chainparams) https://github.com/bitcoin/bitcoin/pull/7829
< bitcoin-git> [bitcoin] gmaxwell closed pull request #9886: In feerate ties, prefer larger packages first. (master...prefer_bigger_packages) https://github.com/bitcoin/bitcoin/pull/9886
< jtimon> sipa: do you want me to s/blockHash/hashBlock/ in https://github.com/bitcoin/bitcoin/pull/10339 ? I'm fine, but I wasn't trying to use hungarian notation there, just blockHash as opposed to txHash
< jtimon> or block_hash or whatever
< jtimon> in some places still says just "hash", but I didn't find that clear enough
< sipa> jtimon: assuming people like the new style guide... i'd say block_hash or hash
< jtimon> sipa: where was the new style guide again, sorry?
< sipa> #10461
< gribble> https://github.com/bitcoin/bitcoin/issues/10461 | Update style guide by sipa · Pull Request #10461 · bitcoin/bitcoin · GitHub
< gmaxwell> Can we get 10454 merged soon, leaving master unbuildable on hosts with qt4 is somewhat unfortunate.
< bitcoin-git> [bitcoin] sipa opened pull request #10480: Improve commit-check-script.sh (master...update_script_check) https://github.com/bitcoin/bitcoin/pull/10480
< bitcoin-git> [bitcoin] achow101 opened pull request #10481: Decodehextx scripts sanity check (master...decodehextx-sanity) https://github.com/bitcoin/bitcoin/pull/10481