-
vtnerd
selsta : yeah next task
-
grc001[m]
Hey folks, wen btc-xmr atomic swaps arrive it'd be really cool to see an xmr-trx atomic swap for usdt(trc20) transfers. erc20 too pricey... that is all. thank you.
-
awhelpqjrcg
'Oddly, no one who was directly involved with the SIR-C missions and currently still working the Lab, remembers the name Howard Chu, except for one who vaguely recalls Eugene Chu as having a brother names Howard'
-
awhelpqjrcg
Ed Caro, NASA Chief Engineer. cryptogazette.com/wp-content/uploads/2019/12/1-768x1445.png
-
juhgg
hello
-
moneromooo
So close.
-
xiewkixsmwkc
What kind of person steals from their own community?
removeddit.com/r/Monero/comments/6d…ds_to_give_up_his_commit_access_and (changed link to removeddit since /r/monero mod removed to try to cover it up - maybe fluffy himself?)
-
selsta
mj-xmr_: btw, it seems the clion is more popular than codeblocks
-
selsta
hmm ok ignore what I said, clion is a paid program
-
bobbieweirder[m]
does someone know anything about DAPS? it claims to be a decentralized BTC layer using RingCT with staking and random ring sizes. i am not finding a lot of discussions yet so i wanted to see if anyone here has any thoughts
-
bobbieweirder[m]
-
moneromooo
This channel is about monero development.
-
bobbieweirder[m]
but monero’s development doesn’t happen in a vacuum right? i just figured if another project is claiming to use our tech better it would invite discussions on whether we need to challenge our earlier assumptions so we can continue get better (if necessary). but i didn’t mean to speak out of turn. i can take the discussion elsewhere. sorry!
-
moneromooo
If you can have better questions than that, they might well be on topic :)
-
moneromooo
Just understanding how some altcoin works does not seem to be though.
-
bobbieweirder[m]
i respect that. let me do some more digging and come back with something better
-
mj-xmr_
selsta, we were using CLion at work, but as you noted. Also MS VS could be something to mention. Both use CMakeLists.txt natively. Same category, but VS is free. CodeBlocks belongs to the same category as Eclipse CDT, which would maybe be a better option than CB for many, yeah.
-
mj-xmr_
and both are equally unstable :D
-
CCosta[m]
so, I'm trying to figure out how to read the binary from `/get_txpool_backlog` from the daemon rpc - is there a piece of code that I could take a look at as reference?
-
CCosta[m]
from
monero-project/monero 55bec1f I can't really tell how that array is being serialized (so that I can deserialize in my code)
-
CCosta[m]
(yes, I see `KV_SERIALIZE_CONTAINER_POD_AS_BLOB(backlog)`, but, not sure how I deserialize it)
-
moneromooo
I think vtnerd added some doc for serialization. find . -name \*.md
-
moneromooo
But the AS_BLOB will be (I think) a varint for the size and a wholesale memcpy.
-
moneromooo
That's the reason for the binary call, to avoid per field processing.
-
floam412
can I buy the dev team a couple bottles of barolo?
-
floam412
:)
-
floam412
you guys have been working your asses off from what it looks like outside of the freenode IRC world
-
mj-xmr_
Really? From here it looks like a dope party :)
-
CCosta[m]
<moneromooo "That's the reason for the binary"> makes sense - I'll take a look at it - thanks!
-
mj-xmr_
-
mj-xmr_
Crunched the numbers of last series of merges from yesterday.
-
wfaressuissia[m]
ping next time if there is anything interesting in those automatically calculated numbers
-
mj-xmr_
The interesting part is inside the reports. What they helped me already with was finding the bottlenecks of the core_tests.
-
wfaressuissia[m]
Could it help to find bottlenecks of randomx miner ?
-
mj-xmr_
But it does require some active detectivism.
-
mj-xmr_
You could for sure use the Valgrind script to setup an appropriate experiment, yes.
-
mj-xmr_
I could tutor you tomorrow, if you like.
-
wfaressuissia[m]
I suppose the answer is it can't
-
mj-xmr_
Why?
-
mj-xmr_
Forgot to add: It doesn't make coffee.
-
mj-xmr_
-
mj-xmr_
Write a unit test for randomX, loop it, and filter out all other tests. Plug it in the config file. Done.
-
wfaressuissia[m]
Let increase difficulty of the task: find the bottleneck and solution to speed up it
-
mj-xmr_
A lot of things are possible. The question is: who has the time to review what you find out?
-
mj-xmr_
For instance, I reduced the core test speed by 90%. Not merged yet. Got your answer?
-
mj-xmr_
So I won't be spamming the devs with 10 branches of the same type. It only increases my maintenance burden.
-
wfaressuissia[m]
It'll be interesting to see that core tests 90% speed up in details after submitted PR
-
mj-xmr_
-
mj-xmr_
It works fine, just by default it was decided to be switched off for valid reasons. But it's just a switch.
-
selsta
I still don't fully get it. How do I know when to to regenerate the cache?
-
wfaressuissia[m]
That's not speed up but reuse of old cache and requires manual approval since it isn't general purpose caching
-
mj-xmr_
selsta, You know that you don't need to regenerate the cache when you know, that you haven't touched the part, that has to do with the data generation.
-
moneromooo
but if you know you didn't, you don't need to run the tests again do you ?
-
wfaressuissia[m]
yes, it's 100% speed up then :D
-
moneromooo
I guess if you changed verification only. But that's fairly rare.
-
moneromooo
Anyway, it seems very dangerous to me.
-
mj-xmr_
These are high level tests, able to catch failures of many types.
-
mj-xmr_
So I don't see why I would be happy with not running the tests if I modified a functionality that can affect the verdict.
-
wfaressuissia[m]
"Anyway, it seems very dangerous to me." if someone don't know what is going on inside the code then 5 minutes blind test is better than nothing and and faster than 52 minutes test
-
wfaressuissia[m]
but it's better to read the code instead
-
mj-xmr_
better is to do both.
-
mj-xmr_
We're not machines
-
wfaressuissia[m]
We're better than machines
-
mj-xmr_
And sometimes we're sleeeepy
-
mj-xmr_
Which reminds me, that in our SCRUM teams, the POs/SMs blindly believe in the code review process to catch ALL the mistakes, that the CI didn't. A typical reason for failure of the projects.
-
mj-xmr_
You wouldn't believe what shit goes to master at times after a formal review ...
-
wfaressuissia[m]
your quote would be appropriate to some PR that adds more tests but not reusing old cache of existing ones
-
mj-xmr_
Which quote now?
-
mj-xmr_
And which old cache of existing ones?
-
mj-xmr_
Each test has its own cache.
-
wfaressuissia[m]
not quote but statement about some; cache of core tests
-
mj-xmr_
In my SCRUM story I was referring more to blind faith in humans ability to catch mistakes.
-
wfaressuissia[m]
* not quote but statement about SCRUM teams; cache of core tests
-
mj-xmr_
Tests can be added anytime. But if you know that adding another test will cost another 20 minutes every time you run it, do you think you and others will have motivation to run these tests frequently?
-
selsta
don't write tests that take 20min :D
-
mj-xmr_
selsta, you're right. We're already full of them. But then again, should I remove them? :)
-
mj-xmr_
Because improving the test's speed at the core by modifying the implementation was rejected here Big Time.
-
mj-xmr_
moneromooo knows the pain as well.
-
mj-xmr_
So there's not much more that one can do to keep everybody happy :)
-
wfaressuissia[m]
correct speed up isn't easy but possible, spend more time and will find good way to speed up it
-
mj-xmr_
Alright I'll think about it when trying to fall asleep. G'night.