12020-06-09T00:00:03 *** Oejet has quit IRC
22020-06-09T00:01:33 *** Daniel has joined #bitcoin-core-dev
32020-06-09T00:01:57 *** Daniel is now known as Guest91026
42020-06-09T00:06:45 *** Dean_Guss has joined #bitcoin-core-dev
52020-06-09T00:06:58 *** justanotheruser has joined #bitcoin-core-dev
62020-06-09T00:09:03 *** DeanGuss has quit IRC
72020-06-09T00:12:00 *** Guest91026 has quit IRC
82020-06-09T00:19:56 *** S3RK has joined #bitcoin-core-dev
92020-06-09T00:21:49 <phantomcircuit> sipa, actually this should make is really easy to do parallel scans
102020-06-09T00:22:06 *** n3glv1 has joined #bitcoin-core-dev
112020-06-09T00:25:35 *** S3RK has quit IRC
122020-06-09T00:30:01 *** justanotheruser has quit IRC
132020-06-09T00:30:30 *** promag has quit IRC
142020-06-09T00:39:04 *** roconnor has quit IRC
152020-06-09T00:42:32 *** S3RK has joined #bitcoin-core-dev
162020-06-09T00:42:55 *** Dean_Guss has quit IRC
172020-06-09T00:43:20 *** Dean_Guss has joined #bitcoin-core-dev
182020-06-09T00:45:54 *** justanotheruser has joined #bitcoin-core-dev
192020-06-09T01:03:55 *** AaronvanW has quit IRC
202020-06-09T01:46:22 *** Mercury_Vapor has quit IRC
212020-06-09T01:47:46 <aj> sipa: oops, was about to say "i'm not seeing any more premature requests either", but then realised when i picked up your updated patch i didn't reinstate the premature logging...
222020-06-09T01:50:21 *** harrigan has joined #bitcoin-core-dev
232020-06-09T01:50:26 *** harrigan- has quit IRC
242020-06-09T01:53:28 <aj> hmm, and i didn't merge your patch very well anyway apparently, drat
252020-06-09T01:53:55 *** troygiorshev has quit IRC
262020-06-09T01:56:37 *** Relis has quit IRC
272020-06-09T02:00:26 *** troygiorshev has joined #bitcoin-core-dev
282020-06-09T02:07:26 *** rafalcpp has quit IRC
292020-06-09T02:13:45 *** Relis has joined #bitcoin-core-dev
302020-06-09T02:31:47 *** troygiorshev has quit IRC
312020-06-09T02:32:12 *** jarthur has quit IRC
322020-06-09T02:33:47 *** troygiorshev has joined #bitcoin-core-dev
332020-06-09T02:34:20 *** Relis has quit IRC
342020-06-09T02:50:26 *** proofofkeags has joined #bitcoin-core-dev
352020-06-09T03:00:02 *** n3glv1 has quit IRC
362020-06-09T03:03:53 *** justanotheruser has quit IRC
372020-06-09T03:18:18 *** Eagle[TM] has joined #bitcoin-core-dev
382020-06-09T03:19:18 *** justanotheruser has joined #bitcoin-core-dev
392020-06-09T03:20:38 *** EagleTM has quit IRC
402020-06-09T03:21:10 *** Beetstra1 has joined #bitcoin-core-dev
412020-06-09T03:56:48 *** bitcoin-git has joined #bitcoin-core-dev
422020-06-09T03:56:48 <bitcoin-git> [bitcoin] pstratem opened pull request #19216: wallet: Remove first parameter to ScanForWalletTransactions start_hash (master...2020-06-07-wallet-scanforwallettransactions) https://github.com/bitcoin/bitcoin/pull/19216
432020-06-09T03:56:49 *** bitcoin-git has left #bitcoin-core-dev
442020-06-09T03:58:55 *** proofofkeags has quit IRC
452020-06-09T03:59:21 *** proofofkeags has joined #bitcoin-core-dev
462020-06-09T04:20:38 *** vasild_ has joined #bitcoin-core-dev
472020-06-09T04:23:19 <phantomcircuit> anybody know why the cirrus ci failed on #19216 ?
482020-06-09T04:23:20 <gribble> https://github.com/bitcoin/bitcoin/issues/19216 | wallet: Remove first parameter to ScanForWalletTransactions start_hash by pstratem · Pull Request #19216 · bitcoin/bitcoin · GitHub
492020-06-09T04:23:21 <phantomcircuit> https://github.com/bitcoin/bitcoin/pull/19216/checks?check_run_id=752472448
502020-06-09T04:23:43 *** vasild has quit IRC
512020-06-09T04:23:44 *** vasild_ is now known as vasild
522020-06-09T04:53:58 *** bitcoin-git has joined #bitcoin-core-dev
532020-06-09T04:53:58 <bitcoin-git> [bitcoin] hebasto closed pull request #18927: Pass bilingual_str argument to AbortNode() (master...200510-abort) https://github.com/bitcoin/bitcoin/pull/18927
542020-06-09T04:54:09 *** bitcoin-git has left #bitcoin-core-dev
552020-06-09T04:54:28 *** bitcoin-git has joined #bitcoin-core-dev
562020-06-09T04:54:28 <bitcoin-git> [bitcoin] hebasto reopened pull request #18927: Pass bilingual_str argument to AbortNode() (master...200510-abort) https://github.com/bitcoin/bitcoin/pull/18927
572020-06-09T04:54:29 *** bitcoin-git has left #bitcoin-core-dev
582020-06-09T05:10:36 *** bitcoin-git has joined #bitcoin-core-dev
592020-06-09T05:10:36 <bitcoin-git> [bitcoin] glowang opened pull request #19217: disambiguate block-relay-only variable names from blocksonly variables (master...2020/06/08/rename-max-block-relay-connection-name) https://github.com/bitcoin/bitcoin/pull/19217
602020-06-09T05:10:37 *** bitcoin-git has left #bitcoin-core-dev
612020-06-09T05:14:30 *** IGHOR has joined #bitcoin-core-dev
622020-06-09T05:20:16 *** proofofkeags has quit IRC
632020-06-09T05:41:35 *** bitcoin-git has joined #bitcoin-core-dev
642020-06-09T05:41:35 <bitcoin-git> [bitcoin] sipa opened pull request #19219: Replace automatic bans with discouragement filter (master...202006_discourage) https://github.com/bitcoin/bitcoin/pull/19219
652020-06-09T05:41:36 *** bitcoin-git has left #bitcoin-core-dev
662020-06-09T05:51:49 <phantomcircuit> `importwallet` doesn't update transactions that are found in the scan initiated when the wallet is loaded if they already exist, which i think could mean they get attributed to the wrong block?
672020-06-09T05:52:55 <sipa> how so?
682020-06-09T05:53:05 <sipa> you mean if there was a reorg in between?
692020-06-09T05:53:50 <phantomcircuit> sipa, i think so yes
702020-06-09T05:54:04 <phantomcircuit> if the reorg happened before you imported the wallet
712020-06-09T05:54:23 <phantomcircuit> it's the only thing that calls scan with fUpdate=false
722020-06-09T05:56:53 <sipa> but the wallet will observe the reorg when your node activates it?
732020-06-09T05:57:11 <sipa> or you mean if this happens while the wallet is not loaded?
742020-06-09T06:00:02 *** Beetstra1 has quit IRC
752020-06-09T06:00:34 *** proofofkeags has joined #bitcoin-core-dev
762020-06-09T06:02:49 <phantomcircuit> sipa, if the reorg happens when you dont have the wallet activated
772020-06-09T06:02:55 <phantomcircuit> sipa, it's kind of a catch 22
782020-06-09T06:03:10 <phantomcircuit> it shouldn't be possible for the fUpdate flag to mean anything
792020-06-09T06:03:14 <phantomcircuit> and yet it's there?
802020-06-09T06:05:22 *** Pavlenex has joined #bitcoin-core-dev
812020-06-09T06:10:03 *** proofofkeags has quit IRC
822020-06-09T06:17:14 *** Pavlenex has quit IRC
832020-06-09T06:19:56 *** Guyver2 has joined #bitcoin-core-dev
842020-06-09T06:21:48 *** bitcoin-git has joined #bitcoin-core-dev
852020-06-09T06:21:48 <bitcoin-git> [bitcoin] hebasto closed pull request #18927: Pass bilingual_str argument to AbortNode() (master...200510-abort) https://github.com/bitcoin/bitcoin/pull/18927
862020-06-09T06:21:49 *** bitcoin-git has left #bitcoin-core-dev
872020-06-09T06:22:08 *** bitcoin-git has joined #bitcoin-core-dev
882020-06-09T06:22:08 <bitcoin-git> [bitcoin] hebasto reopened pull request #18927: Pass bilingual_str argument to AbortNode() (master...200510-abort) https://github.com/bitcoin/bitcoin/pull/18927
892020-06-09T06:22:09 *** bitcoin-git has left #bitcoin-core-dev
902020-06-09T06:22:13 *** GothAlice1 has joined #bitcoin-core-dev
912020-06-09T06:24:38 *** S3RK has quit IRC
922020-06-09T06:24:51 *** jonatack has quit IRC
932020-06-09T06:25:04 *** S3RK has joined #bitcoin-core-dev
942020-06-09T06:28:13 *** Kiminuo has joined #bitcoin-core-dev
952020-06-09T06:29:26 *** S3RK has quit IRC
962020-06-09T06:38:03 *** kristapsk has quit IRC
972020-06-09T06:42:26 *** bitcoin-git has joined #bitcoin-core-dev
982020-06-09T06:42:26 <bitcoin-git> [bitcoin] hebasto opened pull request #19220: refactor: Replace RecursiveMutex with Mutex in warnings.cpp (master...200609-warn-mx) https://github.com/bitcoin/bitcoin/pull/19220
992020-06-09T06:42:35 *** bitcoin-git has left #bitcoin-core-dev
1002020-06-09T06:53:59 *** Pavlenex has joined #bitcoin-core-dev
1012020-06-09T07:07:00 *** S3RK has joined #bitcoin-core-dev
1022020-06-09T07:11:38 *** S3RK has quit IRC
1032020-06-09T07:15:51 *** andytoshi has quit IRC
1042020-06-09T07:27:19 *** andytoshi has joined #bitcoin-core-dev
1052020-06-09T07:27:19 *** andytoshi has quit IRC
1062020-06-09T07:27:19 *** andytoshi has joined #bitcoin-core-dev
1072020-06-09T07:32:34 *** andytoshi has quit IRC
1082020-06-09T07:36:27 *** jonatack has joined #bitcoin-core-dev
1092020-06-09T07:43:40 *** andytoshi has joined #bitcoin-core-dev
1102020-06-09T07:43:40 *** andytoshi has quit IRC
1112020-06-09T07:43:40 *** andytoshi has joined #bitcoin-core-dev
1122020-06-09T07:51:04 *** S3RK has joined #bitcoin-core-dev
1132020-06-09T07:51:38 *** Kiminuo has quit IRC
1142020-06-09T08:31:18 *** justanotheruser has quit IRC
1152020-06-09T08:31:41 *** justanotheruser has joined #bitcoin-core-dev
1162020-06-09T08:35:18 *** meshcollider has quit IRC
1172020-06-09T08:39:04 *** Pavlenex has quit IRC
1182020-06-09T08:43:02 *** proofofkeags has joined #bitcoin-core-dev
1192020-06-09T08:43:43 *** DeanWeen has joined #bitcoin-core-dev
1202020-06-09T08:45:23 *** Dean_Guss has quit IRC
1212020-06-09T08:47:38 *** proofofkeags has quit IRC
1222020-06-09T08:51:13 *** rafalcpp has joined #bitcoin-core-dev
1232020-06-09T08:52:22 *** AaronvanW has joined #bitcoin-core-dev
1242020-06-09T08:52:51 *** rafalcpp has quit IRC
1252020-06-09T08:54:38 *** S3RK has quit IRC
1262020-06-09T08:57:19 *** meshcollider has joined #bitcoin-core-dev
1272020-06-09T09:00:02 *** GothAlice1 has quit IRC
1282020-06-09T09:11:08 *** promag has joined #bitcoin-core-dev
1292020-06-09T09:12:49 *** promag has quit IRC
1302020-06-09T09:13:09 <wumpus> luke-jr: yes, I managed to do it! (was pretty simple- "gpg --cert-digest-algo sha512 --expert --edit-key", then "sign", then re-upload the key to keyservers)
1312020-06-09T09:22:13 *** sankarshan1 has joined #bitcoin-core-dev
1322020-06-09T09:54:14 *** sankarshan1 has quit IRC
1332020-06-09T10:01:04 *** Guyver2_ has joined #bitcoin-core-dev
1342020-06-09T10:01:28 *** Guyver2 has quit IRC
1352020-06-09T10:03:20 *** Corine52Beer has joined #bitcoin-core-dev
1362020-06-09T10:04:52 *** luke-jr has quit IRC
1372020-06-09T10:05:57 *** jonatack has quit IRC
1382020-06-09T10:06:13 *** luke-jr has joined #bitcoin-core-dev
1392020-06-09T10:07:40 *** jonatack has joined #bitcoin-core-dev
1402020-06-09T10:14:10 *** jonatack has quit IRC
1412020-06-09T10:15:11 *** Corine52Beer has quit IRC
1422020-06-09T10:16:57 *** jonatack has joined #bitcoin-core-dev
1432020-06-09T10:19:20 *** Guyver2_ is now known as Guyver2
1442020-06-09T10:23:21 *** S3RK has joined #bitcoin-core-dev
1452020-06-09T10:27:23 *** S3RK has quit IRC
1462020-06-09T10:39:17 *** S3RK has joined #bitcoin-core-dev
1472020-06-09T10:43:55 *** proofofkeags has joined #bitcoin-core-dev
1482020-06-09T10:50:29 *** proofofkeags has quit IRC
1492020-06-09T11:09:51 *** S3RK has quit IRC
1502020-06-09T11:10:27 *** S3RK has joined #bitcoin-core-dev
1512020-06-09T11:14:55 *** S3RK has quit IRC
1522020-06-09T11:26:51 <provoostenator> sipa fjahr: do I understand correctly that MuHash behaves as a set? I.e. if you add a UTXO that already exists, you get the same hash?
1532020-06-09T11:27:27 <provoostenator> If so, does that offer a memory efficient alternative to dbcache? (ignoring consensus failure risk)
1542020-06-09T11:28:20 <aj> provoostenator: what? that can't be right? if it were you could test for membership easily?
1552020-06-09T11:28:29 <provoostenator> You could check if an input exists in the UTXO set by addin git
1562020-06-09T11:28:44 <provoostenator> aj: exactly, so it might be wrong :-)
1572020-06-09T11:29:18 <provoostenator> Or it's computationally expensive, but memory efficient. Either way, I understand before reviewing #19055
1582020-06-09T11:29:20 <gribble> https://github.com/bitcoin/bitcoin/issues/19055 | Add MuHash3072 implementation by fjahr · Pull Request #19055 · bitcoin/bitcoin · GitHub
1592020-06-09T11:31:18 *** Relis has joined #bitcoin-core-dev
1602020-06-09T11:31:38 <provoostenator> Ah I guess I see the practical problem: you still need to track the spending condition, so you still need to update a database, which is faster in RAM
1612020-06-09T11:32:24 <provoostenator> But maybe it allows for a different structure of the dbache.
1622020-06-09T11:32:39 *** bitdex has quit IRC
1632020-06-09T11:33:50 *** larsivi has joined #bitcoin-core-dev
1642020-06-09T11:36:43 *** Relis has quit IRC
1652020-06-09T11:39:56 <provoostenator> Each item in the MuHash UTXO set is made of: outpoint + (height * 2 + isCoinbase) + (value + scriptPubKey)
1662020-06-09T11:40:02 <provoostenator> https://github.com/bitcoin/bitcoin/pull/18000/files#diff-0ab7b981bd35c1a1f45b5ed75013a698R34
1672020-06-09T11:40:39 *** promag has joined #bitcoin-core-dev
1682020-06-09T11:41:10 <provoostenator> Which isn't the right index if you're trying to find out if a given input exists.
1692020-06-09T11:43:14 <provoostenator> We could add another MuHash with just outpoints, which can be used to see if an output exists. But not sure if that's useful.
1702020-06-09T11:43:26 <provoostenator> * if an input exists
1712020-06-09T11:45:59 *** promag has quit IRC
1722020-06-09T11:46:33 *** Mercury_Vapor has joined #bitcoin-core-dev
1732020-06-09T12:00:02 *** larsivi has quit IRC
1742020-06-09T12:01:04 *** bitcoin-git has joined #bitcoin-core-dev
1752020-06-09T12:01:04 <bitcoin-git> [bitcoin] practicalswift opened pull request #19222: tests: Add fuzzing harness for BanMan (master...fuzzers-banman) https://github.com/bitcoin/bitcoin/pull/19222
1762020-06-09T12:01:05 *** bitcoin-git has left #bitcoin-core-dev
1772020-06-09T12:04:15 *** pinheadm_ has quit IRC
1782020-06-09T12:17:11 *** S3RK has joined #bitcoin-core-dev
1792020-06-09T12:19:19 *** Relis has joined #bitcoin-core-dev
1802020-06-09T12:21:39 *** S3RK has quit IRC
1812020-06-09T12:21:58 *** Kampala has joined #bitcoin-core-dev
1822020-06-09T12:22:06 *** troygiorshev has quit IRC
1832020-06-09T12:22:47 *** troygiorshev has joined #bitcoin-core-dev
1842020-06-09T12:23:23 *** jb55 has quit IRC
1852020-06-09T12:23:56 *** jb55 has joined #bitcoin-core-dev
1862020-06-09T12:24:50 *** promag has joined #bitcoin-core-dev
1872020-06-09T12:26:21 <fjahr> provoostenator: unfortunately not, you can add the same number multiple times and the hash changes. In some of the comments I think the word 'set' is misused, did that make you think that? It came up in the last review club as well, I think.
1882020-06-09T12:27:59 <provoostenator> fjahr: because of the terminology in muhash.h: "Multiply (resulting in a hash for the union of the sets)"
1892020-06-09T12:28:06 *** promag has quit IRC
1902020-06-09T12:28:39 <provoostenator> Is it non-determistic as well?
1912020-06-09T12:28:47 <provoostenator> No, that can't be, given the test vector.
1922020-06-09T12:29:03 <fjahr> Yeah, I will fix that.
1932020-06-09T12:29:25 <fjahr> it is deterministic.
1942020-06-09T12:30:51 <provoostenator> I'll read the Review Club minutes to see if I can wrap my head around how something can allow unions, not be a set, yet still bedeterministic.
1952020-06-09T12:32:18 *** pinheadmz has joined #bitcoin-core-dev
1962020-06-09T12:32:30 <provoostenator> It's path dependent?
1972020-06-09T12:33:42 <fjahr> No :) Did you take a look at the python implementation already? We will talk about it tomorrow. It is easier to grasp the algo with it. In the last pr review club we talked more about the project as a whole with the index, tomorrow should be more interesting for the algo.
1982020-06-09T12:39:01 *** promag has joined #bitcoin-core-dev
1992020-06-09T12:39:25 <provoostenator> I'll try to make tomorrows PR review club
2002020-06-09T12:39:41 <fjahr> would be great :)
2012020-06-09T12:40:05 *** pretyflaco has joined #bitcoin-core-dev
2022020-06-09T12:40:08 <provoostenator> I already left a comment on your original PR that we should design the index such that it's easy to add more, if we want to try a different hash function.
2032020-06-09T12:42:17 *** dr-orlovsky has quit IRC
2042020-06-09T12:43:34 *** promag has quit IRC
2052020-06-09T12:44:53 <fjahr> Yeah, I think some of the latest development in #19145 addresses this partly? I implemented marcofalke's idea to give in a hash_type. https://github.com/bitcoin/bitcoin/pull/19145/files#diff-0ab7b981bd35c1a1f45b5ed75013a698R123 and it also has a more extensive integration test now https://github.com/bitcoin/bitcoin/pull/19145/files#diff-75261b195110445462597c3f27c4dee0
2062020-06-09T12:44:54 <gribble> https://github.com/bitcoin/bitcoin/issues/19145 | Add hash_type options for gettxoutsetinfo by fjahr · Pull Request #19145 · bitcoin/bitcoin · GitHub
2072020-06-09T12:45:17 *** pretyflaco has quit IRC
2082020-06-09T12:47:14 <provoostenator> That makes sense
2092020-06-09T12:48:03 *** luke-jr has quit IRC
2102020-06-09T12:48:27 *** luke-jr has joined #bitcoin-core-dev
2112020-06-09T12:56:38 *** bitcoin-git has joined #bitcoin-core-dev
2122020-06-09T12:56:38 <bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/a79bca2f1fb2...9ad6f14175c1
2132020-06-09T12:56:39 <bitcoin-git> bitcoin/master bacbfb6 Hennadii Stepanov: refactor: Replace RecursiveMutex with Mutex in warnings.cpp
2142020-06-09T12:56:39 <bitcoin-git> bitcoin/master 9ad6f14 MarcoFalke: Merge #19220: refactor: Replace RecursiveMutex with Mutex in warnings.cpp
2152020-06-09T12:56:41 *** bitcoin-git has left #bitcoin-core-dev
2162020-06-09T12:56:55 <aj> fjahr: why #define's instead of inline functions in muhash.cpp? mul() sets c2 to 0 even though it doesn't take it as a param which seems pretty weird too (it's already zeroed the only place it's called, so it's technically okay, but...)
2172020-06-09T12:56:58 *** bitcoin-git has joined #bitcoin-core-dev
2182020-06-09T12:56:58 <bitcoin-git> [bitcoin] MarcoFalke merged pull request #19220: refactor: Replace RecursiveMutex with Mutex in warnings.cpp (master...200609-warn-mx) https://github.com/bitcoin/bitcoin/pull/19220
2192020-06-09T12:56:59 *** bitcoin-git has left #bitcoin-core-dev
2202020-06-09T13:00:13 <fjahr> aj: a macro was suggested and I did not have too much of an opinion on it
2212020-06-09T13:02:43 *** bitcoin-git has joined #bitcoin-core-dev
2222020-06-09T13:02:43 <bitcoin-git> [bitcoin] fanquake closed pull request #18542: 0.19: gui: Avoid Wallet::GetBalance in WalletModel::pollBalanceChanged (0.19...2020-04-backport-18160) https://github.com/bitcoin/bitcoin/pull/18542
2232020-06-09T13:02:44 *** bitcoin-git has left #bitcoin-core-dev
2242020-06-09T13:03:59 *** bashco has quit IRC
2252020-06-09T13:06:01 *** pretyflaco has joined #bitcoin-core-dev
2262020-06-09T13:07:17 <provoostenator> Reading the paper it's less like a set, more like an array. Adding stuff isn't a union but a join. Removing stuff implies you can have "negative" entries in the array.
2272020-06-09T13:07:20 <fjahr> aj: oh, I think I misunderstood your question. But the answer is similar. I did not change sipa's code too much unless I needed to fix something. But happy to make changes based on feedback.
2282020-06-09T13:10:27 <fjahr> provoostenator: yes, it is really just like multiplication and division (over a finite field).
2292020-06-09T13:34:10 *** dr-orlovsky has joined #bitcoin-core-dev
2302020-06-09T13:35:33 <provoostenator> Maybe instead of "add to set" say "multiply numerator", and instead of "remove from set" say "multiply denominator"
2312020-06-09T13:40:17 <provoostenator> The compact Python code is certainly easier to follow
2322020-06-09T13:47:51 *** S3RK has joined #bitcoin-core-dev
2332020-06-09T13:52:30 *** S3RK has quit IRC
2342020-06-09T14:11:12 *** proofofkeags has joined #bitcoin-core-dev
2352020-06-09T14:13:44 *** promag has joined #bitcoin-core-dev
2362020-06-09T14:20:51 *** pretyflaco1 has joined #bitcoin-core-dev
2372020-06-09T14:22:54 *** pretyflaco has quit IRC
2382020-06-09T14:28:06 *** bitcoin-git has joined #bitcoin-core-dev
2392020-06-09T14:28:07 <bitcoin-git> [bitcoin] fanquake opened pull request #19224: [0.20] Backports (0.20...0_20_1_backports) https://github.com/bitcoin/bitcoin/pull/19224
2402020-06-09T14:28:07 *** bitcoin-git has left #bitcoin-core-dev
2412020-06-09T14:38:24 *** promag has quit IRC
2422020-06-09T14:48:04 *** pretyflaco has joined #bitcoin-core-dev
2432020-06-09T14:51:50 *** pretyflaco1 has quit IRC
2442020-06-09T14:56:01 *** Relis has quit IRC
2452020-06-09T15:00:01 *** Kampala has quit IRC
2462020-06-09T15:08:00 *** Relis has joined #bitcoin-core-dev
2472020-06-09T15:21:47 *** serAphim has joined #bitcoin-core-dev
2482020-06-09T15:22:11 *** serAphim is now known as Guest26146
2492020-06-09T15:31:23 *** dviola has joined #bitcoin-core-dev
2502020-06-09T15:33:27 *** jarthur has joined #bitcoin-core-dev
2512020-06-09T15:57:41 *** molz_ has quit IRC
2522020-06-09T15:57:48 *** Talkless has joined #bitcoin-core-dev
2532020-06-09T16:13:53 *** pretyflaco has quit IRC
2542020-06-09T16:16:09 *** mol has joined #bitcoin-core-dev
2552020-06-09T16:20:37 *** vasild_ has joined #bitcoin-core-dev
2562020-06-09T16:23:43 *** vasild has quit IRC
2572020-06-09T16:23:44 *** vasild_ is now known as vasild
2582020-06-09T16:27:05 *** pretyflaco has joined #bitcoin-core-dev
2592020-06-09T16:27:43 *** justanotheruser has quit IRC
2602020-06-09T16:30:14 *** mol has quit IRC
2612020-06-09T16:30:53 *** Chris_Stewart_5 has quit IRC
2622020-06-09T16:39:39 *** kristapsk has joined #bitcoin-core-dev
2632020-06-09T16:42:56 *** mol has joined #bitcoin-core-dev
2642020-06-09T16:44:26 *** Chris_Stewart_5 has joined #bitcoin-core-dev
2652020-06-09T16:44:28 <phantomcircuit> sipa, yeah it really looks like the fUpdate flag *shouldn't* do anything, but then again the entire rescan functionality shouldn't do anything
2662020-06-09T16:45:58 <sipa> fjahr: i need to check the paper again, but i'm not sure that MuHash is secure when the same element is allowed to be added multiple times (under control of an attacker)
2672020-06-09T16:46:10 <sipa> fjahr: which is why it's called a set and not a multiset
2682020-06-09T16:46:24 <sipa> ECMH is explicitly secure even when items have multiplicity
2692020-06-09T16:47:27 <sipa> provoostenator: so it is a set, but only by virtue of not being allowed to add the same element multiple times (or not being allowed to remove something that you don't know isn't already in it)
2702020-06-09T16:47:39 <sipa> not by the data structure itself enforcing that
2712020-06-09T16:50:16 <fjahr> sipa: ok, that makes sense. I wasn't sure about security implication, I just knew it is mathematically possible. I will make sure to add that to the documentation.
2722020-06-09T16:51:47 <sipa> fjahr: my somewhat naive intuition is that it's actually fine with multiplicity, but given that i don't remember that being proven, and we don't need, i'd rather be conservative
2732020-06-09T16:57:02 *** mol_ has joined #bitcoin-core-dev
2742020-06-09T16:59:31 *** mol has quit IRC
2752020-06-09T17:02:00 <phantomcircuit> sipa, im inclined to remove the fUpdate flag with an override to always use fUpdate
2762020-06-09T17:24:56 *** justanotheruser has joined #bitcoin-core-dev
2772020-06-09T17:43:41 *** Kiminuo has joined #bitcoin-core-dev
2782020-06-09T17:47:34 *** Highway61 has quit IRC
2792020-06-09T17:59:30 *** mrostecki has quit IRC
2802020-06-09T18:00:02 *** promag has joined #bitcoin-core-dev
2812020-06-09T18:00:02 *** Guest26146 has quit IRC
2822020-06-09T18:16:50 *** bitcoin-git has joined #bitcoin-core-dev
2832020-06-09T18:16:50 <bitcoin-git> [bitcoin] MarcoFalke opened pull request #19226: test: Add BerkeleyDatabase tsan suppression (master...2006-ciTsanSupWallet) https://github.com/bitcoin/bitcoin/pull/19226
2842020-06-09T18:16:54 *** bitcoin-git has left #bitcoin-core-dev
2852020-06-09T18:21:48 *** T-Junk has joined #bitcoin-core-dev
2862020-06-09T18:28:31 *** promag has quit IRC
2872020-06-09T18:28:49 *** tripleslash has quit IRC
2882020-06-09T18:29:04 *** promag has joined #bitcoin-core-dev
2892020-06-09T18:33:33 *** promag has quit IRC
2902020-06-09T18:44:20 *** lightlike has joined #bitcoin-core-dev
2912020-06-09T18:51:35 *** Highway61 has joined #bitcoin-core-dev
2922020-06-09T19:03:50 *** davterra has quit IRC
2932020-06-09T19:05:06 *** proofofkeags has quit IRC
2942020-06-09T19:05:13 *** Talkless has quit IRC
2952020-06-09T19:05:42 *** proofofkeags has joined #bitcoin-core-dev
2962020-06-09T19:07:07 *** sdaftuar has quit IRC
2972020-06-09T19:07:27 *** sdaftuar has joined #bitcoin-core-dev
2982020-06-09T19:07:34 *** harrigan has quit IRC
2992020-06-09T19:09:31 *** Highway62 has joined #bitcoin-core-dev
3002020-06-09T19:09:43 *** harrigan has joined #bitcoin-core-dev
3012020-06-09T19:09:51 *** Highway61 has quit IRC
3022020-06-09T19:09:51 *** Highway62 is now known as Highway61
3032020-06-09T19:10:21 *** proofofkeags has quit IRC
3042020-06-09T19:37:31 *** rafalcpp has joined #bitcoin-core-dev
3052020-06-09T19:38:30 *** proofofkeags has joined #bitcoin-core-dev
3062020-06-09T19:43:16 *** proofofkeags has quit IRC
3072020-06-09T19:57:35 *** Highway61 has quit IRC
3082020-06-09T19:58:30 *** promag has joined #bitcoin-core-dev
3092020-06-09T20:02:47 *** promag has quit IRC
3102020-06-09T20:09:20 *** vasild has quit IRC
3112020-06-09T20:09:35 *** vasild has joined #bitcoin-core-dev
3122020-06-09T20:10:29 *** bitcoin-git has joined #bitcoin-core-dev
3132020-06-09T20:10:29 <bitcoin-git> [bitcoin] TrentZ opened pull request #19227: change blacklist to blocklist (master...master) https://github.com/bitcoin/bitcoin/pull/19227
3142020-06-09T20:10:30 *** bitcoin-git has left #bitcoin-core-dev
3152020-06-09T20:18:04 *** proofofkeags has joined #bitcoin-core-dev
3162020-06-09T20:18:05 *** proofofkeags has quit IRC
3172020-06-09T20:18:17 *** proofofkeags has joined #bitcoin-core-dev
3182020-06-09T20:35:16 *** Zenton has quit IRC
3192020-06-09T20:35:17 *** guest534543 has joined #bitcoin-core-dev
3202020-06-09T20:36:14 *** promag has joined #bitcoin-core-dev
3212020-06-09T20:36:17 *** Zenton has joined #bitcoin-core-dev
3222020-06-09T20:38:41 *** Highway61 has joined #bitcoin-core-dev
3232020-06-09T20:38:51 *** Kiminuo has quit IRC
3242020-06-09T20:39:32 *** guest534543 has quit IRC
3252020-06-09T20:40:11 *** promag has quit IRC
3262020-06-09T20:40:16 *** Dean_Guss has joined #bitcoin-core-dev
3272020-06-09T20:40:32 *** DeanWeen has quit IRC
3282020-06-09T20:50:55 *** bitcoin-git has joined #bitcoin-core-dev
3292020-06-09T20:50:55 <bitcoin-git> [bitcoin] sipa opened pull request #19228: Update libsecp256k1 subtree (master...202006_secp256k1_update) https://github.com/bitcoin/bitcoin/pull/19228
3302020-06-09T20:50:56 *** bitcoin-git has left #bitcoin-core-dev
3312020-06-09T21:00:02 *** T-Junk has quit IRC
3322020-06-09T21:07:27 *** filchef has joined #bitcoin-core-dev
3332020-06-09T21:08:25 *** filchef has quit IRC
3342020-06-09T21:09:16 *** Guyver2 has quit IRC
3352020-06-09T21:11:29 *** Highway61 has quit IRC
3362020-06-09T21:13:12 *** Highway61 has joined #bitcoin-core-dev
3372020-06-09T21:17:38 *** bitcoin-git has joined #bitcoin-core-dev
3382020-06-09T21:17:39 <bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/9ad6f14175c1...221873e15874
3392020-06-09T21:17:39 <bitcoin-git> bitcoin/master 6fc6416 TrentZ: change blacklist to blocklist
3402020-06-09T21:17:40 <bitcoin-git> bitcoin/master 221873e MarcoFalke: Merge #19227: test: change blacklist to blocklist
3412020-06-09T21:17:42 *** bitcoin-git has left #bitcoin-core-dev
3422020-06-09T21:17:58 *** bitcoin-git has joined #bitcoin-core-dev
3432020-06-09T21:17:58 <bitcoin-git> [bitcoin] MarcoFalke merged pull request #19227: test: change blacklist to blocklist (master...master) https://github.com/bitcoin/bitcoin/pull/19227
3442020-06-09T21:17:59 *** bitcoin-git has left #bitcoin-core-dev
3452020-06-09T21:19:57 *** bitcoin-git has joined #bitcoin-core-dev
3462020-06-09T21:19:59 <bitcoin-git> [bitcoin] MarcoFalke pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/221873e15874...f8364df25070
3472020-06-09T21:19:59 <bitcoin-git> bitcoin/master 77b79fa Wladimir J. van der Laan: refactor: Error message bilingual_str consistency
3482020-06-09T21:20:00 <bitcoin-git> bitcoin/master 425e7cb Wladimir J. van der Laan: refactor: Put`TryParsePermissionFlags` in anonymous namespace
3492020-06-09T21:20:01 <bitcoin-git> bitcoin/master 6fe9890 Wladimir J. van der Laan: refactor: Change Node::initError to take bilingual_str
3502020-06-09T21:20:03 *** bitcoin-git has left #bitcoin-core-dev
3512020-06-09T21:20:11 <jeremyrubin> MarcoFalke: I think that's the right call fwiw; err on inclusive
3522020-06-09T21:20:17 *** bitcoin-git has joined #bitcoin-core-dev
3532020-06-09T21:20:17 <bitcoin-git> [bitcoin] MarcoFalke merged pull request #19176: refactor: Error message bilingual_str consistency (master...2020_06_bilingual_str) https://github.com/bitcoin/bitcoin/pull/19176
3542020-06-09T21:20:18 *** bitcoin-git has left #bitcoin-core-dev
3552020-06-09T21:30:43 *** proofofkeags has quit IRC
3562020-06-09T21:31:16 *** proofofkeags has joined #bitcoin-core-dev
3572020-06-09T21:32:10 *** mol has joined #bitcoin-core-dev
3582020-06-09T21:34:26 *** mol_ has quit IRC
3592020-06-09T21:39:02 *** proofofkeags has quit IRC
3602020-06-09T21:39:15 *** proofofkeags has joined #bitcoin-core-dev
3612020-06-09T21:40:32 *** proofofkeags has quit IRC
3622020-06-09T21:41:04 *** proofofkeags has joined #bitcoin-core-dev
3632020-06-09T21:45:43 *** proofofkeags has quit IRC
3642020-06-09T21:57:02 *** retroj1 has joined #bitcoin-core-dev
3652020-06-09T21:58:17 *** proofofkeags has joined #bitcoin-core-dev
3662020-06-09T22:40:50 *** promag has joined #bitcoin-core-dev
3672020-06-09T22:45:11 *** promag has quit IRC
3682020-06-09T23:02:02 *** justanotheruser has quit IRC
3692020-06-09T23:13:57 *** justanotheruser has joined #bitcoin-core-dev
3702020-06-09T23:27:48 *** baldur has quit IRC
3712020-06-09T23:38:06 *** berndj has quit IRC
3722020-06-09T23:38:59 *** berndj has joined #bitcoin-core-dev
3732020-06-09T23:39:34 <stevenroose> kallewoof: did signet copy the bitcoin blockchain??
3742020-06-09T23:39:40 <stevenroose> I seem to be 9 years behind :o
3752020-06-09T23:40:12 *** luke-jr has quit IRC
3762020-06-09T23:41:03 *** luke-jr has joined #bitcoin-core-dev
3772020-06-09T23:41:03 *** baldur has joined #bitcoin-core-dev
3782020-06-09T23:49:27 *** proofofkeags has quit IRC
3792020-06-09T23:50:01 *** proofofkeags has joined #bitcoin-core-dev
3802020-06-09T23:54:14 *** proofofkeags has quit IRC