12016-09-02T00:03:02  *** gabridome has joined #bitcoin-core-dev
  22016-09-02T00:06:50  *** Ylbam has quit IRC
  32016-09-02T00:12:07  *** Alopex has quit IRC
  42016-09-02T00:13:12  *** Alopex has joined #bitcoin-core-dev
  52016-09-02T00:16:47  *** ghtdak has joined #bitcoin-core-dev
  62016-09-02T00:23:04  <GitHub92> [bitcoin] gmaxwell opened pull request #8644: [0.13 backport] Check for compatibility with download in FindNextBlocksToDownload (0.13...findnext_backport) https://github.com/bitcoin/bitcoin/pull/8644
  72016-09-02T00:24:58  <gmaxwell> Do we want to backport, "Reduce default number of blocks to check at startup"?  It's a trivial fix to long startup time complaints, and arguably the dbcache increase in 0.13 was a startup time regression for some people.
  82016-09-02T00:25:59  <GitHub29> [bitcoin] droark opened pull request #8645: Remove unused Qt 4.6 patch. (master...master) https://github.com/bitcoin/bitcoin/pull/8645
  92016-09-02T00:26:35  <sipa> i would be fime with that, though calling it a bugfix is a stretch
 102016-09-02T00:27:15  <gmaxwell> well I gave a pragmatic argument for it being one.
 112016-09-02T00:27:33  <gmaxwell> but I don't think I needed to, it's pretty obviously riskless with no "interface" impact.
 122016-09-02T00:28:10  <sipa> yes, i agree it has very low risk
 132016-09-02T00:28:26  <gmaxwell> which I think beyond 'fix' should be the goal for point releases, nothing that is going to turn a working deployment into a broken one.
 142016-09-02T00:32:19  <luke-jr> sounds reasonable to do, in any case
 152016-09-02T00:36:01  <GitHub176> [bitcoin] gmaxwell opened pull request #8646: [0.13 backport] Reduce default number of blocks to check at startup (0.13...reduceblocks_backport) https://github.com/bitcoin/bitcoin/pull/8646
 162016-09-02T00:39:33  *** gabridome has quit IRC
 172016-09-02T00:41:16  *** gabridome has joined #bitcoin-core-dev
 182016-09-02T00:43:44  *** gabridome has joined #bitcoin-core-dev
 192016-09-02T00:44:16  <GitHub102> [bitcoin] gmaxwell opened pull request #8648: [0.13 backport] Precompute sighashes (0.13...precompute_backport) https://github.com/bitcoin/bitcoin/pull/8648
 202016-09-02T00:44:21  <gmaxwell> wumpus: feel free to close my backport prs if you don't find them helpful, I was just feeling usless simply saying X or Y should be backported rather than doing it.
 212016-09-02T00:55:17  *** Alopex has quit IRC
 222016-09-02T00:56:22  *** Alopex has joined #bitcoin-core-dev
 232016-09-02T00:57:11  *** gabridome has quit IRC
 242016-09-02T01:00:26  *** TomMc has joined #bitcoin-core-dev
 252016-09-02T01:13:10  *** grubles has quit IRC
 262016-09-02T01:16:32  *** Chris_St1 has joined #bitcoin-core-dev
 272016-09-02T01:17:04  *** justanotheruser has quit IRC
 282016-09-02T01:17:34  *** justanotheruser has joined #bitcoin-core-dev
 292016-09-02T01:34:13  *** Giszmo has quit IRC
 302016-09-02T01:34:29  *** achow101 has quit IRC
 312016-09-02T01:35:07  *** achow101 has joined #bitcoin-core-dev
 322016-09-02T01:38:42  <jl2012> We need a nulldummy only bip?
 332016-09-02T01:38:49  *** Chris_St1 has quit IRC
 342016-09-02T01:42:03  <jl2012> And I already have a PR for empty failing signature https://github.com/bitcoin/bitcoin/pull/8634
 352016-09-02T01:47:37  *** dcousens has joined #bitcoin-core-dev
 362016-09-02T01:51:36  *** Chris_St1 has joined #bitcoin-core-dev
 372016-09-02T01:57:01  *** Alopex has quit IRC
 382016-09-02T01:58:06  *** Alopex has joined #bitcoin-core-dev
 392016-09-02T02:01:00  *** Chris_St1 has quit IRC
 402016-09-02T02:04:56  <cfields> sdaftuar: did you say you were able to reproduce the segwit.py failure locally?
 412016-09-02T02:05:39  <cfields> sdaftuar: if so: https://github.com/theuni/bitcoin/commit/fa88a425c0c1089ab75ea9223699286f40c8eea7
 422016-09-02T02:06:10  <cfields> it's pretty hackish, needs a cleanup. But should be enough to determine if it fixes the problem
 432016-09-02T02:07:18  <cfields> wumpus: let me know if ^^ is what you had in mind
 442016-09-02T02:09:13  <cfields> that not only eliminates polling, but also drops cs_main locking, so i should think it'd be a good bit quicker
 452016-09-02T02:19:22  *** Alopex has quit IRC
 462016-09-02T02:20:27  *** Alopex has joined #bitcoin-core-dev
 472016-09-02T02:21:14  *** ghtdak has quit IRC
 482016-09-02T02:25:34  *** Samdney has left #bitcoin-core-dev
 492016-09-02T02:25:44  *** achow101 has quit IRC
 502016-09-02T02:27:04  *** TomMc has quit IRC
 512016-09-02T02:29:04  *** jtimon has quit IRC
 522016-09-02T02:30:21  *** Alopex has quit IRC
 532016-09-02T02:31:26  *** Alopex has joined #bitcoin-core-dev
 542016-09-02T02:35:46  *** adiabat has joined #bitcoin-core-dev
 552016-09-02T02:41:01  *** Alopex has quit IRC
 562016-09-02T02:42:07  *** Alopex has joined #bitcoin-core-dev
 572016-09-02T02:55:01  *** Alopex has quit IRC
 582016-09-02T02:55:29  *** fengling has joined #bitcoin-core-dev
 592016-09-02T02:56:06  *** Alopex has joined #bitcoin-core-dev
 602016-09-02T03:00:26  *** fengling has quit IRC
 612016-09-02T03:04:20  *** justan0theruser has joined #bitcoin-core-dev
 622016-09-02T03:05:57  *** justanotheruser has quit IRC
 632016-09-02T03:24:06  *** Alopex has quit IRC
 642016-09-02T03:25:12  *** Alopex has joined #bitcoin-core-dev
 652016-09-02T03:36:38  *** jcorgan has joined #bitcoin-core-dev
 662016-09-02T03:45:32  *** dcousens has quit IRC
 672016-09-02T03:47:48  *** TomMc has joined #bitcoin-core-dev
 682016-09-02T03:55:21  *** Alopex has quit IRC
 692016-09-02T03:56:27  *** Alopex has joined #bitcoin-core-dev
 702016-09-02T03:57:16  *** fengling has joined #bitcoin-core-dev
 712016-09-02T04:01:35  *** spudowiar has quit IRC
 722016-09-02T04:02:06  *** fengling has quit IRC
 732016-09-02T04:05:22  *** TomMc has quit IRC
 742016-09-02T04:11:14  *** JackH has quit IRC
 752016-09-02T04:11:49  *** JackH has joined #bitcoin-core-dev
 762016-09-02T04:22:16  *** Alopex has quit IRC
 772016-09-02T04:23:21  *** Alopex has joined #bitcoin-core-dev
 782016-09-02T04:33:40  *** kadoban has quit IRC
 792016-09-02T04:37:25  *** cfields has quit IRC
 802016-09-02T04:40:40  *** achow101 has joined #bitcoin-core-dev
 812016-09-02T04:42:38  *** achow101 has quit IRC
 822016-09-02T04:44:06  *** Alopex has quit IRC
 832016-09-02T04:45:12  *** Alopex has joined #bitcoin-core-dev
 842016-09-02T04:48:33  *** cfields has joined #bitcoin-core-dev
 852016-09-02T04:52:07  *** achow101 has joined #bitcoin-core-dev
 862016-09-02T04:57:49  *** achow101 has quit IRC
 872016-09-02T04:58:58  *** fengling has joined #bitcoin-core-dev
 882016-09-02T05:03:30  <GitHub186> [bitcoin] JeremyRubin opened pull request #8650: Make tests much faster by replacing BOOST_CHECK with FAST_CHECK (master...faster_tests) https://github.com/bitcoin/bitcoin/pull/8650
 892016-09-02T05:03:46  *** fengling has quit IRC
 902016-09-02T05:10:51  *** justan0theruser has quit IRC
 912016-09-02T05:13:55  <gmaxwell> down with 'test frameworks'.
 922016-09-02T05:18:43  *** gabridome has joined #bitcoin-core-dev
 932016-09-02T05:19:04  *** paveljanik has quit IRC
 942016-09-02T05:22:29  *** Ylbam has joined #bitcoin-core-dev
 952016-09-02T05:29:21  *** Alopex has quit IRC
 962016-09-02T05:30:26  *** Alopex has joined #bitcoin-core-dev
 972016-09-02T05:31:12  *** gabridome has quit IRC
 982016-09-02T05:45:01  *** dcousens has joined #bitcoin-core-dev
 992016-09-02T05:47:00  *** fengling has joined #bitcoin-core-dev
1002016-09-02T05:48:56  <jeremyrubin> gmaxwell: Hear, hear!
1012016-09-02T05:51:09  <GitHub124> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f061415d12ce...91990ee01dab
1022016-09-02T05:51:10  <GitHub124> bitcoin/master 854f1af Pieter Wuille: Make the dummy argument to getaddednodeinfo optional
1032016-09-02T05:51:10  <GitHub124> bitcoin/master 91990ee Wladimir J. van der Laan: Merge #8272: Make the dummy argument to getaddednodeinfo optional...
1042016-09-02T05:51:14  <GitHub133> [bitcoin] laanwj closed pull request #8272: Make the dummy argument to getaddednodeinfo optional (master...optionaladdnodedummy) https://github.com/bitcoin/bitcoin/pull/8272
1052016-09-02T05:52:07  <gmaxwell> jeremyrubin: I had someone 'testframeworkize' one of my tests in another project and it caused it to go from 45 second runtime something like an hour.
1062016-09-02T05:55:19  *** jannes has quit IRC
1072016-09-02T06:02:32  *** Alopex has quit IRC
1082016-09-02T06:03:37  *** Alopex has joined #bitcoin-core-dev
1092016-09-02T06:05:02  *** assder has joined #bitcoin-core-dev
1102016-09-02T06:15:00  <jeremyrubin> gross :/
1112016-09-02T06:26:47  *** ghtdak has joined #bitcoin-core-dev
1122016-09-02T06:44:46  *** fengling has quit IRC
1132016-09-02T06:52:13  *** gabridome has joined #bitcoin-core-dev
1142016-09-02T06:54:29  *** gabridome has quit IRC
1152016-09-02T06:56:54  *** BashCo has quit IRC
1162016-09-02T07:03:49  *** kyletorpey has quit IRC
1172016-09-02T07:10:48  *** gabridome1 has quit IRC
1182016-09-02T07:11:16  *** Alopex has quit IRC
1192016-09-02T07:12:22  *** Alopex has joined #bitcoin-core-dev
1202016-09-02T07:15:45  *** BashCo has joined #bitcoin-core-dev
1212016-09-02T07:20:49  *** pmienk has quit IRC
1222016-09-02T07:22:05  *** paveljanik has joined #bitcoin-core-dev
1232016-09-02T07:32:54  *** pmienk has joined #bitcoin-core-dev
1242016-09-02T07:37:31  *** pmienk has quit IRC
1252016-09-02T07:37:50  *** BashCo_ has joined #bitcoin-core-dev
1262016-09-02T07:41:00  *** BashCo has quit IRC
1272016-09-02T07:41:42  *** laurentmt has joined #bitcoin-core-dev
1282016-09-02T07:44:05  <jonasschnelli> Hmm... IBDing from a local peer (same host) seems to take much longer the bootstraping from random peers.
1292016-09-02T07:46:21  <gmaxwell> early in the sync it fetches to few a number of blocks at a time, so fetching from a single peer is slow. hardly matters overall.
1302016-09-02T07:46:31  <gmaxwell> other than that, they'll obviously contend a bit for IO
1312016-09-02T07:46:50  <gmaxwell> I'm not aware of any other reasons for slowdowns other than that.
1322016-09-02T07:47:23  <paveljanik> jonasschnelli, OS X<
1332016-09-02T07:47:24  <paveljanik> ?
1342016-09-02T07:48:02  <jonasschnelli> paveljanik: no debian on my Intel(R) Xeon(R) CPU E3-1275 v5 @ 3.60GHz
1352016-09-02T07:48:28  <paveljanik> I had the same issue on OS X in the past. unplugging the network cable, turning off firewall helped 8)
1362016-09-02T07:48:43  <gmaxwell> are you only looking at the first 150k blocks? or are you seeing the rate per block slower later?
1372016-09-02T07:48:47  <paveljanik> the process named powerd took 100% of CPU power
1382016-09-02T07:49:29  <jonasschnelli> gmaxwell: It took more then 1h for progress=0.035480
1392016-09-02T07:49:53  <jonasschnelli> (where is did sync with random peers up to progress 1.0 in ~3h [2 month ago though])
1402016-09-02T07:49:55  *** pmienk has joined #bitcoin-core-dev
1412016-09-02T07:50:56  <gmaxwell> better to look at time vs height, e.g. grab a chunk of 100 blocks wherever it is and comare the blocks per second at that height with the same height in your prior logs.
1422016-09-02T07:50:59  <jonasschnelli> Also... my extremely scientific measure of how fast the tail -f output scrolls though.... did tell me its slower
1432016-09-02T07:51:57  <jonasschnelli> I guess i let it sync up to h400'000 and then compare it against a sync from random peers
1442016-09-02T07:51:59  *** laurentmt has quit IRC
1452016-09-02T07:52:28  <jonasschnelli> The only differences are: -prune=550 and connecting to a node thats running on the same machine (in sync, should not cause a huge slowdown)
1462016-09-02T07:52:49  * jonasschnelli needs to write a IBD benchmark tool
1472016-09-02T07:53:00  <gmaxwell> prune might make it slower.
1482016-09-02T07:53:20  <jonasschnelli> the prune data eviction seems pretty fast
1492016-09-02T07:53:21  <gmaxwell> after all it's writing things to only delete it after.
1502016-09-02T07:53:35  <jonasschnelli> it's running on a 1GB/s SSD
1512016-09-02T07:55:18  <gmaxwell> I doubt anyone has benmarked prune vs not, if you told me that it was doing some linear scan of all the blocks to decide what to prune, every block-- and was thus massively slower, I wouldn't not be too surprised.
1522016-09-02T07:57:00  <GitHub55> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/91990ee01dab...6f939c9080a3
1532016-09-02T07:57:01  <GitHub55> bitcoin/master cdd79eb Jorge Timón: C++11: s/boost::scoped_ptr/std::unique_ptr/
1542016-09-02T07:57:01  <GitHub55> bitcoin/master 6f939c9 Wladimir J. van der Laan: Merge #8629: C++11: s/boost::scoped_ptr/std::unique_ptr/...
1552016-09-02T07:57:08  * jonasschnelli is trying again without prune
1562016-09-02T07:57:15  <GitHub51> [bitcoin] laanwj closed pull request #8629: C++11: s/boost::scoped_ptr/std::unique_ptr/ (master...0.13-boost-scoped-ptr) https://github.com/bitcoin/bitcoin/pull/8629
1572016-09-02T08:00:14  <sipa> pruning causes flushing
1582016-09-02T08:01:11  <gmaxwell> oh obviously, it needs to make sure that after a restart it can continue, so it hast to at least be consistent to the point where it pruned.
1592016-09-02T08:07:12  *** AaronvanW has joined #bitcoin-core-dev
1602016-09-02T08:07:40  *** AaronvanW has quit IRC
1612016-09-02T08:07:40  *** AaronvanW has joined #bitcoin-core-dev
1622016-09-02T08:10:44  <jonasschnelli> What is the reason for the txn_count=0x00 in the headers p2p message?
1632016-09-02T08:11:49  *** fengling has joined #bitcoin-core-dev
1642016-09-02T08:13:10  <sipa> because CBlockHeader did not originally exist
1652016-09-02T08:13:30  <sipa> and headers messages just contained a CBlock with 0 transactions
1662016-09-02T08:13:55  <jonasschnelli> thanks.
1672016-09-02T08:20:16  <jeremyrubin> Is it correct that the maximum number of items deleted from sigcache while checking a block is max sigops?
1682016-09-02T08:20:58  <sipa> i believe so
1692016-09-02T08:21:55  <jeremyrubin> And is that "MAX_BLOCK_SIGOPS_COST" or is that something else
1702016-09-02T08:22:03  <jeremyrubin> because I see a 20k number floating around too
1712016-09-02T08:22:08  <jeremyrubin> but MAX_BLOCK_SIGOPS_COST=80k
1722016-09-02T08:22:32  <sipa> yes, 4x factor from segwit
1732016-09-02T08:22:57  <sipa> sigops in non-witness part count as 4, in witness part they count as 1
1742016-09-02T08:23:39  <jeremyrubin> so if you have 0 sigops in witness, you can have the full 80k?
1752016-09-02T08:24:13  <sipa> if you have 0 sigops in non-witness
1762016-09-02T08:24:15  <sipa> yes
1772016-09-02T08:24:18  <jeremyrubin> ok cool
1782016-09-02T08:24:22  <sipa> pre segwit the max is 20k
1792016-09-02T08:24:29  <sipa> post segwit it is 80k
1802016-09-02T08:39:01  *** Alopex has quit IRC
1812016-09-02T08:40:06  *** Alopex has joined #bitcoin-core-dev
1822016-09-02T08:40:32  *** JackH has quit IRC
1832016-09-02T08:58:43  <jonasschnelli> with disabled pruning it took also >1h for progress=0.031407... now trying a sync from random peers
1842016-09-02T09:00:16  <gmaxwell> you're killin me with the testing with just the first 3%.
1852016-09-02T09:00:41  <jonasschnelli> gmaxwell: heh. You think comparing the first 3% does not make sense?
1862016-09-02T09:01:14  <jonasschnelli> I just try to not waste time. :)
1872016-09-02T09:01:33  <gmaxwell> it's very unrepresentative, as I explaned earlier. We only fetch 16 blocks per peer at a time ... early in the chain this is far too few to keep the system well pipelined, because the blocks take no work to process. :)
1882016-09-02T09:02:42  <jonasschnelli> gmaxwell: Okay. I see. But anyways, a sync with a single local peer will then take always longer, right?
1892016-09-02T09:03:18  <gmaxwell> yes, somewhat. the difference will only be in the initial part of the chain, but go away later-- at least, whatever issue was due to that.
1902016-09-02T09:04:09  <gmaxwell> it's like the 0.13 vs 0.12 reindex, the 0.13 reindex spends 20 minutes at the front reading headers. So it starts off 20 minutes behind, it still finishes earlier.
1912016-09-02T09:04:35  <gmaxwell> but if you were to compare reintext for 0.13 vs 0.12 for the first 3% you would likely find 0.13 was much slower. :)
1922016-09-02T09:11:30  *** AaronvanW has quit IRC
1932016-09-02T09:16:36  *** AaronvanW has joined #bitcoin-core-dev
1942016-09-02T09:16:37  *** AaronvanW has quit IRC
1952016-09-02T09:16:37  *** AaronvanW has joined #bitcoin-core-dev
1962016-09-02T09:33:46  *** spudowiar has joined #bitcoin-core-dev
1972016-09-02T09:36:00  *** murch has joined #bitcoin-core-dev
1982016-09-02T09:38:31  *** jtimon has joined #bitcoin-core-dev
1992016-09-02T09:44:15  <GitHub197> [bitcoin] sipa opened pull request #8651: Predeclare PrecomputedTransactionData as struct (master...classtructblah) https://github.com/bitcoin/bitcoin/pull/8651
2002016-09-02T09:48:03  <GitHub77> [bitcoin] yurizhykin opened pull request #8652: [qa]: remove root test directory for RPC tests (master...cleanup) https://github.com/bitcoin/bitcoin/pull/8652
2012016-09-02T09:52:46  *** fengling has quit IRC
2022016-09-02T10:04:35  *** fengling has joined #bitcoin-core-dev
2032016-09-02T10:19:20  *** jannes has joined #bitcoin-core-dev
2042016-09-02T10:26:26  <GitHub76> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/6f939c9080a3...381d0ddc8aaa
2052016-09-02T10:26:27  <GitHub76> bitcoin/master cce3024 Pavel Janík: Do not shadow local variable, cleanup
2062016-09-02T10:26:28  <GitHub76> bitcoin/master a159f25 Pavel Janík: Remove redundand (and shadowing) declaration
2072016-09-02T10:26:28  <GitHub76> bitcoin/master 381d0dd Wladimir J. van der Laan: Merge #8449: [Trivial] Do not shadow local variable, cleanup...
2082016-09-02T10:26:32  <GitHub190> [bitcoin] laanwj closed pull request #8449: [Trivial] Do not shadow local variable, cleanup (master...20160803_shadow_blockencodings) https://github.com/bitcoin/bitcoin/pull/8449
2092016-09-02T10:30:06  *** BashCo has joined #bitcoin-core-dev
2102016-09-02T10:31:52  *** BashCo_ has quit IRC
2112016-09-02T10:43:26  *** fengling has quit IRC
2122016-09-02T10:45:52  *** owowo has quit IRC
2132016-09-02T10:52:18  <GitHub29> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/381d0ddc8aaa...cbe9ae8c69b9
2142016-09-02T10:52:19  <GitHub29> bitcoin/master b7c349d Pavel Janík: Do not shadow variables in networking code
2152016-09-02T10:52:19  <GitHub29> bitcoin/master cbe9ae8 Wladimir J. van der Laan: Merge #8466: [Trivial] Do not shadow variables in networking code...
2162016-09-02T10:52:23  <GitHub92> [bitcoin] laanwj closed pull request #8466: [Trivial] Do not shadow variables in networking code (master...20160805_Wshadow_net) https://github.com/bitcoin/bitcoin/pull/8466
2172016-09-02T10:54:59  *** fengling has joined #bitcoin-core-dev
2182016-09-02T11:03:03  *** jcorgan_ has joined #bitcoin-core-dev
2192016-09-02T11:06:02  *** timothy has quit IRC
2202016-09-02T11:06:04  *** drizztbsd has joined #bitcoin-core-dev
2212016-09-02T11:06:36  *** drizztbsd is now known as timothy
2222016-09-02T11:07:31  *** jcorgan has quit IRC
2232016-09-02T11:07:32  *** mn3monic has quit IRC
2242016-09-02T11:07:32  *** sanada` has quit IRC
2252016-09-02T11:07:33  *** Bootvis has quit IRC
2262016-09-02T11:07:34  *** crudel has quit IRC
2272016-09-02T11:07:34  *** harding has quit IRC
2282016-09-02T11:07:34  *** Taek has quit IRC
2292016-09-02T11:09:54  *** mn3monic has joined #bitcoin-core-dev
2302016-09-02T11:09:54  *** sanada` has joined #bitcoin-core-dev
2312016-09-02T11:09:54  *** Bootvis has joined #bitcoin-core-dev
2322016-09-02T11:09:54  *** crudel has joined #bitcoin-core-dev
2332016-09-02T11:09:54  *** harding has joined #bitcoin-core-dev
2342016-09-02T11:09:54  *** Taek has joined #bitcoin-core-dev
2352016-09-02T11:19:46  *** fengling has quit IRC
2362016-09-02T11:30:26  *** fengling has joined #bitcoin-core-dev
2372016-09-02T11:38:48  <cfields> jonasschnelli: so your comment on #8085 may not be relevant to those changes?
2382016-09-02T11:42:08  <GitHub97> [bitcoin] jl2012 closed pull request #8533: Implement LOW_S and NULLDUMMY softfork (BIP146) (master...bip146) https://github.com/bitcoin/bitcoin/pull/8533
2392016-09-02T11:42:56  *** BashCo has quit IRC
2402016-09-02T11:55:06  *** fengling has quit IRC
2412016-09-02T11:55:26  *** cryptapus_afk is now known as cryptapus
2422016-09-02T12:22:04  *** dcousens has quit IRC
2432016-09-02T12:36:42  *** paveljanik has quit IRC
2442016-09-02T12:37:15  *** LeMiner2 has joined #bitcoin-core-dev
2452016-09-02T12:39:07  *** LeMiner has quit IRC
2462016-09-02T12:39:07  *** LeMiner2 is now known as LeMiner
2472016-09-02T12:45:49  *** owowo has joined #bitcoin-core-dev
2482016-09-02T12:58:12  <cfields> wumpus: re-ping incase you missed in backog. what do you think of the approach here to fix the travis timeouts? https://github.com/theuni/bitcoin/commit/fa88a425c0c1089ab75ea9223699286f40c8eea7
2492016-09-02T13:03:42  *** dcousens has joined #bitcoin-core-dev
2502016-09-02T13:20:51  *** jcorgan_ is now known as jcorgan
2512016-09-02T13:23:57  <GitHub117> [bitcoin] sstone opened pull request #8653: doc (trivial): add tip for cross-builds on ubuntu (master...wip-doccrosscompile) https://github.com/bitcoin/bitcoin/pull/8653
2522016-09-02T13:29:44  *** Guyver2 has joined #bitcoin-core-dev
2532016-09-02T13:41:14  <sipa> cfields: looks good to me
2542016-09-02T13:41:58  <sipa> jonasschnelli: 3%, how many blocks is that?
2552016-09-02T13:42:15  * jonasschnelli checking the logs
2562016-09-02T13:42:54  <jonasschnelli> sipa: aprox 211000
2572016-09-02T13:43:15  <jonasschnelli> My assumption is that IBD with a single peer is much slower... but can't prove it right now.
2582016-09-02T13:43:18  <sipa> jonasschnelli: that seems very wrong
2592016-09-02T13:43:25  <sipa> it should take minutes to get to 211000
2602016-09-02T13:43:58  <sipa> not an hour
2612016-09-02T13:44:12  <sipa> the effect greg talks about it real, but shouldn't last more than a few minutes
2622016-09-02T13:44:35  <jonasschnelli> 2016-09-02 08:58:49 UpdateTip: new best=0000000000000345b371caa3f829cacbe2b4d38ecd15a5a02031efae79934d15 height=211000
2632016-09-02T13:44:38  <jonasschnelli> 2016-09-02 07:56:40 Bitcoin version v0.13.99.0-df98230
2642016-09-02T13:45:00  *** paveljanik has joined #bitcoin-core-dev
2652016-09-02T13:48:12  *** warren has quit IRC
2662016-09-02T13:49:58  *** warren has joined #bitcoin-core-dev
2672016-09-02T13:54:37  *** shesek has quit IRC
2682016-09-02T13:56:16  *** grubles has joined #bitcoin-core-dev
2692016-09-02T13:57:00  <jonasschnelli> sipa: I guess its caused by --enable-debug
2702016-09-02T13:58:43  <sipa> oh, yes, that will slow things down tremendously
2712016-09-02T14:00:02  <jonasschnelli> sry for the noise... I should finally remember to disable debug mode when benachmarking.
2722016-09-02T14:04:31  *** jtimon has quit IRC
2732016-09-02T14:16:36  *** TomMc has joined #bitcoin-core-dev
2742016-09-02T14:17:14  *** Cheeseo has quit IRC
2752016-09-02T14:17:34  *** Cheeseo has joined #bitcoin-core-dev
2762016-09-02T14:17:34  *** Cheeseo has joined #bitcoin-core-dev
2772016-09-02T14:28:37  *** grubles has quit IRC
2782016-09-02T14:29:58  *** Samdney has joined #bitcoin-core-dev
2792016-09-02T14:31:48  *** Giszmo has joined #bitcoin-core-dev
2802016-09-02T14:37:22  *** warren has quit IRC
2812016-09-02T14:38:06  *** grubles has joined #bitcoin-core-dev
2822016-09-02T14:39:48  *** warren has joined #bitcoin-core-dev
2832016-09-02T14:43:39  *** grubles has quit IRC
2842016-09-02T14:48:11  *** gabridome has joined #bitcoin-core-dev
2852016-09-02T14:48:42  *** gabridome has quit IRC
2862016-09-02T14:48:57  *** gabridome has joined #bitcoin-core-dev
2872016-09-02T14:51:14  *** grubles has joined #bitcoin-core-dev
2882016-09-02T14:53:37  *** grubles has quit IRC
2892016-09-02T14:56:39  *** grubles has joined #bitcoin-core-dev
2902016-09-02T15:05:33  *** Giszmo has quit IRC
2912016-09-02T15:07:36  *** gabridome has quit IRC
2922016-09-02T15:09:27  *** Giszmo has joined #bitcoin-core-dev
2932016-09-02T15:12:56  *** achow101 has joined #bitcoin-core-dev
2942016-09-02T15:15:47  <sturles> a/away
2952016-09-02T15:25:40  <cfields> sipa: thanks
2962016-09-02T15:26:19  <cfields> jonasschnelli: so I should ignore https://github.com/bitcoin/bitcoin/pull/8085#issuecomment-244077083 ?
2972016-09-02T15:26:40  <jonasschnelli> cfields: I think so...
2982016-09-02T15:26:50  <jonasschnelli> I'm working on more authentic IBD benchmarks.
2992016-09-02T15:27:39  <cfields> jonasschnelli: ok. apples to apples IBD is a good idea though. I'll do some syncs over the weekend
3002016-09-02T15:27:39  *** achow101 has quit IRC
3012016-09-02T15:28:11  <jonasschnelli> cfields: Yes. I was fooled (again) by the bias of --enable-debug and --prune
3022016-09-02T15:30:12  <cfields> jonasschnelli: ah, heh. I've fallen into the same trap. Turns out profiles generated with -O0 (for better info) aren't at all representative of real-world usage
3032016-09-02T15:30:28  <cfields> so it makes sense that it'd be a major performance killer
3042016-09-02T15:33:27  <gmaxwell> jonasschnelli: I think you got caught on debug before, you realize now I'm gonna start asking you? "Are you benchmarking with enable debug again?" :P
3052016-09-02T15:34:05  <jonasschnelli> heh... yes.
3062016-09-02T15:34:07  <btcdrak> Jonas 'debug' schnelli
3072016-09-02T15:34:09  <sdaftuar> cfields: i don't think i have seen segwit.py fail locally for the reason mentioned in #8532
3082016-09-02T15:34:19  <sdaftuar> gmaxwell: please set up an irc autoresponder so the rest of us don't forget to ask him too :)
3092016-09-02T15:34:49  <cfields> sdaftuar: ah, ok
3102016-09-02T15:35:20  *** cryptapus has quit IRC
3112016-09-02T15:35:45  *** cryptapus has joined #bitcoin-core-dev
3122016-09-02T15:36:11  <sdaftuar> it seems like the most common reason my rpc tests fail is the communicat() timeout thing i reported in #8649, and then new test runs failing when old bitcoind's haven't been killed from prior test runs (sigh)
3132016-09-02T15:44:57  <GitHub85> [bitcoin] jl2012 opened pull request #8654: Reuse sighash computations across evaluation (rebase of #4562) (master...sighashcache) https://github.com/bitcoin/bitcoin/pull/8654
3142016-09-02T15:56:23  <wumpus> cfields: looking
3152016-09-02T15:59:04  <wumpus> cfields: looks good to me, doesn't even require any new signals!
3162016-09-02T16:00:35  <wumpus> cfields: any reason to not put the signal subscription/unsubscription in StartRPC/StopRPC?
3172016-09-02T16:01:09  <wumpus> cfields: oh duh, those are general methods
3182016-09-02T16:01:29  <wumpus> cfields: if there was a rpcblockchain.init/deinit, it would fit there
3192016-09-02T16:02:32  <wumpus> cfields: but this is ok; maybe mark the RPC calls that they are primarily meant for testing and/or set them as hidden?
3202016-09-02T16:02:59  <wumpus> (e.g. I can see complaints from people hanging the RPC server this way with no clue what they're doing)
3212016-09-02T16:03:28  <sipa> agree
3222016-09-02T16:05:25  <cfields> wumpus: yep, makes sense
3232016-09-02T16:05:44  <cfields> and yes, i kinda abused the existing signals there. I figured you'd be grumpier about it, hence the poke rather than PR :)
3242016-09-02T16:07:48  <wumpus> cfields: well the important thing is that all other processing has been done at the time of the signal; does this guarantee that?
3252016-09-02T16:08:21  <wumpus> (I guess the answer is yes, as there is no async processing, although it could depend on the order in which the handlers are called?)
3262016-09-02T16:09:20  <sipa> by default i think handlers are called in the order they were added
3272016-09-02T16:09:37  *** jtimon has joined #bitcoin-core-dev
3282016-09-02T16:09:43  <cfields> wumpus: i believe it's fine, but i'll double-check.
3292016-09-02T16:10:15  *** kadoban has joined #bitcoin-core-dev
3302016-09-02T16:10:21  <cfields> wumpus: since it abuses the ui signal, i'm not sure we need to worry about the order? Looking to see who else receives that
3312016-09-02T16:11:28  <wumpus> sipa: yes, but that is a really fragile thing to rely on
3322016-09-02T16:11:58  <wumpus> cfields: ok, yes that makes sense, if the UI signal is last
3332016-09-02T16:12:17  <wumpus> cfields: indeed there's no need to be worried doing things concurrently with the GUI
3342016-09-02T16:12:33  <wumpus> (usually that won't be running anyhow during RPC testing, although it's possible)
3352016-09-02T16:12:47  <cfields> it doesn't guarantee that we've acted on relaying appropriately, though all we could guarantee there anyway is that we've queued up some messages
3362016-09-02T16:13:31  <cfields> right
3372016-09-02T16:15:59  <cfields> yes, only other user is qt for updating gui
3382016-09-02T16:27:58  <GitHub78> [bitcoin] paveljanik opened pull request #8655: Do not shadow variables (trivials) (master...20160902_Wshadow_trivials) https://github.com/bitcoin/bitcoin/pull/8655
3392016-09-02T16:32:20  *** jujumax has joined #bitcoin-core-dev
3402016-09-02T16:32:34  *** aalex has quit IRC
3412016-09-02T16:41:38  *** aalex has joined #bitcoin-core-dev
3422016-09-02T16:52:13  <phantomcircuit> jonasschnelli, the p2p network contains a bunch of nodes which exist only to monitor people and as a side effect have bizarre behaviour
3432016-09-02T16:58:48  *** jujumax has quit IRC
3442016-09-02T16:59:02  *** Chris_Stewart_5 has joined #bitcoin-core-dev
3452016-09-02T17:18:18  *** gabridome has joined #bitcoin-core-dev
3462016-09-02T17:20:27  *** gabridome has quit IRC
3472016-09-02T17:21:21  *** Chris_St1 has joined #bitcoin-core-dev
3482016-09-02T17:22:09  *** Chris_Stewart_5 has quit IRC
3492016-09-02T17:22:44  *** jtimon has quit IRC
3502016-09-02T17:23:17  *** jtimon has joined #bitcoin-core-dev
3512016-09-02T17:35:54  *** spudowiar has quit IRC
3522016-09-02T17:59:06  *** spudowiar has joined #bitcoin-core-dev
3532016-09-02T18:00:34  *** Chris_St1 has quit IRC
3542016-09-02T18:08:59  *** spudowiar has quit IRC
3552016-09-02T18:16:09  *** Chris_St1 has joined #bitcoin-core-dev
3562016-09-02T18:27:25  *** spudowiar has joined #bitcoin-core-dev
3572016-09-02T18:31:29  *** Chris_St1 has quit IRC
3582016-09-02T18:37:20  *** btcdrak has quit IRC
3592016-09-02T18:47:16  *** jtimon has quit IRC
3602016-09-02T18:47:28  *** Chris_St1 has joined #bitcoin-core-dev
3612016-09-02T18:55:11  *** CubicEarth has joined #bitcoin-core-dev
3622016-09-02T18:58:04  <GitHub12> [bitcoin] paveljanik opened pull request #8656: Do not shadow global variable fileout (master...20160902_Wshadow_fileout) https://github.com/bitcoin/bitcoin/pull/8656
3632016-09-02T19:01:24  <instagibbs> sipa, where is chainActive loaded on init? Having trouble tracking that down.
3642016-09-02T19:04:19  <sipa> instagibbs: ActivateBestChain?
3652016-09-02T19:04:26  <sipa> LoadBlockIndex?
3662016-09-02T19:06:49  <instagibbs> eh yes, but it got moved around and I'm flailing
3672016-09-02T19:07:04  <instagibbs> I'll track it down
3682016-09-02T19:07:51  *** CubicEar_ has joined #bitcoin-core-dev
3692016-09-02T19:09:58  <instagibbs> ah i misunderstood what wallet init was doing with it, nevermind, thanks
3702016-09-02T19:10:02  *** btcdrak has joined #bitcoin-core-dev
3712016-09-02T19:10:03  *** CubicEarth has quit IRC
3722016-09-02T19:16:31  *** e4xit has quit IRC
3732016-09-02T19:17:01  *** CubicEar_ has quit IRC
3742016-09-02T19:18:04  *** CubicEarth has joined #bitcoin-core-dev
3752016-09-02T19:19:33  *** TomMc has quit IRC
3762016-09-02T19:21:36  *** spudowiar has quit IRC
3772016-09-02T19:27:02  *** CubicEarth has quit IRC
3782016-09-02T19:30:31  *** e4xit has joined #bitcoin-core-dev
3792016-09-02T19:35:19  *** laurentmt has joined #bitcoin-core-dev
3802016-09-02T19:37:20  *** laurentmt has quit IRC
3812016-09-02T19:38:57  *** laurentmt has joined #bitcoin-core-dev
3822016-09-02T19:39:23  *** laurentmt has quit IRC
3832016-09-02T19:57:33  *** e4xit has quit IRC
3842016-09-02T19:58:08  *** e4xit has joined #bitcoin-core-dev
3852016-09-02T20:08:40  *** drizztbsd has joined #bitcoin-core-dev
3862016-09-02T20:08:55  *** timothy has quit IRC
3872016-09-02T20:09:06  *** drizztbsd is now known as timothy
3882016-09-02T20:09:26  <GitHub70> [bitcoin] paveljanik opened pull request #8658: WIP/DO NOT MERGE: Remove unused statements in serialization (master...20160902_nVersion_serialization_cleanup) https://github.com/bitcoin/bitcoin/pull/8658
3892016-09-02T20:25:17  *** shesek has joined #bitcoin-core-dev
3902016-09-02T20:34:33  *** shesek has quit IRC
3912016-09-02T20:47:19  *** shesek has joined #bitcoin-core-dev
3922016-09-02T21:01:01  *** Guyver2 has quit IRC
3932016-09-02T21:18:09  *** veleiro has joined #bitcoin-core-dev
3942016-09-02T21:58:39  *** gluytium has quit IRC
3952016-09-02T22:03:08  *** pmienk has quit IRC
3962016-09-02T22:03:13  *** gluytium has joined #bitcoin-core-dev
3972016-09-02T22:15:01  *** veleiro has quit IRC
3982016-09-02T22:15:50  *** pmienk has joined #bitcoin-core-dev
3992016-09-02T22:20:32  *** cryptapus is now known as cryptapus_afk
4002016-09-02T22:20:37  *** justanotheruser has joined #bitcoin-core-dev
4012016-09-02T22:33:34  *** pmienk has quit IRC
4022016-09-02T22:46:23  *** pmienk has joined #bitcoin-core-dev
4032016-09-02T22:59:23  *** spudowiar has joined #bitcoin-core-dev
4042016-09-02T23:13:45  *** murch has quit IRC
4052016-09-02T23:14:58  *** jtimon has joined #bitcoin-core-dev
4062016-09-02T23:18:19  *** crudel has quit IRC
4072016-09-02T23:24:28  *** slackircbridge has quit IRC
4082016-09-02T23:49:53  *** CyrusV has joined #bitcoin-core-dev
4092016-09-02T23:50:12  *** AaronvanW has quit IRC