arkworks-rs / crypto-primitives

Interfaces and implementations of cryptographic primitives, along with R1CS constraints for them
https://www.arkworks.rs
Apache License 2.0
165 stars 79 forks source link

Fix bug in `Absorb` and update `Cargo.toml` #132

Closed autquis closed 7 months ago

autquis commented 7 months ago

With the current implementation of Poseidon, when trying to absorb a field element from a different field of sponge, there is no error and the developer would think everything is right :) The unwrap helps with making panics.

Also, I changed the repos from curves to algebra.


Before we can merge this PR, please make sure that all the following items have been checked off. If any of the checklist items are not applicable, please leave them but write a little note why.

z-tech commented 7 months ago

Sweet! Thx.

autquis commented 7 months ago

What is happening to this merge queue? :D