05:05:58 <Inge-> moneromooo: combining the metadata so that I can consolidate all the transaction details?
05:06:19 <Inge-> Or is there One Secret Trick that lets me concat those files or something?
07:03:23 <rbrunner> For all I know technically it should be traightforward to write a tool for merging meta-info from a group of wallet files for the same private key. Maybe would need one or the other new method in `wallet2` to support, but nothing wild I would say.
10:00:14 <bysnack> I'm confused about how transactions get added to the pool with dandlelion, when I send a transaction to my node it displays the "Transaction added to pool" message twice, one seems to be triggered by /sendrawtransaction and the other one by the levin packet NOTIFY_NEW_TRANSACTIONS.
10:00:14 <bysnack> Looking at tx_pool.cpp there is a set called m_txs_by_fee_and_receive_time and the transaction gets emplaced into that set in tx_pool::add_tx, this is getting called once for each "Transaction added to pool" message and the problem I encountered is that the receive time, used to order the transactions in the set and calculated with time(nullptr)
10:00:14 <bysnack> everytime tx-pool::add_tx is called, might vary from one call to another and therefore the same transaction can be found twice in that set.
10:00:14 <bysnack> After mining a block to confirm the transaction it's is going to be considered twice and a "key images already seen" message will be shown, then if I decide to mine another block one of those repeated transactions will still be alive in the set and because of that I encounter different behaviors for everytime I run a test case, from "failed to find
10:00:14 <bysnack> tx meta" because the transaction was removed from the db to having the same transaction in the db with invalid values which look like UB (i.e. extremely big block weight).
10:00:15 <bysnack> Did someone see anything related to this?
11:14:47 <moneromooo> I don't understand the question.
11:15:27 <moneromooo> Oh, you want to merge several wallet files ?
11:16:00 <moneromooo> Starting to get really complicated. Just use the same in the first place.
11:17:42 <moneromooo> Interesting, I did not know the meta not found was caused by dandelion, I wanted to look at that at some point.
11:18:11 <moneromooo> The txpool code became really hairy with dandelion, I don't think I really understand it anymore :/
11:18:36 <moneromooo> It is supposed to handle this though. Buggy, I guess.
11:24:31 <bysnack> Gonna look at it for a bit longer and open a PR if I can find a decent solution to the problem
11:29:48 <moneromooo> OK. You'll want vtnerd to look at it otherwise, he groks that new code well.
14:03:25 <kiwi_99> Hi guys. What is the contrib folder used for? Specifically the .mk files inside contrib/depends/packages?
14:03:40 <selsta> depends build system
14:03:49 <selsta> for cross compiling / reproducible builds
14:03:55 <kiwi_99> So only if you do make depends?
14:04:07 <kiwi_99> or for release-static-linux-x86_64 too?
14:04:31 <selsta> if you use gitian it will use depends
14:04:42 <kiwi_99> Alright thanks
14:04:56 <selsta> release-static-linux-x86_64 will not use it AFAIK
14:05:25 <moneromooo> contrib is for external stuff, typically helpers, glue between monero and other software, packaging stuff, etc.
14:05:55 <moneromooo> It's also got some source monero needs for historical reasons.
14:06:12 <caralho> Good evening
14:06:32 <kiwi_99> thanks for that
16:39:33 <vtnerd> bysnack : on the first /sendrawtransaction call, the tx is considered to be in the dandelion++ stempool, and therefore when a node eventually announces the tx to you it gets upgraded to the normal txpool
16:40:06 <vtnerd> unfortunately, these are the same txpool object which might make it harder conceptually for people reviewing the code at-a-glance
16:41:47 <vtnerd> I cant parse out what you are saying about mining blocks exactly
16:43:09 <moneromooo> I read it as "it tries to mine it twice".
16:44:24 <nakedpon_> does the monero gui solo miner use XMRIG?
16:44:58 <moneromooo> If you mean monero-wallet-gui, no.
16:45:10 <nakedpon_> ok, thank you
16:46:04 <nakedpon_> I'm curious why it wouldn't, but that's probably something best researched on my own
16:51:20 <sech1> Because xmrig is 3rd-party miner, it's not part of official monero binaries
16:56:13 <caralho> Rip fireiceUK
16:56:16 <nakedpon_> sech1: ahhh okay that makes total sense. it's all clicking now. the donations, why it's its own thing. thanks I feel less oblivious now
16:57:12 <sech1> xmrig can also mine like 20 other coins, so it's not monero-only miner
17:00:44 <nakedpon_> oh cool! I thought the xmrig was "monero rig"
17:01:25 <nakedpon_> which still looks to be the case. but I see now how it's able to mine other coin
20:45:40 <selsta> luigi1111: do you have to do merges this weekend?
20:45:59 <selsta> there are a lot of PRs in the merge bot so rip
20:46:01 <selsta> time*
20:50:48 <caralho> .merges
20:50:49 -xmr-pr- 6329 6337 6493 6500 6542 6546 6593 6600 6601 6614 6618 6632 6634 6661 6662 6677 6679 6689 6691 6693 6698 6703 6712 6715 6716 6718 6720 6722 6727 6733 6736
21:14:04 <luigi1111w> yikers
21:18:13 <sarang> ikr
21:18:20 <sarang> code freeze in 4 days
21:18:53 <selsta> yea that’s not going to work
21:18:54 <selsta> lol
21:19:01 <selsta> but merges would help
21:23:07 <sarang> ?
21:24:18 <selsta> code freeze in 4 days will not work
21:24:28 <selsta> there are still large unreviewed PRs
21:24:46 <selsta> or is code freeze no new feature PRs?
21:25:32 <moneromooo> Bug fixes are obviously ok, otherwise there wouldn't be any point in waiting before releasing.
21:26:14 <sarang> Well, the stated code freeze is August 17
21:26:34 <sarang> If this is not the case, it should be addressed
21:26:43 <selsta> sarang: what is your definition on code freeze?
21:26:54 <sarang> I assumed no new feature PRs
21:27:01 <selsta> that’s okay
21:27:03 <sarang> Otherwise why bother setting a code freeze?
21:27:40 <selsta> no new feature PRs should work, I don’t think there is anything planned for next HF that is not PRed yet
21:29:42 <caralho> Code freeze!
21:30:15 <selsta> sarang: is there a link with the HF timeline? I didn’t find it
21:31:14 <sarang> It was in meetings
21:31:27 <sarang> Code freeze Aug 17, release Sep 17, fork Oct 17
21:31:42 <sarang> If this is not the case, I would like to know ASAP
21:32:02 <sarang> This was to provide accurate information to hardware wallet teams
21:32:38 <caralho> There's a hardware wallet?
21:33:37 <selsta> sarang: should be good
21:33:53 <dsc_> .merges
21:33:53 -xmr-pr- 6329 6337 6493 6500 6542 6546 6593 6600 6601 6614 6618 6632 6634 6661 6662 6677 6679 6689 6691 6693 6698 6703 6712 6715 6716 6718 6720 6722 6727 6733 6736
21:34:26 <selsta> clsag PR still needs an approval on github, I don’t know how thorough the reviews have to be because the code is audited
21:34:30 <selsta> or is only part of the code audited?
21:35:06 <selsta> sarang: is approval something you can do for the CLSAG PR?
21:35:10 <dsc_> .merges
21:35:10 -xmr-pr- 6329 6337 6493 6500 6542 6546 6593 6600 6601 6614 6618 6632 6634 6661 6662 6677 6679 6689 6691 6693 6698 6703 6712 6715 6716 6718 6720 6722 6727 6733 6736
21:35:21 <dsc_> sorry for testing. it's broken.
21:35:22 <caralho> Kinda sexy, huh dsc_
21:35:55 <dsc_> will fix later (+v requirement)
21:41:02 <kpcyrd> what's up with the release notes for 0.16.0.3 in monero and monero-gui? is this urgent?
21:41:33 <selsta> luigi kinda only half uploaded them
21:42:35 <selsta> kpcyrd: it’s a placeholder, the website has release notes and they should also get uploaded to github soon
21:45:51 <sarang> selsta: the PR should receive additional scrutiny IMO
21:46:41 <kpcyrd> selsta: ok, thank you :)
21:47:36 <sarang> I wrote some of that code and should not be reviewing it
21:48:44 <caralho> I can haz PR?
22:23:16 <selsta> sarang: can you review the commits you did not write?
22:24:45 <selsta> also which commits were part of the audit?
22:28:22 <moneromooo> The audit reviewed all the commits. I did squash a little bit when rebasing, but the conflicts were minimal.
22:28:31 <selsta> ok that’s good
23:09:04 <luigi1111> the paste wasn't working sooo
23:13:55 <selsta> not true
23:14:23 <selsta> ok maybe true
23:16:58 <luigi1111> I must be quick before it expires