Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: new Poseidon2 relations #3406
feat: new Poseidon2 relations #3406
Changes from 24 commits
e1c732d
ea12f99
95e6572
8c30828
86cf018
6dd2511
3dbd8df
3631294
910b303
c7b4157
7bcee20
752bd14
7d0fe79
20a7eb2
f09ed91
d679581
35ab7d9
5e94698
14157c1
e04037e
6da3335
0c961b1
8ef04db
b144cce
80958c5
a3c8eed
52d260b
a53e861
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you please explain what exactly is being calculated here in terms of the notation of the paper? I say more about that below. I think the intent is the external round function of Section 6 of the paper, but I'm worried there's a discrepancy: looks like you're using$M_{\mathcal{E}} = M_4$ , but to my eyes it looks like there might be a missing 2, ie should we have $M_{\mathcal{E}} = 2M_4$ ? I didn't do a thorough read of the paper--did you look into this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Name clash here: you mean l r o 4 not 1 2 3 4 right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there's a name clash everywhere and I don't know whether we want to use 1 2 3 4 or l r o 4. It looked like we use l r o 4 in actual code and 1 2 3 4 in comments, so thats the convention I've been following.