1 2017-05-11 02:13:47	0|bitcoin-git|[13bitcoin] 15kallewoof opened pull request #10386: [wallet] Optional '-avoidreuse' flag which defaults to not reusing addresses in sends (06master...06feature-white-black-address) 02https://github.com/bitcoin/bitcoin/pull/10386
  2 2017-05-11 05:18:46	0|bitcoin-git|13bitcoin/06master 14f203ecc 15Pavel Janík: Shadowing is not enabled by default, update doc accordingly.
  3 2017-05-11 05:18:46	0|bitcoin-git|[13bitcoin] 15laanwj pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/a26280bc1415...7f2b9e0868f5
  4 2017-05-11 05:18:47	0|bitcoin-git|13bitcoin/06master 147f2b9e0 15Wladimir J. van der Laan: Merge #10381: Shadowing warnings are not enabled by default, update doc accordingly...
  5 2017-05-11 05:19:17	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #10381: Shadowing warnings are not enabled by default, update doc accordingly (06master...0620170510_Wshadow_not_enabled_by_default) 02https://github.com/bitcoin/bitcoin/pull/10381
  6 2017-05-11 07:04:42	0|ckm|anybody know why bitcoin started on microsoft/windows?
  7 2017-05-11 07:05:01	0|ckm|seems odd for an open-source project
  8 2017-05-11 07:08:34	0|kallewoof|Probably because Satoshi was using Windows when he wrote it
  9 2017-05-11 07:15:44	0|ckm|okay, yeah, that's a pastability. not sure though. maybe it was that the alternatives were less prevalent then, so it could have been strategic to get more miners. i kinda want a logical explanation. like it could have been a statement like hacking microsoft/windows in a phoenix type of way. hmm. your thoughts?
 10 2017-05-11 07:22:56	0|sipa|my assumption is that satoshi or whoever wrote the first code was a windows programmer
 11 2017-05-11 07:23:10	0|sipa|the naming style of variables etc is consistent with that
 12 2017-05-11 07:29:09	0|ckm|okay going with that then is there any indication that this was a microsoft job? they do seem to be the most supportive big tech company, including support in visual basic, openly wanting a cashless money system, and with statements of public approval
 13 2017-05-11 07:59:08	0|sipa|offtopic
 14 2017-05-11 08:05:30	0|wumpus|there are likely more windows programmers than unix programmers in the world, so the chance of a software project starting as windows software is fairly high. Seems kind of a big leap to say it's a microsoft job then. If it had started as linux project, would you have personally implicated Linus?
 15 2017-05-11 08:05:36	0|wumpus|but yes, offtopic, #bitcoin please
 16 2017-05-11 08:16:41	0|ckm|i just looked through all the original code and it doesn't seem like a "microsoft job" from the comments et cettera. i wasn't very into computers, i thought linux was hard to use back then, so i was using windows/mac but a programmer would i think have been able to use it. although it would be fairest to the public to make a windows version so everybody could in theory mine the genisis block.
 17 2017-05-11 08:16:52	0|ckm|and sorry for offtopic i will switch over to there, thanks!
 18 2017-05-11 09:03:36	0|jonasschnelli|Can anyone explain when a peer does relay its own CAddress to a connected peer? When I connect with mininode and inject a getaddr, I get now addr message
 19 2017-05-11 09:04:20	0|wumpus|on incoming connections, yes
 20 2017-05-11 09:04:45	0|jonasschnelli|wumpus: hmm... maybe I got fooled by the AddNewAddresses time penalty?
 21 2017-05-11 09:05:12	0|wumpus|sure, or maybe there's a bug, I don't know
 22 2017-05-11 09:05:30	0|jonasschnelli|Okay... i'll check.. maybe my local changes broke thinsg...
 23 2017-05-11 09:05:41	0|wumpus|it seems advertizing works, but the issues about IP changes not propagating are kind of strange
 24 2017-05-11 09:06:28	0|wumpus|sorry I meant on outgoing connections, incoming wouldn't make sense as the peer already found you :)
 25 2017-05-11 09:06:58	0|jonasschnelli|wumpus: Ah... right..
 26 2017-05-11 09:07:48	0|wumpus|so when your peer connects to someone else, it learns its own address (through the peer's version message), it starts advertizing that
 27 2017-05-11 09:08:31	0|jonasschnelli|Though, if I start two peers (A,B) and connect_nodes_bi(A.B), then connect with a mininode (C) to B, I guess B should relay addr(A) when I send a getaddr
 28 2017-05-11 09:09:55	0|wumpus|yes I think so, it should gossip about addresses it knows, though it's heavily randomized
 29 2017-05-11 09:15:58	0|sipa|eventually, yes
 30 2017-05-11 09:16:04	0|sipa|but that may take hours
 31 2017-05-11 09:16:51	0|jonasschnelli|sipa: do you see a better/simpler way how to test if a peer has relayed a addr (through our test/functional/ framework)?
 32 2017-05-11 09:17:42	0|jonasschnelli|(examine peers.dat is probably a very bad idea)
 33 2017-05-11 09:18:27	0|wumpus|parsing peers.dat from the test framework seems painful
 34 2017-05-11 09:19:06	0|jonasschnelli|If there would be a way to retrieve addrmans content from the outside..
 35 2017-05-11 09:19:08	0|wumpus|also it's not really an interface to be tested, the peers.dat format should be flexible
 36 2017-05-11 09:19:16	0|jonasschnelli|yeah.. indeed
 37 2017-05-11 09:19:22	0|wumpus|agreed
 38 2017-05-11 09:19:41	0|wumpus|some testing-only RPC?
 39 2017-05-11 09:20:11	0|warren|why testing only?
 40 2017-05-11 09:20:16	0|wumpus|that part of the code needs to be more testable
 41 2017-05-11 09:20:26	0|wumpus|so we can assure that it works and track regressions
 42 2017-05-11 09:20:49	0|wumpus|I don't see much point in having it as documented outside interface, but if you have use cases, sure...
 43 2017-05-11 09:21:05	0|jonasschnelli|wumpus: Yes. Thought the same... I try to add an debug RPC to outputs addrmans content
 44 2017-05-11 09:21:09	0|warren|I mean, it would be useful to be able to dump what it thinks it knows about peers during runtime of an ordinary node.
 45 2017-05-11 09:31:17	0|sipa|we could add an rpc to report all of addrman's db through json?
 46 2017-05-11 10:47:43	0|wumpus|yes, exactly
 47 2017-05-11 11:03:20	0|kallewoof|Heh. I always thought getpeerinfo did just that.
 48 2017-05-11 11:03:44	0|kallewoof|But I guess addrman can have entries for nodes you aren't currently connected to..
 49 2017-05-11 11:04:41	0|wumpus|right, getpeerinfo shows the peers that the node is currently connected to, not the ones it knows about
 50 2017-05-11 11:29:26	0|jonasschnelli|A simpler approach would be to ensure we directly answer a getaddr request when the debug GetArg("-getaddr_direct") is set
 51 2017-05-11 12:02:42	0|jonasschnelli|The whole address relay is pretty hard to test on localhost. If 127.0.0.1 then IsRoutable() == false, GetLocalAddress() does not allow 127.0.0.1,..
 52 2017-05-11 12:45:50	0|paveljanik|jonasschnelli, can you please rebase #9697 to make it easier for testing? Thank you.
 53 2017-05-11 12:45:52	0|gribble|https://github.com/bitcoin/bitcoin/issues/9697 | [Qt] simple fee bumper with user verification by jonasschnelli · Pull Request #9697 · bitcoin/bitcoin · GitHub
 54 2017-05-11 12:46:42	0|jonasschnelli|paveljanik: I did fresh compile it on Ubuntu 14.10 and didn't had any problems... There is already an ACK on the PR... but I can rebase if it's unavoidable.
 55 2017-05-11 12:47:37	0|paveljanik|jonasschnelli, ok, np.
 56 2017-05-11 12:48:02	0|paveljanik|there are three new commits anyway...
 57 2017-05-11 12:48:13	0|jonasschnelli|Yeah.. right. Let me rebase then.
 58 2017-05-11 12:48:33	0|jonasschnelli|Give me some minutes (working branch is different right now)
 59 2017-05-11 12:49:14	0|paveljanik|It is an issue with my workflow... rebase will make it much easier for me (I'll be able to use the default/automated workflow).
 60 2017-05-11 12:49:23	0|jonasschnelli|sure... will do
 61 2017-05-11 13:27:38	0|jonasschnelli|paveljanik: #9697 is now rebased
 62 2017-05-11 13:27:40	0|gribble|https://github.com/bitcoin/bitcoin/issues/9697 | [Qt] simple fee bumper with user verification by jonasschnelli · Pull Request #9697 · bitcoin/bitcoin · GitHub
 63 2017-05-11 15:09:02	0|bitcoin-git|[13bitcoin] 15jonasschnelli opened pull request #10387: [WIP] Define and signal NODE_NETWORK_LIMITED (pruned peers) (06master...062017/05/node_network_limited) 02https://github.com/bitcoin/bitcoin/pull/10387
 64 2017-05-11 16:56:02	0|bitcoin-git|[13bitcoin] 15laanwj pushed 5 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/7f2b9e0868f5...79aeff6e08f3
 65 2017-05-11 16:56:03	0|bitcoin-git|13bitcoin/06master 149970219 15Matt Corallo: Update contrib/debian to latest Ubuntu PPA upload....
 66 2017-05-11 16:56:03	0|bitcoin-git|13bitcoin/06master 14a8e9286 15Matt Corallo: Bump minimum boost version in contrib/debian
 67 2017-05-11 16:56:04	0|bitcoin-git|13bitcoin/06master 14c5071e1 15Matt Corallo: Build with QT5 on Debian-based systems using contrib/debian
 68 2017-05-11 16:56:36	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #10328: Update contrib/debian to latest Ubuntu PPA upload. (06master...062017-05-update-debian) 02https://github.com/bitcoin/bitcoin/pull/10328
 69 2017-05-11 17:20:04	0|bitcoin-git|13bitcoin/06master 14a637734 15Luke Dashjr: rpc/wallet: Workaround older UniValue which returns a std::string temporary for get_str
 70 2017-05-11 17:20:04	0|bitcoin-git|[13bitcoin] 15laanwj pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/79aeff6e08f3...18c9debe602d
 71 2017-05-11 17:20:05	0|bitcoin-git|13bitcoin/06master 1418c9deb 15Wladimir J. van der Laan: Merge #10341: rpc/wallet: Workaround older UniValue which returns a std::string temporary for get_str...
 72 2017-05-11 17:20:36	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #10341: rpc/wallet: Workaround older UniValue which returns a std::string temporary for get_str (06master...06rpcwallet_uv_workaround) 02https://github.com/bitcoin/bitcoin/pull/10341
 73 2017-05-11 17:27:37	0|bitcoin-git|13bitcoin/06master 140c60c63 15practicalswift: Remove unused Python imports
 74 2017-05-11 17:27:37	0|bitcoin-git|[13bitcoin] 15laanwj pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/18c9debe602d...eb8263bdc9d3
 75 2017-05-11 17:27:38	0|bitcoin-git|13bitcoin/06master 14eb8263b 15Wladimir J. van der Laan: Merge #10317: Remove unused Python imports...
 76 2017-05-11 17:28:06	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #10317: Remove unused Python imports (06master...06remove-unused-python-imports-ii) 02https://github.com/bitcoin/bitcoin/pull/10317
 77 2017-05-11 17:58:06	0|bitcoin-git|13bitcoin/06master 146c914ac 15Thomas Snider: [wallet] Securely erase potentially sensitive keys/values
 78 2017-05-11 17:58:06	0|bitcoin-git|[13bitcoin] 15laanwj pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/eb8263bdc9d3...94e52273f30f
 79 2017-05-11 17:58:07	0|bitcoin-git|13bitcoin/06master 1494e5227 15Wladimir J. van der Laan: Merge #10308: [wallet] Securely erase potentially sensitive keys/values...
 80 2017-05-11 17:58:36	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #10308: [wallet] Securely erase potentially sensitive keys/values (06master...06tjps_secure_erase) 02https://github.com/bitcoin/bitcoin/pull/10308
 81 2017-05-11 18:20:25	0|bitcoin-git|[13bitcoin] 15morcos opened pull request #10388: Output line to debug.log when IsInitialBlockDownload latches to false (06master...06printIBD) 02https://github.com/bitcoin/bitcoin/pull/10388
 82 2017-05-11 18:29:35	0|jonasschnelli|gmaxwell: The idea behind the signaling 7056 blocks in NODE_NETWORK_LIMITED was, that SPV peers and other very irregular network participants can catch up filtered (or non filtered) blocks of about a month.
 83 2017-05-11 18:30:40	0|jonasschnelli|Though the value itself is questionable (I though NODE_NETWORK_LIMITED_HIGH^2 may be resonable).
 84 2017-05-11 18:33:22	0|gmaxwell|I'm pretty doubtful people are going to set pruning to anything but not at all or all the way right now.
 85 2017-05-11 18:38:01	0|jonasschnelli|gmaxwell: I don't know. I personally would consider pruning to 10GB or something that doesn't hurt. But yeah.... I see the point with UTXO syncs and maybe reserve it then for future use cases (though we could also add another bit)
 86 2017-05-11 18:39:36	0|morcos|gmaxwell: given the size requirements for other aspects of the data directory (such as chainstate), it seems to me its not unreasonable to keep several gigs of blocks...  I think if there was a 10GB option, it might be heavily used, especially if it was either recommended or somehow the default
 87 2017-05-11 18:43:03	0|instagibbs|Defaults are obviously quite sticky. In my experience almost no one even knows about pruning.
 88 2017-05-11 18:43:33	0|instagibbs|(so people just shut if off)
 89 2017-05-11 18:44:25	0|jonasschnelli|Also consider the use case (thats getting more and more popular) where people link their SPV wallets with their full node. There a "month of blocks" could make sense...
 90 2017-05-11 18:44:38	0|jonasschnelli|And I'm not saying this is a good use case... :)
 91 2017-05-11 18:47:59	0|jonasschnelli|instagibbs: Some people told me that the prices per GB storage falls after moors law, but I personally feel that it was much simpler to run a full node back in 2013,... and not at least because of the disk requirement.
 92 2017-05-11 18:49:39	0|instagibbs|Most people buying computers aren't optimizing for 100GB+ chain sizes.
 93 2017-05-11 18:50:56	0|wumpus|"Some people told me that the prices per GB storage falls after moors law", indeed, that hasn't been true for quite a while
 94 2017-05-11 18:51:26	0|gmaxwell|The setting based on size thing really doesn't work that well, the two logical settings are unpruned and the minimum.
 95 2017-05-11 18:52:06	0|gmaxwell|so I'm not saying that keeping X or Y is useless, but we need a different interface to make them ever used.
 96 2017-05-11 18:52:41	0|gmaxwell|And the big difference is "can people sync from you" -- p2p spv nodes are almost non-existant these days.
 97 2017-05-11 18:52:43	0|wumpus|there is currently no reason to select anything but the minimum if you set up pruning
 98 2017-05-11 18:53:11	0|wumpus|if it did serve the latter blocks I'd gladly set the pruning to larger, and I've heard similar things from other people, but there's just no reason now
 99 2017-05-11 18:54:19	0|gmaxwell|The context here though is having yet another level.
100 2017-05-11 18:54:55	0|murchandamus|Howdy
101 2017-05-11 18:55:15	0|gmaxwell|I don't think having three levels of just pruning makes a lot of sense (nor do I think it's supported by the fetching data we have), when also there will be proposals for start-from-utxo-assumevalid which will have its own levels of storage.
102 2017-05-11 18:55:34	0|jonasschnelli|Right, the question is if both NODE_NETWORK_LIMITED_(LOW|HIGH) is enabled, should we then use a third, larger value (bit 1: 144, bit 2: 1008, both bits: ?)
103 2017-05-11 18:55:54	0|sipa|we could leave it undefined for now
104 2017-05-11 18:56:04	0|wumpus|on the other hand if there are three possibiilties with those bits, it makes sense to give the combination meaning too
105 2017-05-11 18:56:13	0|wumpus|or leave it explicitly undefined for future expansion,s ure
106 2017-05-11 18:56:28	0|sipa|merely the presence of having server-but-not-full-archival nodes will likely change things already
107 2017-05-11 18:56:31	0|gmaxwell|What I suggested was that it's undefined for setting, but for recieving you know it means at least X blocks, but it may also require other things, so you can't set it yet.
108 2017-05-11 18:56:32	0|sipa|and we'll learn from that
109 2017-05-11 18:56:38	0|jonasschnelli|Yes. It felt after a waste if we don't define it. Well, undefined (for future usecases) is also a definition.
110 2017-05-11 18:56:57	0|wumpus|agree
111 2017-05-11 18:57:00	0|instagibbs|gmaxwell, s/setting/sending/?
112 2017-05-11 18:57:13	0|gmaxwell|instagibbs: both.
113 2017-05-11 18:57:25	0|wumpus|gmaxwell: yes "it means at least..." makes a lot of sense for backwards compatibility
114 2017-05-11 18:58:05	0|gmaxwell|If you see both bits set, then it means they have at least 2016*2 blocks (which is what I suggested). But you don't set it yet, because it may (will) later be defined to say "And I offer a UTXO syncpoint" or whatnot.
115 2017-05-11 18:58:19	0|wumpus|yes
116 2017-05-11 18:58:31	0|gmaxwell|to be extra conservative it could just be defined to mean at least _HIGH  now. I don't have a strong opinion.
117 2017-05-11 18:58:38	0|gmaxwell|you can always increase the requirement later.
118 2017-05-11 19:00:16	0|wumpus|#startmeeting
119 2017-05-11 19:00:17	0|lightningbot|Meeting started Thu May 11 19:00:16 2017 UTC.  The chair is wumpus. Information about MeetBot at http://wiki.debian.org/MeetBot.
120 2017-05-11 19:00:17	0|lightningbot|Useful Commands: #action #agreed #help #info #idea #link #topic.
121 2017-05-11 19:00:26	0|gmaxwell|#bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier
122 2017-05-11 19:00:32	0|sipa|present
123 2017-05-11 19:00:40	0|murchandamus|present
124 2017-05-11 19:00:44	0|wumpus|topics?
125 2017-05-11 19:00:49	0|sipa|murchandamus == murch?
126 2017-05-11 19:00:51	0|cfields|hi, here
127 2017-05-11 19:00:54	0|instagibbs|yes sipa
128 2017-05-11 19:00:54	0|murchandamus|aye!
129 2017-05-11 19:00:56	0|morcos|gmaxwell: oh ok, i agree with not defining now..  maybe we should make just _HIGH more then though
130 2017-05-11 19:00:59	0|kanzure|hi.
131 2017-05-11 19:01:36	0|gmaxwell|morcos: yes, I was thinking HIGH would be targeted at hosts syncing 2016 blocks, but I forget where the breakpoints were exactly in sipas' data.
132 2017-05-11 19:01:40	0|BlueMatt|damnit, can enver remember topics i wanted to bring up come meeting time :(
133 2017-05-11 19:01:40	0|wumpus|any proposed topics? (we can continue the pruning service bits topic if people want that)
134 2017-05-11 19:01:57	0|luke-jr|O.o?
135 2017-05-11 19:02:18	0|gmaxwell|wumpus: well we should talk about per-txo. I think it's ready for merge except for more testing/review.
136 2017-05-11 19:02:26	0|instagibbs|suggested topic: fee targeting/coin selection overhaul
137 2017-05-11 19:02:35	0|wumpus|#topic per-txo utxo database
138 2017-05-11 19:02:52	0|sipa|#10148
139 2017-05-11 19:02:55	0|gribble|https://github.com/bitcoin/bitcoin/issues/10148 | [WIP] Use non-atomic flushing with block replay by sipa · Pull Request #10148 · bitcoin/bitcoin · GitHub
140 2017-05-11 19:02:56	0|gmaxwell|if people haven't seen the graphs: https://cloud.githubusercontent.com/assets/548488/25769030/c84fe65e-31c4-11e7-8819-264c44e50ddf.png
141 2017-05-11 19:03:02	0|sipa|oops, no, #10195
142 2017-05-11 19:03:04	0|gribble|https://github.com/bitcoin/bitcoin/issues/10195 | Switch chainstate db and cache to per-txout model by sipa · Pull Request #10195 · bitcoin/bitcoin · GitHub
143 2017-05-11 19:03:13	0|BlueMatt|I'm still halfway through review
144 2017-05-11 19:03:13	0|morcos|sorry sipa, that has been on my list, but my list has been gathering dust for the last couple weeks
145 2017-05-11 19:03:16	0|gmaxwell|The graphs should be making all your mouths water.
146 2017-05-11 19:03:30	0|instagibbs|why is y-axis in block time :P
147 2017-05-11 19:03:34	0|wumpus|ah yes I was still testing #10148, should probably switch to #10195
148 2017-05-11 19:03:37	0|gribble|https://github.com/bitcoin/bitcoin/issues/10148 | [WIP] Use non-atomic flushing with block replay by sipa · Pull Request #10148 · bitcoin/bitcoin · GitHub
149 2017-05-11 19:03:39	0|gribble|https://github.com/bitcoin/bitcoin/issues/10195 | Switch chainstate db and cache to per-txout model by sipa · Pull Request #10195 · bitcoin/bitcoin · GitHub
150 2017-05-11 19:03:39	0|instagibbs|clock*
151 2017-05-11 19:03:41	0|gmaxwell|instagibbs: so that the flushing graph works out.
152 2017-05-11 19:03:52	0|sipa|instagibbs: so that the x axis of both graphs lines up
153 2017-05-11 19:04:00	0|cfields|i've made it through review, but I lack enough confidence to ACK the thing :\
154 2017-05-11 19:04:04	0|BlueMatt|gm2051: #10192
155 2017-05-11 19:04:05	0|gmaxwell|The most impressive thing about that chart isn't the ~33% speedup, it's the reduction in flushing frequency.
156 2017-05-11 19:04:09	0|gribble|https://github.com/bitcoin/bitcoin/issues/10192 | Cache full script execution results in addition to signatures by TheBlueMatt · Pull Request #10192 · bitcoin/bitcoin · GitHub
157 2017-05-11 19:04:17	0|BlueMatt|gmaxwell: ^
158 2017-05-11 19:04:20	0|cfields|gmaxwell: yes, that's great.
159 2017-05-11 19:04:34	0|sipa|i was very surprised by how much it reduces flushing
160 2017-05-11 19:04:50	0|wumpus|nice chart
161 2017-05-11 19:04:54	0|sipa|my guess is that the resulting speedup isn't that dramatic because it's running on a system with pretty fast I/O
162 2017-05-11 19:05:12	0|wumpus|heh, but most users have systems with slow i/o
163 2017-05-11 19:05:27	0|sipa|oh, one downside: chainstate db goes from 2.2G to 2.7G
164 2017-05-11 19:05:30	0|gmaxwell|right, testing it on a system with slow i/o would be interesting. But will take forever.
165 2017-05-11 19:05:45	0|BlueMatt|2.7 seems fine
166 2017-05-11 19:06:10	0|luke-jr|hmm
167 2017-05-11 19:06:31	0|sipa|the TODOs that i know of are a few code cleanups (marked as TODO in the code), and better UI wrt the upgrade process
168 2017-05-11 19:06:45	0|sipa|there is a one-time upgrade of the old db to the new db at startup, which can be interrupted
169 2017-05-11 19:06:55	0|gmaxwell|The upgrade doesn't take long on a system with fast IO at least.
170 2017-05-11 19:06:57	0|BlueMatt|sipa: it needs way more review
171 2017-05-11 19:07:02	0|sipa|BlueMatt: of course
172 2017-05-11 19:07:03	0|BlueMatt|there are lots of things in that queue, sadly
173 2017-05-11 19:07:10	0|wumpus|upgrade should be fast if it just iterates through the db in sorted order
174 2017-05-11 19:07:18	0|wumpus|even on systems with fairly slow (seek) i/o
175 2017-05-11 19:07:28	0|sipa|wumpus: it does, it takes a couple of minutes on a system with fast I/O
176 2017-05-11 19:08:02	0|wumpus|I'll try it out this week
177 2017-05-11 19:08:36	0|gmaxwell|One thing to keep in mind is that keeping it unmerged reduces testing. Absolutely it needs more review before being merged, but we also should try to get it merged sooner rather than later.
178 2017-05-11 19:08:46	0|gmaxwell|So that we get more time baking on it in master.
179 2017-05-11 19:09:00	0|BlueMatt|gmaxwell: we're not even close to that point
180 2017-05-11 19:09:12	0|BlueMatt|but, yes, we should be agressive about merge, as long as its a ways before release
181 2017-05-11 19:09:14	0|morcos|agreed gmaxwell, same thing with #10199
182 2017-05-11 19:09:16	0|gribble|https://github.com/bitcoin/bitcoin/issues/10199 | Better fee estimates by morcos · Pull Request #10199 · bitcoin/bitcoin · GitHub
183 2017-05-11 19:09:28	0|sipa|morcos: i'll do another review pass on that today
184 2017-05-11 19:09:41	0|wumpus|yes, this shouldn't be something to merge last minute for 0.15
185 2017-05-11 19:09:44	0|morcos|good thing about that one, is its a lot less likely to casue a disaster
186 2017-05-11 19:10:13	0|gmaxwell|I strongly disagree.
187 2017-05-11 19:10:44	0|wumpus|they're both important for different reasons
188 2017-05-11 19:10:56	0|wumpus|it's comparing apples and oranges
189 2017-05-11 19:11:09	0|BlueMatt|anyway, do we have any real topics?
190 2017-05-11 19:11:29	0|sipa|i have one low-priority idea i'd like to talk about (running utxo commitments)
191 2017-05-11 19:11:37	0|BlueMatt|sipa: shoot
192 2017-05-11 19:11:46	0|gmaxwell|BlueMatt: it's beautiful here in mountain view today.
193 2017-05-11 19:11:46	0|wumpus|#topic fee targeting/coin selection overhaul (instagibbs)
194 2017-05-11 19:12:07	0|sipa|instagibbs: topic
195 2017-05-11 19:12:17	0|murchandamus|gmaxwell: A bit cloudy here in Palo Alto though. ;)
196 2017-05-11 19:12:24	0|instagibbs|So I tried my hand at doing redoing fee targeting #10360 without directly changing coin selection
197 2017-05-11 19:12:25	0|gribble|https://github.com/bitcoin/bitcoin/issues/10360 | [WIP] [Wallet] Target effective value during transaction creation by instagibbs · Pull Request #10360 · bitcoin/bitcoin · GitHub
198 2017-05-11 19:12:36	0|instagibbs|Was wondering if people were wary of that, versus a complete overhaul
199 2017-05-11 19:12:44	0|instagibbs|morcos has concerns
200 2017-05-11 19:12:49	0|morcos|wary is a good word for it
201 2017-05-11 19:12:55	0|morcos|not opposed
202 2017-05-11 19:13:02	0|gmaxwell|I feel a little uneasy about the changes to augment selection while we don't have a strategy to sweep dust. It worries me that we're potentially going to again untintentionally create another UTXO count blowup event.
203 2017-05-11 19:13:12	0|sipa|instagibbs: i haven't looked into the details... is it just treating the net value of its output as amount - feerate*size_of_spend ?
204 2017-05-11 19:13:15	0|murchandamus|I've talked a bit with instagibbs about it and it seems to me that when that approach finds a direct match it may have a huge number of inputs
205 2017-05-11 19:13:20	0|instagibbs|sipa yes
206 2017-05-11 19:13:36	0|gmaxwell|murchandamus: that sounds great to me. :P
207 2017-05-11 19:13:43	0|murchandamus|I've combined a similar approach but trying to combine inputs by size large to small
208 2017-05-11 19:14:05	0|instagibbs|sipa, so you may get more smaller inputs that directly match, for example
209 2017-05-11 19:14:13	0|murchandamus|gmaxwell: I think that slightly bigger average transaction with lower variance in transaction size are better than huge variance in transaction size
210 2017-05-11 19:14:39	0|morcos|i like the idea of being at least somewhat fee smart, and including more inputs when fees are lower
211 2017-05-11 19:14:41	0|sipa|but even if it spends many inputs, it always does so in a way that is economical
212 2017-05-11 19:14:57	0|instagibbs|sipa, it will always use "positive effective value" inputs
213 2017-05-11 19:14:59	0|sipa|so unless you assume that the feerate is going down in the future, there is little reason to postpone spending them
214 2017-05-11 19:15:05	0|murchandamus|morcos: Exactly. We have huge variance in fees over the week
215 2017-05-11 19:15:10	0|gmaxwell|murchandamus: high to low sounds like it would destroy privacy.
216 2017-05-11 19:15:10	0|morcos|the idea of wanting to do a quick transaction and paying 200 sat/byte to do it, and you include tons of little inputs that have little to none net value seems bad to me
217 2017-05-11 19:15:46	0|murchandamus|sipa: But adding a single input costs as much as four outputs, so you'd probably rather add a change output when fees are high than add another input, and later in the week do a few consolidation transactions
218 2017-05-11 19:16:05	0|murchandamus|I've been hearing that some people do this to save money
219 2017-05-11 19:16:08	0|gmaxwell|morcos: I though about that and I think a nice estimate would be the ratio between the current feerate set and the BIG_NUM target (e.g. 1008 block target).  If this ratio is low, then you should be agressive in spending.
220 2017-05-11 19:16:12	0|morcos|admittedly its a difficult problem...  how do you distinguish a user that only does 200 sat/byte txs vs one that has a range
221 2017-05-11 19:16:32	0|murchandamus|gmaxwell: I don't think it does, because it's only the selection for making exact matches
222 2017-05-11 19:16:43	0|morcos|gmaxwell: yes, i agree... something like that
223 2017-05-11 19:16:46	0|murchandamus|and you don't know if the first combination that matched was actually among the largest in your wallet
224 2017-05-11 19:17:22	0|morcos|anyway, clearly there are a lot of ideas here, and i kidn of think that the amount of consideration that has to go into most changes is almost the same, and it might not be worthing doing all of that sanity checking if we're only making a small change
225 2017-05-11 19:17:24	0|instagibbs|yes, so there are plenty of interesting strategies, the question is should we not be making semi-obvious fixing until we can agree on those :)
226 2017-05-11 19:17:35	0|murchandamus|gmaxwell: interesting approach
227 2017-05-11 19:18:22	0|instagibbs|related: can we kill minimum total fee?
228 2017-05-11 19:18:22	0|morcos|at the very least, i'd think this should be lower priority for 0.15 than the prevously mentioned things right now... so we have to think about resources
229 2017-05-11 19:18:33	0|morcos|instagibbs: +1 on that for a first step
230 2017-05-11 19:18:37	0|sipa|instagibbs: ack
231 2017-05-11 19:18:39	0|instagibbs|morcos, yes that was the feeling I get
232 2017-05-11 19:18:41	0|murchandamus|instagibbs: Accounting for UTXO in selection by effective value does take a load of pain out of selection strategies
233 2017-05-11 19:19:21	0|gmaxwell|instagibbs: but if a 'semi obvious fix' blows up other considerations thats bad.  Do you have a reason to believe your change won't cause a massive increase in utxo accumulation?
234 2017-05-11 19:19:33	0|morcos|also i have a PR #9343 that also removes edge case logic.  cleaning these things up now will make future improvements easier to do and reason about
235 2017-05-11 19:19:35	0|gribble|https://github.com/bitcoin/bitcoin/issues/9343 | Dont create change at dust limit by morcos · Pull Request #9343 · bitcoin/bitcoin · GitHub
236 2017-05-11 19:19:38	0|murchandamus|instagibbs: I'm not sure that just this change with the current selection strategy of Core does much better in the selection, because of the selecting/deselecting pass
237 2017-05-11 19:19:44	0|instagibbs|gmaxwell, no, I wasn't assuming as much, sorry if that seemed implied
238 2017-05-11 19:20:09	0|murchandamus|gmaxwell: Pretty sure it would rather decrease UTXO footprints
239 2017-05-11 19:20:36	0|murchandamus|right now Core will often fail in the first few passes when it selects more inputs because it hadn't accounted for the fees in advance
240 2017-05-11 19:20:38	0|sipa|instagibbs: which feerate are you using?
241 2017-05-11 19:20:57	0|instagibbs|sipa, whatever the user has selected via settings
242 2017-05-11 19:21:01	0|gmaxwell|instagibbs: well, thats why I ask. basically, in any mature system you can have 'bugs' which you depend on. at first glance it semeed to me that your PR might fix a bug where we're overly agressively spending TXO that are negative value. But we may depend on that bug to help manage the UTXO size.  At least that was my impression.
243 2017-05-11 19:21:05	0|murchandamus|with this change it would always succeed as soon as it finds an exact match
244 2017-05-11 19:21:09	0|sipa|instagibbs: oh, duh
245 2017-05-11 19:21:10	0|instagibbs|well, aside from total minimum fee, because that's stupid
246 2017-05-11 19:21:55	0|instagibbs|gmaxwell, could be. Maybe next step, aside from general refactor and cleaning, is to get better data
247 2017-05-11 19:22:09	0|gmaxwell|murchandamus: I'm not following your logic. Yes, the first few passes will fail, then it will target a higher amount, and be successful.
248 2017-05-11 19:22:18	0|murchandamus|gmaxwell: Yes, it shoudl never select UTXO that are negative effective value
249 2017-05-11 19:22:22	0|instagibbs|and stop throwing away money as fees egregiously: #10333
250 2017-05-11 19:22:23	0|gribble|https://github.com/bitcoin/bitcoin/issues/10333 | [wallet] fee fixes: always create change, adjust value, and p… by instagibbs · Pull Request #10333 · bitcoin/bitcoin · GitHub
251 2017-05-11 19:22:35	0|gmaxwell|murchandamus: By "should" do you mean the current behavior?
252 2017-05-11 19:22:44	0|instagibbs|current behavior almost certainly does
253 2017-05-11 19:22:49	0|gmaxwell|The current behavior absolutely will select txos with negative value.
254 2017-05-11 19:23:03	0|sipa|So how about we fix that first?
255 2017-05-11 19:23:06	0|gmaxwell|And "fixing" that may have severely deletarious effects on the network.
256 2017-05-11 19:23:08	0|murchandamus|gmaxwell: It only fails if it doesn't find a transaction input set within the number of estimated inputs from the previous tries
257 2017-05-11 19:23:11	0|morcos|sipa: yeah i was thinking that
258 2017-05-11 19:23:12	0|instagibbs|I could always just make it ignore negative effective value
259 2017-05-11 19:23:13	0|instagibbs|...
260 2017-05-11 19:23:13	0|morcos|gmaxwell: disagree
261 2017-05-11 19:23:15	0|instagibbs|lol
262 2017-05-11 19:23:38	0|instagibbs|I mean either it's a feature, and we shouldnt fix it, or a bug and we should
263 2017-05-11 19:23:46	0|morcos|instagibbs: yeah exactly
264 2017-05-11 19:24:09	0|gmaxwell|It can be both! :)
265 2017-05-11 19:24:18	0|murchandamus|gmaxwell: I meant "should" as in when you calculate the effective value of a UTXO, if it is a negative effective value it shouldn't be selected
266 2017-05-11 19:24:28	0|instagibbs|aspirational should?
267 2017-05-11 19:24:29	0|sdaftuar|so there's a difference here between generally factoring in feerates in the coin selection, and throwing out inputs that have negative value.  i assume that's what we're getting at?
268 2017-05-11 19:24:33	0|murchandamus|gmaxwell: actually yes
269 2017-05-11 19:24:37	0|gmaxwell|It's a feature when its mild and happening during times of low feerate. And a bug when its severe when it is insane and happening during high feerate.
270 2017-05-11 19:24:42	0|murchandamus|current behavior would select them
271 2017-05-11 19:24:48	0|instagibbs|gmaxwell, fair enough...
272 2017-05-11 19:25:00	0|wumpus|yes, eating utxos with negative value cleans up the utxo set
273 2017-05-11 19:25:06	0|morcos|gmaxwell: right now when feerates vary from 10 to 200 sat/byte.   something that is 0 at 200 sat/byte should be cleaned up at a lower fee rate, not thrown away
274 2017-05-11 19:25:27	0|gmaxwell|Fixing it unconditionally without doing something about dust cleanup may be quite harmful to the network.
275 2017-05-11 19:25:30	0|wumpus|of course it's better to not create them in the first place
276 2017-05-11 19:25:43	0|morcos|wumpus: yes!  see #9343
277 2017-05-11 19:25:44	0|gribble|https://github.com/bitcoin/bitcoin/issues/9343 | Dont create change at dust limit by morcos · Pull Request #9343 · bitcoin/bitcoin · GitHub
278 2017-05-11 19:25:57	0|murchandamus|morcos: That's what I meant
279 2017-05-11 19:25:58	0|instagibbs|^I should review that one
280 2017-05-11 19:26:00	0|morcos|i'm not sure thats the only way they are created, perhaps we could do more to avoid creating them also
281 2017-05-11 19:26:12	0|gmaxwell|morcos: well they're created by people being buttheads.
282 2017-05-11 19:26:15	0|morcos|i should say, i'm sure thats not the only way
283 2017-05-11 19:26:21	0|instagibbs|I noticed in current logic we create near-dust, but when we're modifying change, we have much higher bar to clear
284 2017-05-11 19:26:23	0|gmaxwell|and no amount of fixing the wallet will prevent their creation.
285 2017-05-11 19:26:24	0|instagibbs|we should sync this
286 2017-05-11 19:26:37	0|wumpus|yes, we should get that one in
287 2017-05-11 19:26:49	0|wumpus|gmaxwell: well not creating them ourselves is not a total solution, but it helps
288 2017-05-11 19:27:02	0|gmaxwell|wumpus: absolutely, sorry if it sounded like I said otherwise.
289 2017-05-11 19:27:07	0|morcos|gmaxwell: yeah its a good question how many are created unintentionally vs intentionally
290 2017-05-11 19:27:47	0|morcos|it wouldn't be unreasonable to raise the limit for intentionally creating them in Core, in adddition to imporving unintentional behavior
291 2017-05-11 19:27:53	0|gmaxwell|and at least as far as the "some anonymous third party sends you a few bitcents"  I think it's fine to spend those at a slight loss, esp if its in a privacy preserving way, and double especially if it's at as low a fee rate as you expect to see.
292 2017-05-11 19:27:59	0|murchandamus|morcos: Core never creates change outputs smaller than 0.01 BTC unless the wallet is being almost depleted with the transaction, right?
293 2017-05-11 19:28:22	0|morcos|murchandamus: well.. thats the design goal, i don't think its achieved
294 2017-05-11 19:28:29	0|instagibbs|it's not achieved at all
295 2017-05-11 19:28:30	0|gmaxwell|what morcos said
296 2017-05-11 19:28:30	0|instagibbs|sad!
297 2017-05-11 19:28:45	0|gmaxwell|well it's better since 0.13.whatever when we fixed some things.
298 2017-05-11 19:28:53	0|gmaxwell|with the target/2 checks.
299 2017-05-11 19:29:04	0|murchandamus|I'll have to check out the linked issue
300 2017-05-11 19:29:11	0|morcos|in particular, if you have nTotalLower < target + CENT, you aim for target, and almost definitely create some stupidly small change
301 2017-05-11 19:29:26	0|morcos|among other posibilities
302 2017-05-11 19:29:37	0|murchandamus|morcos: Ah right, I forgot about the pre-selection before knapsack
303 2017-05-11 19:29:39	0|morcos|the linked issue is even more edge case
304 2017-05-11 19:29:41	0|murchandamus|thanks for the reminder
305 2017-05-11 19:30:07	0|morcos|but like nTotalFee needs to go away before its easy to clean up the general case
306 2017-05-11 19:30:22	0|morcos|easIER
307 2017-05-11 19:30:41	0|murchandamus|morcos: I think the way to go would be to dissect and modularize the coin selection out of wallet.cpp, if that's possible
308 2017-05-11 19:30:44	0|murchandamus|right now it is such a moloch
309 2017-05-11 19:31:06	0|instagibbs|I mean you can basically just throw away anything at or below SelectCoins, imo
310 2017-05-11 19:31:10	0|wumpus|murchandamus: yes please
311 2017-05-11 19:31:27	0|gmaxwell|Moloch whose mind is pure machinery!
312 2017-05-11 19:32:16	0|murchandamus|I think instagibbs and I might coordinate something there, and jnewberry was also interested AFAIK
313 2017-05-11 19:32:21	0|gmaxwell|can we new-subject?  good discussion on this, lots of PRs for people to look at and discuss more. :)
314 2017-05-11 19:32:28	0|instagibbs|morcos is also interested
315 2017-05-11 19:32:34	0|instagibbs|yes I'm satisfied, we can continue offline
316 2017-05-11 19:32:46	0|murchandamus|thanks, me too
317 2017-05-11 19:32:49	0|wumpus|#topic running utxo commitments (sipa)
318 2017-05-11 19:32:57	0|sipa|ok
319 2017-05-11 19:32:58	0|gmaxwell|instagibbs: in any case, that one concern: that we might 'fix' what is dedusting the UTXO is the only reason I didn't ACK your patch. So we should try to get confidence there or add some other fix for that.
320 2017-05-11 19:33:13	0|instagibbs|gmaxwell, my firster iteration didnt even "fix" it :P
321 2017-05-11 19:33:20	0|sipa|so, gmaxwell and i have been thinking about the possibility of maintaining a UTXO commitment hash all the time
322 2017-05-11 19:34:03	0|sipa|this would be useful for making gettxoutsetinfo instantaneous, or for syncing from someone else's UTXO set, or as the basis for a softfork later
323 2017-05-11 19:34:25	0|wumpus|yes, that would be useful
324 2017-05-11 19:34:36	0|sipa|and it seems it would be possible to have an implementation that does this at a cost of a few microseconds per input and per output
325 2017-05-11 19:34:42	0|gmaxwell|A first requirement for any kind of UTXO assumevalid is having a continuous commitment value to simplify review.
326 2017-05-11 19:34:45	0|sipa|in a cryptographically secure way
327 2017-05-11 19:35:33	0|wumpus|jonasschnelli: that's pretty much just "a better GUI for coin control" right?
328 2017-05-11 19:35:40	0|gmaxwell|Effectively, we construct an incremental unordered hash for sets-- so you can add and remove entries one at a time... Because just one scheme isn't enough, we actually constructed several, and now have a fun tradeoffs challenge to decide between them.
329 2017-05-11 19:35:41	0|sipa|there are a few different possible implementations (one is based on multiplying big numbers mod a big prime, one is based on EC math, one is based on adding large hashes toether)... with different performance and security tradeoffs, i'll send a mail about it to the ML soon
330 2017-05-11 19:36:00	0|BlueMatt|ah, ok, was gonna ask what the design was, cool
331 2017-05-11 19:36:07	0|instagibbs|i love the alternative explanations...
332 2017-05-11 19:36:19	0|BlueMatt|i mean we can also not use complicated solutions and be willing to do it in the background at a cost of however many milliseconds later
333 2017-05-11 19:36:29	0|BlueMatt|dunno how complicated your proposal is, of course
334 2017-05-11 19:36:59	0|sipa|BlueMatt: map every UTXO to either a 3072-bit integer and multiply those for all outputs, or to an EC point and all those for all outputs
335 2017-05-11 19:37:14	0|sipa|the multiplication approach is faster, but harder to cache
336 2017-05-11 19:37:18	0|gmaxwell|The strength of our proposals is strictly better than the discrete log assumption.  Neighter are especially complex, though the multiplying one is probably simpler for joe-fool to implement as long as they don't care about performance.
337 2017-05-11 19:37:34	0|sipa|the EC approach could allow us to cache the effect of a single transaction, and then instantly apply it to the running commitment
338 2017-05-11 19:38:20	0|sipa|hahaha
339 2017-05-11 19:38:28	0|wumpus|interesting proposal
340 2017-05-11 19:38:56	0|sipa|so, even though 5us per input/output may not seem much, it's several hours of CPU time for a node syncing from scratch
341 2017-05-11 19:39:04	0|BlueMatt|sipa: hmm, i assume you have a pointer to a crypto accumulator somewhere?
342 2017-05-11 19:39:13	0|BlueMatt|paper*
343 2017-05-11 19:39:22	0|BlueMatt|ehh, I'll just wait for your ml post
344 2017-05-11 19:39:28	0|sipa|BlueMatt: it's a really uninteresting accumulator, as it can't be used to prove anything
345 2017-05-11 19:39:47	0|sipa|i'll include some references
346 2017-05-11 19:39:58	0|gmaxwell|BlueMatt: there have been several papers on related schemes-- however.  Of course, the papers ignore the performance considerations.  And especially in our case we have tradeoffs around block validation latency.
347 2017-05-11 19:40:38	0|instagibbs|I still want compact proofs, get back to work.
348 2017-05-11 19:40:48	0|instagibbs|(ducks)
349 2017-05-11 19:40:48	0|sipa|also, the DL approach would mean we need an implementation of a fast multiplication mod a (fixed) prime , or a GMP dependency
350 2017-05-11 19:41:14	0|sipa|instagibbs: so one advantage that these things have is that they're not incompatible with other UTXO/TXO commitment approaches
351 2017-05-11 19:41:16	0|gmaxwell|in any case, ML post will talk about the tradeoffs and schemes.
352 2017-05-11 19:41:17	0|cfields|is ordering relevant? any effects on parallel validation/caching?
353 2017-05-11 19:41:25	0|sipa|cfields: it's 100% parallellizable
354 2017-05-11 19:41:30	0|gmaxwell|cfields: they're totally independant of ordering.
355 2017-05-11 19:41:32	0|wumpus|if at least it can be done in parallel with some of the i/o  (e.g. database lookups) it wouldn't have to add that much to the total validation time
356 2017-05-11 19:41:34	0|cfields|whew
357 2017-05-11 19:41:38	0|instagibbs|sipa, ack
358 2017-05-11 19:41:54	0|gmaxwell|And as sipa is pointing out, unlike other utxo commitments they do not break STXO like schemes where you have nodes that don't store the whole utxo set.
359 2017-05-11 19:41:54	0|sipa|ordering is irrelevant... it's effectively a set commitment that is homomorphic wrt to set union and subtraction
360 2017-05-11 19:43:17	0|BlueMatt|gmaxwell: well the validation latency tradeoffs are mostly removed by commiting to the previous block's utxo commitment, no?
361 2017-05-11 19:43:17	0|sipa|but if we have this, we could for example log the UTXO commitment hash in the UpdateTip debug.log lines
362 2017-05-11 19:43:25	0|BlueMatt|is there some reason we should avoid doing so?
363 2017-05-11 19:43:54	0|gmaxwell|So the applications I see for this are: an instant gettxouset info,  being able to have updatetip log the UTXO state (making checking nodes much easier), a start on an ability to do an ASSUMEVALID UTXO sync. ('start on' because there is a security/philosophy debate if the value must also be commited to the chain if we're to do an assumevalid like sync)
364 2017-05-11 19:43:55	0|sipa|BlueMatt: well we're not even talking about commiting to it in blocks (though that is an interesting possibility later)
365 2017-05-11 19:44:21	0|BlueMatt|sipa: great! so lets not do it inline and do it in the background later
366 2017-05-11 19:44:33	0|sipa|BlueMatt: and delayed commitments are more complicated if you actually want the latency reduction... you need a backlog and background processing
367 2017-05-11 19:44:34	0|BlueMatt|as long as its under 100ms rpc is still instant
368 2017-05-11 19:44:34	0|BlueMatt|ish
369 2017-05-11 19:44:54	0|gmaxwell|oh well none of this is anywhere near that slow in any case.
370 2017-05-11 19:44:55	0|sipa|which would interfere with CPU demand for signature validation
371 2017-05-11 19:45:25	0|sipa|gmaxwell: if we need an mod inverse in the RPC, it could be 10ms or so with a naive implementation
372 2017-05-11 19:45:30	0|gmaxwell|I think with sutiable caching we're talking about worst case impact, if done inline, on the order of 10ms.
373 2017-05-11 19:45:52	0|sipa|anyway, not that much more to say about it
374 2017-05-11 19:45:55	0|BlueMatt|i think we can live with 10ms :p
375 2017-05-11 19:46:02	0|BlueMatt|anyway, looking forward to the ml post :)
376 2017-05-11 19:46:09	0|wumpus|#topic PRs high priority for review
377 2017-05-11 19:46:10	0|sipa|i just wanted to give a heads up
378 2017-05-11 19:46:14	0|BlueMatt|(in an rpc that would otherwise block cs_main...)
379 2017-05-11 19:46:41	0|sipa|BlueMatt: that 10ms can even run without cs_main
380 2017-05-11 19:46:43	0|instagibbs|a less-extreme wallet PR for consideration to 0.15: #10333
381 2017-05-11 19:46:44	0|gribble|https://github.com/bitcoin/bitcoin/issues/10333 | [wallet] fee fixes: always create change, adjust value, and p… by instagibbs · Pull Request #10333 · bitcoin/bitcoin · GitHub
382 2017-05-11 19:47:02	0|instagibbs|there are still n > 1 reports of extremely high feerates with large num inputs
383 2017-05-11 19:47:03	0|BlueMatt|sipa: thats my point :p
384 2017-05-11 19:47:19	0|instagibbs|they seem to match up with this case
385 2017-05-11 19:48:16	0|wumpus|instagibbs: added 0.15 tag
386 2017-05-11 19:48:26	0|instagibbs|thanks
387 2017-05-11 19:48:35	0|gmaxwell|Where are we with multiwallet?
388 2017-05-11 19:49:05	0|wumpus|there were plenty of review comments on luke-jr's pull, but he hasn't addressed them yet AFAIK
389 2017-05-11 19:49:08	0|gmaxwell|I haven't been following it closely because I've been more focused on per-txo/the above commitment stuff/etc.
390 2017-05-11 19:49:17	0|gmaxwell|luke-jr: ^ plz.
391 2017-05-11 19:49:19	0|jonasschnelli|#8694 needs still rebase
392 2017-05-11 19:49:21	0|gribble|https://github.com/bitcoin/bitcoin/issues/8694 | Basic multiwallet support by luke-jr · Pull Request #8694 · bitcoin/bitcoin · GitHub
393 2017-05-11 19:49:55	0|luke-jr|wumpus: it's pending on jtimon's PR
394 2017-05-11 19:50:05	0|wumpus|luke-jr: which one?
395 2017-05-11 19:50:16	0|wumpus|I merged a few jtimon PRs this week
396 2017-05-11 19:50:21	0|luke-jr|#9494
397 2017-05-11 19:50:23	0|gribble|https://github.com/bitcoin/bitcoin/issues/9494 | Introduce an ArgsManager class encapsulating cs_args, mapArgs and mapMultiArgs by jtimon · Pull Request #9494 · bitcoin/bitcoin · GitHub
398 2017-05-11 19:50:43	0|luke-jr|which actually looks mergable?
399 2017-05-11 19:50:49	0|wumpus|ok, seems that one is already in high priority for review
400 2017-05-11 19:50:56	0|sipa|wumpus: yeah, i marked it so last week
401 2017-05-11 19:51:16	0|luke-jr|reason being 8694 touches mapMultiArgs
402 2017-05-11 19:51:25	0|wumpus|luke-jr: good to know, will take a look at it soon and merge it
403 2017-05-11 19:51:29	0|luke-jr|rather than avoid that, it seems better to just fix the locking for ti
404 2017-05-11 19:51:53	0|wumpus|yes
405 2017-05-11 19:52:02	0|wumpus|sipa: thanks
406 2017-05-11 19:52:46	0|jonasschnelli|Also, consider reviewing HD-Auto-Restore #10240 (it's currently not in high prio), we should have this in 0.15, otherwise users need to do loop10000(getnewaddress), rescan(genesis) to restore funds.
407 2017-05-11 19:52:48	0|gribble|https://github.com/bitcoin/bitcoin/issues/10240 | Add HD wallet auto-restore functionality by jonasschnelli · Pull Request #10240 · bitcoin/bitcoin · GitHub
408 2017-05-11 19:53:51	0|wumpus|jonasschnelli: added 0.15 milestone
409 2017-05-11 19:53:56	0|gmaxwell|jonasschnelli: I'd missed that you got that going. good to hear.
410 2017-05-11 19:54:10	0|jonasschnelli|I'll rebase and fix the points soon.
411 2017-05-11 19:54:45	0|gmaxwell|I'll take a look at it after I finish my code review on some other PRs that I'm currently working on.
412 2017-05-11 19:55:18	0|jonasschnelli|thanks gmaxwell
413 2017-05-11 19:57:05	0|instagibbs|3 minutes
414 2017-05-11 19:57:39	0|sipa|my #1 is still #10195 :)
415 2017-05-11 19:57:41	0|gribble|https://github.com/bitcoin/bitcoin/issues/1 | JSON-RPC support for mobile devices ("ultra-lightweight" clients) · Issue #1 · bitcoin/bitcoin · GitHub
416 2017-05-11 19:57:42	0|gribble|https://github.com/bitcoin/bitcoin/issues/10195 | Switch chainstate db and cache to per-txout model by sipa · Pull Request #10195 · bitcoin/bitcoin · GitHub
417 2017-05-11 19:57:43	0|sipa|eh
418 2017-05-11 19:57:46	0|sipa|ah!
419 2017-05-11 19:57:55	0|wumpus|yep
420 2017-05-11 19:58:15	0|lightningbot|Log:            http://www.erisian.com.au/meetbot/bitcoin-core-dev/2017/bitcoin-core-dev.2017-05-11-19.00.log.html
421 2017-05-11 19:58:15	0|lightningbot|Meeting ended Thu May 11 19:58:15 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
422 2017-05-11 19:58:15	0|lightningbot|Minutes:        http://www.erisian.com.au/meetbot/bitcoin-core-dev/2017/bitcoin-core-dev.2017-05-11-19.00.html
423 2017-05-11 19:58:15	0|lightningbot|Minutes (text): http://www.erisian.com.au/meetbot/bitcoin-core-dev/2017/bitcoin-core-dev.2017-05-11-19.00.txt
424 2017-05-11 19:58:15	0|wumpus|#endmeeting
425 2017-05-11 19:58:23	0|jonasschnelli|wumpus: The 3D graphical coin selection was a joke.. though, for educational use, it would be great
426 2017-05-11 19:58:43	0|wumpus|jonasschnelli: yes I like the idea, I'd probably use it, I currently always use manual coin selection already
427 2017-05-11 19:58:46	0|jonasschnelli|You have a wallet, then you 3d-pick some coins... and you get back a change into your wallet
428 2017-05-11 19:59:02	0|jonasschnelli|Yes. I personally always do manual coin selection.
429 2017-05-11 19:59:15	0|jonasschnelli|But not sure about the majority of users. :)
430 2017-05-11 19:59:29	0|wumpus|jonasschnelli: I think it was in Milan that someone showed some wallet app that had 2d physics effects with coins
431 2017-05-11 19:59:31	0|instagibbs|I've never done coin selection personally
432 2017-05-11 19:59:32	0|luke-jr|2D would be enough
433 2017-05-11 19:59:57	0|wumpus|don't remember which one, but it looked quite funny
434 2017-05-11 19:59:58	0|jonasschnelli|luke-jr: Today everything needs to be 3D,... even 2D. :)
435 2017-05-11 20:00:15	0|gmaxwell|2d is probably better, 3d illustrations often confuse people about volumes.  But if you show more valuable coins as bigger that will create a false impression that bigger coins cost more to spend, no?
436 2017-05-11 20:00:18	0|instagibbs|simply tossing current coin selection and adding random selection would probably be a huge plus sadly.
437 2017-05-11 20:00:21	0|luke-jr|jonasschnelli: ☹
438 2017-05-11 20:00:29	0|wumpus|I don't think it's *useful* for most people but those things can be fun to play with and people learn how it works
439 2017-05-11 20:00:47	0|jonasschnelli|3D would be in a form you woudnl't recognize. The depth and shadows only slighly change when you drag the coins.
440 2017-05-11 20:01:06	0|wumpus|jonasschnelli: oh, so no tetris blocks? :-)
441 2017-05-11 20:01:18	0|jonasschnelli|heh
442 2017-05-11 20:02:11	0|jonasschnelli|Someone I feel if – from the beginning – coin selection would have been a "manual process", people would understand Bitcoin transactions better.
443 2017-05-11 20:02:18	0|wumpus|plane tiling puzzle but with the goal to get as close as possible to the spend value :p
444 2017-05-11 20:02:25	0|jonasschnelli|Also the "feerate" problem.. that it's not absolute.
445 2017-05-11 20:02:37	0|wumpus|jonasschnelli: I tend to agree
446 2017-05-11 20:03:10	0|gmaxwell|A better visualization might be be the coins being sized based on how much weight they add to the txn, and different metals based on value. :P
447 2017-05-11 20:03:16	0|jonasschnelli|Automatic coin selection may be something large wallets / exchanges are using.
448 2017-05-11 20:03:32	0|jonasschnelli|gmaxwell: Oh. Different metals... I like this.
449 2017-05-11 20:03:59	0|instagibbs|"Silver, since when did I have Litecoin?"
450 2017-05-11 20:04:11	0|jonasschnelli|With a SW badge? :-)
451 2017-05-11 20:04:41	0|luke-jr|gmaxwell: yeah, I was thinking numbers rather than colours
452 2017-05-11 20:04:44	0|luke-jr|maybe both?
453 2017-05-11 20:05:27	0|wumpus|niec idea
454 2017-05-11 20:05:31	0|gmaxwell|luke-jr: well yea, both of course.
455 2017-05-11 20:06:01	0|luke-jr|gold for > 1 ᵐTBC, silver for > ᵗTBC, bronze for > 1 TBC, and "plastic" for smaller
456 2017-05-11 20:06:17	0|gmaxwell|hah
457 2017-05-11 20:38:53	0|instagibbs|my p2p-fullblocks test is timing out on a couple of my machines... is this known issue
458 2017-05-11 20:41:17	0|instagibbs|oh scratch that, this seems to be an assertion error
459 2017-05-11 22:00:36	0|gmaxwell|jonasschnelli: sipa: petertodd: luke-jr:   Bitcoin-dev is as useful as we make it.  There are certian reliable parties that will crap-post reply to every useful idea posted to the list.  When we respond directly point by point arguing their foolish positions, the list becomes useless for technical discussions.
460 2017-05-11 22:01:27	0|gmaxwell|I would strongly recommend that whenever you get a message from someone who has reliably made useless progress blocking responses that you do not respond to them point by point, but instead make sure their concerns are addressed in a message responding to a productive point raised by someone else.
461 2017-05-11 22:02:22	0|gmaxwell|Otherwise, the thread just becomes an endless argument over stupidity and people who are likely to make reasonable responses will ignore the thread.
462 2017-05-11 22:29:14	0|sipa|frequency of depths of blocks downloaded from my node over the past 3 months: http://bitcoin.sipa.be/depths.png
463 2017-05-11 23:45:25	0|chatter29|hey guys
464 2017-05-11 23:45:34	0|chatter29|allah is doing
465 2017-05-11 23:45:40	0|chatter29|sun is not doing allah is doing
466 2017-05-11 23:45:41	0|chatter29|to accept Islam say that i bear witness that there is no deity worthy of worship except Allah and Muhammad peace be upon him is his slave and messenger
467 2017-05-11 23:45:59	0|chatter29|As-salāmu ʿalaykum (Arabic: السَّلَامُ عَلَيْكُمْ‎‎ [asːaˈlaːmu ʕaˈlaikum]) is a greeting in Arabic that means "peace be upon you". The greeting is a standard salutation among Muslims and is routinely used whenever and wherever Muslims gather and interact, whether socially or within worship and other contexts. [1] The typical response to the greeti
468 2017-05-11 23:45:59	0|chatter29|ng is waʿalaykumu s-salām (وَعَلَيْكُم السَّلَام [waʕaˈlaikumu sːaˈlaːm]; "and upon you, peace").