-
Notifications
You must be signed in to change notification settings - Fork 148
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
restructure imt folders in the circuit package #1419
Comments
Hey @ctrlc03, would love to work on this ? |
sure thing, can assign you @0xshikhar thanks |
Hey @0xshikhar, are you still working on this? want to see if we can add it to the hackthoberfest |
Yup, started working on this. Was quite busy due to ETHKL |
Hey @ctrlc03, was working on it and ran the current setup successfully. Just want to confirm - am I working in the right direction : it need restructuring of |
Hi @0xshikhar - this would be for the circuits folder not the ts code: https://github.com/privacy-scaling-explorations/maci/tree/dev/packages/circuits/circom/trees |
Hi @ctrlc03, i was thinking that it can be incremental merkle tree, but it has only 1 folder. ? Could you clarify the tasks for this issue in detail? Any additional context would be appreciated. Thanks! |
@0xshikhar I believe the original idea of this issue was to have one file for each template inside that folder. So if you open https://github.com/privacy-scaling-explorations/maci/blob/dev/packages/circuits/circom/trees/incrementalMerkleTree.circom you can see there's different templates. |
Restructure folders to have one file per each template
The text was updated successfully, but these errors were encountered: