-
gingeropolous
-
UkoeHB
gingeropolous: cool :) jberman and I leaving our mark on the ledger
-
jeffro256[m]
That's awesome !
-
jeffro256[m]
Do we know who runs xmrchain.net ?
-
plowsof[m]
The web/admins name of xmrchain beings with g and rhymes with gingeropolous , he will know xd
-
jeffro256[m]
haha gingeropolous thanks
-
jeffro256[m]
I use that site all the time
-
jeffro256[m]
Is there anything keeping UkoeHB's new Seraphis proposal from moving to funding required?
repo.getmonero.org/monero-project/ccs-proposals/-/merge_requests/314
-
Guest82
djh
-
ooo123ooo1234[m]
<jeffro256[m]> "@ooo123ooo1234 I guess we have..." <- "
en.cppreference.com/w/cpp/io/c/setvbuf#Example", So "Uses the system-defined buffer size instead of a hardcoded 4096 value" is literally gratuitous change
-
ooo123ooo1234[m]
"Possible output: BUFSIZ is 8192, but optimal block size is 65536"
-
ooo123ooo1234[m]
<jeffro256[m]> "I probably won't be able to..." <- All changes except those 9 lines are noise since it just adds overhead between openssl api and thing wrapper in c++
-
ooo123ooo1234[m]
And this overhead makes verification harder without any benefits
-
ooo123ooo1234[m]
The final goal determines what is important and what is not, and if you are not approaching it then you're just adding useless overhead
-
ooo123ooo1234[m]
which in the best case will be fixed others, and in the worst case will lead to vulnerability
-
ooo123ooo1234[m]
Full code is complex graph which you don't want to study. And your changes is like doing bit flipping of few edges/vertices without taking into consideration relevant scope.
-
ooo123ooo1234[m]
Context free canges
-
ooo123ooo1234[m]
hahaha
-
ooo123ooo1234[m]
So instead of changing 1 edge to swap SHA256 for EVP_, you've decided to add new entity
-
ooo123ooo1234[m]
and say that it helps
-
ooo123ooo1234[m]
interesting theory
-
ooo123ooo1234[m]
<UkoeHB> "need a new fundamental concept..." <- Fundamental concept is to approach common goal without compromises and with critical thinking.
-
ooo123ooo1234[m]
I thought that correct cryptography is the most uncontroversial goal. If it would be true then at least code that isn't proven to be correct couldn't be merged.
-
ooo123ooo1234[m]
But it isn't true
-
ooo123ooo1234[m]
In all my previous attempts to deduce goal from the context and attack it fails. I suppose it's time to set goal explicitly
-
ooo123ooo1234[m]
s/In//, s/fails/failed/
-
garth
Can’t create view wallet on testnet:
monero-project/monero #8339
-
moneromooo
You say "on testnet" but the command line specifically omits --testnet. Is there a reason ?
-
garth
Shoot you’re right
-
garth
My bad
-
ooo123ooo1234[m]
rhetorical question: how many bits of security does monero protocol have ?
-
ooo123ooo1234[m]
any guesses ?
-
merope
That question doesn't make much sense. Also, read Zero to Monero
-
moneromooo
It was just a forgotten flag on the command line.
-
ooo123ooo1234[m]
<merope> "That question doesn't make..." <- Definition from here
en.wikipedia.org/wiki/Security_level. Have you already read Z2M ?
-
ooo123ooo1234[m]
To be precise, monero transaction protocol (the thing on top of pow)
-
merope
I know what security level means. I'm telling you that you can't blindly apply that definition to "the transaction protocol" as a whole, because it is composed of multiple parts - not all of which rely on bits of security. Therefore, go read ZtM2 and look up the security levels of the cryptographic primitives involved
-
ooo123ooo1234[m]
merope: Which part doesn't rely on bits of security ?
-
ooo123ooo1234[m]
Try to call at least 1
-
ooo123ooo1234[m]
merope: No, you don't know.
-
merope
The number of ring members in a ring signature, for example
-
merope
Additionally, different parts of the protocol rely on different primitives, so they will have different security levels against different types of attacks
-
merope
Something you would know, if you bothered doing your own reading instead of demanding to be spoonfed
-
merope
Do your homework
-
ooo123ooo1234[m]
merope: Number of ring signatures is a parameter of cryptography. It isn't a component. Ring signature is a component.
-
ooo123ooo1234[m]
s/signatures/members/
-
ooo123ooo1234[m]
merope: Call at least 2 different primitives
-
merope
This isn't highschool, and you are not a teacher
-
ooo123ooo1234[m]
merope: critical thinking can be used outside of highschool too
-
ooo123ooo1234[m]
merope: What would the goal of that homework ? How to verify that it was reached ? Have you done it already ?
-
ooo123ooo1234[m]
* What would be the goal
-
merope
Oh look - a shiny ignore button!
-
ooo123ooo1234[m]
merope: as expected
-
ooo123ooo1234[m]
merope: can't answer question -> can't call a troll -> use ignore button
-
ooo123ooo1234[m]
<merope> "That question doesn't make..." <- I would read something like this
reddit.com/r/AskEngineers/comments/…mpetent_is_ruining_my_topperforming. More revelant in current context.
-
ooo123ooo1234[m]
s/a_lazy_incompetent_is_ruining_my_topperforming/a\_lazy\_incompetent\_is\_ruining\_my\_topperforming/, s/revelant/relevant/
-
UkoeHB
ooo123ooo1234[m]: it’s supposed to be ~128 bit. Did you find something that makes it less?
-
ooo123ooo1234[m]
UkoeHB: can't comment
-
ooo123ooo1234[m]
So you guess is 128, ok
-
ooo123ooo1234[m]
s/you/your/
-
UkoeHB
Why can’t you comment?
-
ooo123ooo1234[m]
What's the precision interval for ~128 ? +-1 ?
-
UkoeHB
Hmm my ballpark would be within 10bits or less
-
ooo123ooo1234[m]
<jeffro256[m]> "Is there anything keeping UkoeHB..." <- I would not do so detailed c++ reference implementation without checking security of cryptography.
-
UkoeHB
ooo123ooo1234[m]: most of the code is not dependent on unproven cryptography (can be fixed if/when a problem is found with probably <10hr work)
-
ooo123ooo1234[m]
UkoeHB: hmm, I think it's possible to find at least 1 example that is very dependent
-
ooo123ooo1234[m]
are you really sure ?
-
ooo123ooo1234[m]
it's literally minimum wrapper on top cryptography
-
UkoeHB
Are you saying ed25519 is unproven? Lol
-
ooo123ooo1234[m]
UkoeHB: ed25519 -> transaction protocol of seraphis -> seraphis lib
-
ooo123ooo1234[m]
I'm about "transaction protocol of seraphis" part
-
UkoeHB
Nah, thanks to good code quality it’s very easy to replace any proof or even the entire squashed enote model.
-
UkoeHB
But that’s a bit too big brain I guess
-
UkoeHB
Let me know if you find an unproven piece that is really so critical though. I won’t hold my breath
-
ooo123ooo1234[m]
<UkoeHB> "Let me know if you find an..." <- In case of cryptography there is presumption of unproven by default, so it must be proven firstly.
-
ooo123ooo1234[m]
<UkoeHB> "Nah, thanks to good code quality..." <- It would be true if proofs can be combined in arbitrary but still secure way
-
ooo123ooo1234[m]
<UkoeHB> "Nah, thanks to good code quality..." <- In general it isn't true. But if your assumptions hold then you're right.
-
UkoeHB
ooo123ooo1234[m]: man I am so tired of you typing words but saying nothing; let me know when you have something to contribute and are done with this bs negotiation charade
-
ooo123ooo1234[m]
<UkoeHB> "ooo123ooo1234: man I am so tired..." <- "bs negotiation charade" every meeting without critical questions looks like this for me
-
ooo123ooo1234[m]
and from spectator point of view it looks like everyone is enjoying participating or pretending that enjoying
-
jeffro256[m]
ooo123ooo1234 Have you ever contributed code to Monero?
-
ooo123ooo1234[m]
jeffro256[m]: And then gratuitous patches will be used as justification to get badge of monero contributor
-
ooo123ooo1234[m]
typical octorberfest particpant
-
jeffro256[m]
Maybe that SHA256 patch is "gratuituous", but you're avoiding the question. For someone who hasn't contributed at all, you seem to be ready to argue with anyone about anything on here, not just myself. Why don't you channel some of that energy into something more productive?
-
ooo123ooo1234[m]
<jeffro256[m]> "ooo123ooo1234 Have you ever..." <- "who hasn't contributed at all" or contributed, how did you check ?
-
ooo123ooo1234[m]
jeffro256[m]: I've mentioned problem in current game design that can't be fixed with patch to github
-
ooo123ooo1234[m]
or few problems
-
ooo123ooo1234[m]
jeffro256[m]: I don't talk in DM, don't even try
-
UkoeHB
ooo123ooo1234[m]: ‘critical questions’ pretty rich coming from someone who thinks he is too important to answer questions. Nice double standard
-
ercdude[m]
I just discovery an "ignore" button on ooo123ooo1234 profile :D
-
ooo123ooo1234[m]
<UkoeHB> "ooo123ooo1234: ‘critical..." <- Important critical questions. Since there is no common goal or at least it isn't announced explicitly then it's possible to treat any question as not important to answer.
-
ooo123ooo1234[m]
This kind of thing isn't possible in proper competition
-
ooo123ooo1234[m]
Or unimportant critical questions should be penalized
-
ercdude[m]
s/discovery/discovered/
-
ooo123ooo1234[m]
<jeffro256[m]> "Maybe that SHA256 patch is "..." <- probably with anyone but not about anything
-
ooo123ooo1234[m]
* <del>probably with, * about anything</del> only anything related to monero development
-
sech1
The amount of effort that went into arguing here is 10x more than needed to review the SHA256 patch from top to bottom. Someone just likes to steal devs time.
-
ooo123ooo1234[m]
sech1: It was balanced with 9 lines patch. And the whole talk was about bigger problem related to many such PRs, not just that one. So you're not right
-
sech1
I am right and you're not right
-
ooo123ooo1234[m]
sech1: explain then
-
sech1
The patch does the needed change and also improves code structure. I don't see any problem with this. Your approach will eventually lead to "hack on top of hack on top of hack on top hack" code
-
sech1
that will need full rewrite
-
ooo123ooo1234[m]
sech1: No. Do you want an example of patch that you would ignore in xmrig or p2pool ?
-
ooo123ooo1234[m]
just because of you don't some changes that are needed since you know code better
-
ooo123ooo1234[m]
* you don't want some changes
-
hyc
the principle of "just fix one thing and don't touch anything else" is sound software engineering. time-tested. used by all large software projects.
-
hyc
"I'm changing this because it's easier for me to read" isn't really a good argument against it.
-
UkoeHB
ooo123ooo1234[m]: It's a matter of basic decency. Oh look, my 'putting up with indecent behavior' meter ran out. Guess you'll need a new account if you want me to read your messages.
-
sech1
I personally wouldn't do this patch the way it was done, I would reduce the changes to the minimum. But I'm totally ok if someone else did tidy up the code in the process.
-
sech1
The patch is not huge and can be reviewed in one go
-
sech1
Bigger refactorings without purpose are no go, but this is below the "no" threshold
-
hyc
sech1: ok, I'm glad you at least take this into consideration and have a threshold in mind
-
hyc
it doesn't sound to me like many others do
-
ooo123ooo1234[m]
sech1: As long as those who are doing bigger changes have capacity. But there are already few examples where you ignore something in xmrig / p2pool
-
hyc
I still personally believe shuffling files and requiring a new header file to be included by multiple source files is out of scope for what the issue was meant to fix.
-
hyc
I mean seriously. it's like saying "hey, this square root function needs to be fixed" and requiring a dozen other files to be updated due to the fix.
-
sech1
I don't merge patches in xmrig. As for p2pool, there are very few examples, you can count them by fingers on one hand. It's either totally broken patches (doesn't compile) or it does something unnecessary. Either way, I comment there and write what's wrong with the patch.
-
ooo123ooo1234[m]
<sech1> "I don't merge patches in xmrig..." <- There are some ignored issues in p2pool, but 0 ignored PRs. And p2pool is mostly solo project at the moment. But for xmrig which isn't your solo project you ignore almost all PRs since they don't help to achieve the goal (performance optimization). It depends on the goal of project / capacity of developers.
-
hyc
sech1 isn't lead dev on xmrig
-
hyc
you seem to be dragging up a bunch of stuff beyond the scope of this conversation now
-
sech1
p2pool issues are not ignored, they're in my backlog :P
-
ooo123ooo1234[m]
hyc: I've tried to bring some context familiar for sech1. If it isn't true then I can stop.
-
ooo123ooo1234[m]
> <@hyc:libera.chat> you seem to be dragging up a bunch of stuff beyond the scope of this conversation now
-
ooo123ooo1234[m]
* I've tried to bring some context familiar for sech1. If it isn't a cache miss then I can stop.
-
ooo123ooo1234[m]
* I've tried to bring some context familiar for sech1. If it is a cache miss then I can stop.
-
sech1
Do you know that all your edits just spam the IRC side?
-
hyc
the main point remains the same: if it ain't broke don't fix it. only touch what is absolutely needed to fix an issue, and nothing more.
-
sech1
-
hyc
doing anything else - reformatting, restructuring, only adds the risk of introducing new bugs.
-
hyc
for this SHA256 issue the patch is simple enough that we can see it doesn't introduce new bugs. but at the same time, the issue is simple enough that the patch ought to be much much smaller.
-
hyc
the only time any refactoring is justified is when needed to add new features
-
jeffro256[m]
I want to guage the general sentiment of the SHA256 PR because this is taking up too much bandwidth:
monero-project/monero #8322#issuecomment-1129118658. If it's generally negative, I'll close it
-
ooo123ooo1234[m]
hyc: +1. If this project would be about some simple like piping data from left to right with simple processing then it would be ok to allow arbitrary changes.
-
ooo123ooo1234[m]
hyc: +1
-
hyc
jeffro256[m]: you understand that the patch is only an example at this point, highlighting the overal issue, development best practices.
-
ooo123ooo1234[m]
jeffro256[m]: sentiment
-
ooo123ooo1234[m]
it isn't about emotions, but about really problems that will come with it
-
ooo123ooo1234[m]
* it isn't about emotions, but about real problems that comes with that kind of changes
-
ooo123ooo1234[m]
sech1: Yes, but IRC isn't compatible with tor.
-
Epsilon
It is. Currently connected via Tor right now.
-
hyc
I think up until recently it was tacitly understood by developers. Now I believe it needs to be explicitly stated.
-
ooo123ooo1234[m]
hyc: +1, 4 years ago I thought the same
-
ooo123ooo1234[m]
> <@hyc:libera.chat> I think up until recently it was tacitly understood by developers. Now I believe it needs to be explicitly stated.
-
ooo123ooo1234[m]
* +1, 4 years ago it was more obvious
-
ooo123ooo1234[m]
since moneromooo was maintaining it vocally
-
ooo123ooo1234[m]
4 years ago I hated it but it looks like it's really necessary
-
ooo123ooo1234[m]
Epsilon: How did you register your account ?
-
Epsilon
ooo123ooo1234[m]: I just followed the guide on Libera's website
libera.chat/guides/connect worked for me back when Libera launched.
-
ooo123ooo1234[m]
Epsilon: registration is prohibited for proxy/tor/anonymous connections
-
ooo123ooo1234[m]
if you used your real ip then your tor connection is useless since you've already leaked your location
-
Epsilon
ooo123ooo1234[m]: I don't remember exactly what I did but iirc I just constantly tried reconnecting with different exit nodes or I used a VPN Its been a while
-
ooo123ooo1234[m]
Epsilon: VPN means you leaked your destination too
-
ooo123ooo1234[m]
anyway matrix account setup is easier to do and there is no problems for tor users
-
ooo123ooo1234[m]
s/problems/problem/
-
hyc
selsta: thanks for your commentary on #8322 that sums it up pretty well.
-
ooo123ooo1234[m]
<UkoeHB> "ooo123ooo1234: It's a matter..." <- It looks like decency about being conformist within social environment. It isn't compatible with solving hard technical problems.
-
ooo123ooo1234[m]
sometimes pretty evil things are treated as moral
-
UkoeHB
ooo123ooo1234[m]: I am a sucker and checked the logs. I guarantee you'll never accomplish anything when no one wants to work with you. Is this the CLSAG issue you've been alluding to? I swear to god reading tea leaves is a mind-boggling waste of time and energy.
usercontent.irccloud-cdn.com/file/t…ijsixQU/0001-clsag-patch-demo.patch
-
UkoeHB
This will never get merged without a concrete proof or argument.
-
sech1
I don't understand what this patch does
-
UkoeHB
sech1: there is some material missing from hashes in the clsag implementation (an oversight)
-
sech1
I see that it adds something to the hash inputs. But the reasoning?
-
UkoeHB
although 'oversight' may be an overstatement; more like 'it could be a little bit more robust'
-
UkoeHB
the reasoning is mostly paranoia, unless mr annoying as hell has a concrete proof
-
UkoeHB
mr annoying as hell has implied there is an issue with clsag, and this is all I could find; if that's the issue, it's up to him to prove it's worth a hf to fix
-
koe000[m]
is the bridge down?
-
UkoeHB
irc side?
-
selsta
no
-
selsta
or not sure, I'm only on irc
-
UkoeHB
irc msgs not showing up on matrix
-
dspringer71[m]
<koe000[m]> "is the bridge down?" <- My first assumption was that someone banned me from monero-dev.
-
dspringer71[m]
hahahaha
-
ooo123ooo1234[m]
<koe000[m]> "is the bridge down?" <- #monero-dev:libera.chat doesn't work at the moment
-
ooo123ooo1234[m]
<UkoeHB> "ooo123ooo1234: I am a sucker and..." <- It could be used as a submission for competition.
-
ooo123ooo1234[m]
<UkoeHB> "ooo123ooo1234: I am a sucker and..." <- Do you really believe that I'll waste someone time with something trivial ?
-
ooo123ooo1234[m]
and unimportant
-
ooo123ooo1234[m]
"I guarantee you'll never accomplish anything when no one wants to work with you." I don't know any way to motivate people to not stick with path of least resistance
-
ooo123ooo1234[m]
"I swear to god reading tea leaves is a mind-boggling waste of time and energy." If there would be at least one rule that I could use as justification that my work is hard and worth something
-
ooo123ooo1234[m]
then i would talk somehow differently
-
ooo123ooo1234[m]
<UkoeHB> "ooo123ooo1234: I am a sucker and..." <- You said it monero-research-lab that some pushback helped to do something. Is this patch a result of pushback from my side too ? Was it written solely by you ?
-
ooo123ooo1234[m]
s/it/in/
-
vtnerd_
looks like 8149 needs review again? got a lot going (haven't bailed, promise), working on carving out some time shortly on that
-
UkoeHB
vtnerd_: yeah that would be awesome
-
selsta
hyc: that should be the final version of the patch now, if you can take one last look
monero-project/monero #8340
-
hyc
selsta: just 1 trivial change. otherwise looks ok
-
selsta
hyc: done
-
hyc
looks good