04:28 < leangjia> Hi,all.

11:17 < GitHub119> [bitcoin] MarcoFalke opened pull request #8863: univalue: Pull subtree (master...Mf1610-univalue) https://github.com/bitcoin/bitcoin/pull/8863

12:30 < luke-jr> weird, starting my node shows all transactions back to 2010 as "Offline" even after a few connections

12:31 < luke-jr> would expect it to show the ones that confirmed in the past as confirmed at least..

12:31 < luke-jr> (I can understand not trusting newly confirmed stuff with few connections)

12:33 < sipa> transactions can be "offline" ?

12:34 < luke-jr> sipa: apparently ☺

12:34 < luke-jr> they showed normal after 5 or 6 connections

12:47 < wumpus> yes, transactions (in the gui) can be "offline". That transaction classification code is pretty much what it was in satoshi's code, unchanged over all that time

12:47 < wumpus> GetAdjustedTime() - wtx.nTimeReceived > 2 * 60 && wtx.GetRequestCount() == 0

12:49 < wumpus> most notably it doesn't check the depth, so something can be 100/offline, for whatever that's worth

12:50 < wumpus> 'longer than 2 hours old and never requested by other peers'

13:03 < luke-jr> wumpus: all of these have been long-confirmed by other peers :p

13:31 < wumpus> luke-jr: the logic makes little sense. I wouldn't mind removing that status completely. I don't think it's documented anywhere, and if it surprises even the devs :)

16:34 < wumpus> FYI no dev meeting this week https://lists.linuxfoundation.org/pipermail/bitcoin-core-dev/2016-October/000020.html

16:50 < achow101> how close are we to 0.13.1 release?

16:53 < btcdrak> achow101: see the milestone.

16:53 < BlueMatt> wumpus: whats you opinion on big code moves?

16:53 < BlueMatt> wumpus: specifically, https://github.com/TheBlueMatt/bitcoin/commit/af4caa3fd5000da6187a71dbba9b0ca3ff18e3d4

16:54 < BlueMatt> grrr, damn github...whatever, pulls out the CNodeState management from the top of main.cpp to the end as a part of https://github.com/TheBlueMatt/bitcoin/commits/net_processing_file, which disentangles that stuff somewhat and is like 2 functions away from splitting main.cpp in two

17:27 < wumpus> achow101: people keep adding new pulls for 0.13.1, but by far the largest change that still has to go in is https://github.com/bitcoin/bitcoin/pull/8393

17:27 < wumpus> BlueMatt: code moves are easy to review/verify, so if it makes sense from a code organization viewpoint (and people agree on that) it's fine

17:32 < btcdrak> wumpus: there are some trivial merges tho. I think can merge #8848, #8857 and #8817. Havent looked at #8862 but seems trivial enough.

19:05 < BlueMatt> wumpus: i was asking in the context of 0.14...when in the release cycle did we say we wanted to do code moves?

19:20 < paveljanik> BlueMatt, in the Next one :-)

20:58 < GitHub35> [bitcoin] TheBlueMatt opened pull request #8865: Decouple peer-processing-logic from block-connection-logic (master...net_processing_1) https://github.com/bitcoin/bitcoin/pull/8865

21:05 < luke-jr> wumpus: well, we really should have better logic for the statuses in general. removing offline alone may make things worse.

21:05 < luke-jr> wumpus: if we have few peers or block rate suggests we may be partitioned, we should distrust confirmation

21:15 < GitHub168> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6faffb8a83db...6013c73b3312

21:15 < GitHub168> bitcoin/master fa66609 MarcoFalke: [qa] mininode: Only allow named args in wait_until

21:15 < GitHub168> bitcoin/master 6013c73 MarcoFalke: Merge #8857: [qa] mininode: Only allow named args in wait_until...

21:15 < GitHub84> [bitcoin] MarcoFalke closed pull request #8857: [qa] mininode: Only allow named args in wait_until (master...Mf1610-qaMininodeWaitUntil) https://github.com/bitcoin/bitcoin/pull/8857