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.

Sign up for GitHub

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

Jump to bottom

Aviad/blake2s cpu #580

Closed
wants to merge 62 commits into from
Closed

Aviad/blake2s cpu #580

wants to merge 62 commits into from

Conversation

aviadingo
Copy link
Contributor

Describe the changes

  1. Added template Hasher class.
  2. Added blake2s cpu implementation which inherits from the general-purpose hasher and overrides the run_hash function.
  3. Hash many is currently a dummy implementation runing the hashes in a loop
  4. Included tests for the batched version and the single version using input vectors with variable length.

aviadingo and others added 15 commits September 1, 2024 18:14
@aviadingo
typo fix
654866d
@yshekel @ShanieWinitz
@Koren-Brand @emirsoyturk @mickeyasa @ChickenLover @jeremyfelder @HadarIngonyama @nonam3e @Otsar-Raikou
ICICLE V3 (#526)
5ef9ed1 
Co-authored-by: Shanie Winitz <[email protected]>
Co-authored-by: Koren-Brand <[email protected]>
Co-authored-by: emirsoyturk <[email protected]>
Co-authored-by: Miki <[email protected]>
Co-authored-by: ChickenLover <[email protected]>
Co-authored-by: Jeremy Felder <[email protected]>
Co-authored-by: hadaringonyama <[email protected]>
Co-authored-by: nonam3e <[email protected]>
Co-authored-by: Otsar <[email protected]>
@Otsar-Raikou
updated docs (#602)
0b53c1c
@yshekel
Bump rust crates' version
94aa03b 
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]

Generated by cargo-workspaces
@Otsar-Raikou
broken link
7608ffa
@Otsar-Raikou
fixed typo
6d392b6
@Otsar-Raikou
links and typos (#603)
ca745f3 
## Describe the changes

This PR...

## Linked Issues

Resolves #
@Otsar-Raikou
Fixed image NEW (#606)
c5fec85
@yshekel
introduce base hash C++ API
1c57852
@yshekel
add keccak256,512 and sha3-256,512 frontend APIs and CPU mock impleme…
d399cee 
…ntation
@yshekel
keccak C api
985ea45
@yshekel
minor
818b5f9
@yshekel
add merkle tree API
2e96da3
@yshekel
cpu backend for merkle tree
866a988
@aviadingo
Merge remote-tracking branch 'origin/yshekel/V3' into aviad/blake2s_cpu
e50b557
@ChickenLover ChickenLover changed the base branch from yshekel/V3 to main September 8, 2024 08:48
@ChickenLover ChickenLover changed the base branch from main to yshekel/V3 September 8, 2024 08:48
@ChickenLover
Copy link
Contributor

@aviadingo there is no more yshekel/V3 afaik. I guess you need to rebase to main

@ChickenLover
Copy link
Contributor

ChickenLover commented Sep 9, 2024 •
edited
Loading

@aviadingo is this still relevant? can we close this?

@aviadingo aviadingo closed this Sep 9, 2024
@aviadingo
Copy link
Contributor Author

replaced with PR610

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChickenLover ChickenLover ChickenLover left review comments

@yshekel yshekel Awaiting requested review from yshekel

@mickeyasa mickeyasa Awaiting requested review from mickeyasa

@HadarIngonyama HadarIngonyama Awaiting requested review from HadarIngonyama

@danny-shterman danny-shterman Awaiting requested review from danny-shterman

@LeonHibnik LeonHibnik Awaiting requested review from LeonHibnik

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@aviadingo @ChickenLover @yshekel @Otsar-Raikou