-
Notifications
You must be signed in to change notification settings - Fork 98
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
Emir/poseidon #626
base: main
Are you sure you want to change the base?
Emir/poseidon #626
Conversation
unsigned int alpha; ///< Sbox power. | ||
unsigned int partial_rounds; ///< Number of upper full rounds of a single hash. | ||
unsigned int full_rounds_half; ///< Number of partial rounds of a single hash. | ||
S* round_constants; ///< Round constants (both of the full and the partial rounds). The order of the constants in the memory is according to the rounds order. |
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.
is the matrices dims known and fixed? if yes, can't we simply define them and then copy the entire struct to GPU as is without an additional indirection
@@ -165,32 +165,1090 @@ TEST_F(HashApiTest, MerkleTree) | |||
ASSERT_TRUE(verification_valid); | |||
} | |||
|
|||
#ifdef POSEIDON | |||
// #ifdef POSEIDON |
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.
why did you remove? we can disable poseidon
|
||
PoseidonConstantsInitOptions<scalar_t> options = {}; | ||
|
||
unsigned char poseidon_constants_2[] = { |
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.
can you maybe randomize it? if we compare CPU to CUDA it should work with any constants right? it should be a more robust test too.
No description provided.