search

thor314 / pebble-stark

A community-developed re-implementation of the Starkware Stone Prover
70 stars 6 forks source link

migrate randomness #6

Open thor314 opened 10 months ago

thor314 commented 10 months ago

migrate randomness. Relatively small part of the code-base, will rely on finding a decent blake2 hashing library.

agnxsh commented 10 months ago

https://docs.rs/blake2/latest/blake2/ for blake, any thoughts on this? https://docs.rs/blake2/latest/blake2/type.Blake2s256.html also this is a similar type to https://github.com/starkware-libs/stone-prover/blob/main/src/starkware/crypt_tools/blake2s.inl#L93-L101 as far i can understand for now

thor314 commented 10 months ago

That seems reasonable, it would also be good to keep the randomness generator generic over the hash function of choice, so that we may potentially support other hashes in the future if you'd like to tackle that