Skip to content
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

Add missing credit #1865

Merged
merged 8 commits into from
May 20, 2024
Merged

Add missing credit #1865

merged 8 commits into from
May 20, 2024

Conversation

tzerrell
Copy link
Member

We mistakenly omitted credit to @nhukc for the original draft of the Poseidon2 code. This PR adds this missing credit, both via GitHub (as a coauthor) and via inline comment.

3lkn and others added 8 commits May 20, 2024 11:36
---------

Co-authored-by: Tim Zerrell <tim.zerrell@risczero.com>
Co-authored-by: Paul <paul.gafni@gmail.com>

Fix names of voting functions (#917)

This PR fixes an incorrect name for the voting functions on the Governor
interface.
Poseidon2 is an arithmetization-oriented hash function based on the
Poseidon hash algorithm.  Poseidon2 is 10% more performant, more memory
efficient, and simpler to implement than Poseidon.
This boosts performance of poseidon2 from 10% to 376% faster than poseidon.
@tzerrell tzerrell requested a review from jbruestle May 20, 2024 18:58
@tzerrell tzerrell self-assigned this May 20, 2024
Copy link

vercel bot commented May 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs-website ✅ Ready (Inspect) Visit Preview May 20, 2024 6:58pm
reports-and-benchmarks ✅ Ready (Inspect) Visit Preview May 20, 2024 6:58pm

tzerrell referenced this pull request May 20, 2024
Co-authored-by: Tim Zerrell <tim.zerrell@risczero.com>
@tzerrell tzerrell merged commit 0278191 into main May 20, 2024
38 of 40 checks passed
@tzerrell tzerrell deleted the tzerrell/credit-nhuck branch May 20, 2024 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants