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.

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

refactor(circuits): remove duplicate constraints #1644

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

ctrlc03
Copy link
Collaborator

@ctrlc03 ctrlc03 commented Jul 15, 2024

Description

Remove duplicate constraints on circuits by re-using signals and removing unnecessary checks.

Related issue(s)

fix #305

Confirmation

Copy link

vercel bot commented Jul 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
maci-website ✅ Ready (Inspect) Visit Preview Jul 16, 2024 10:58am

@ctrlc03 ctrlc03 self-assigned this Jul 15, 2024
@ctrlc03 ctrlc03 added the circuits Related to the Circom circuits label Jul 15, 2024
Copy link
Collaborator

@kittybest kittybest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just left some comments

circuits/circom/utils/non-qv/messageValidator.circom Outdated Show resolved Hide resolved
circuits/circom/utils/non-qv/messageValidator.circom Outdated Show resolved Hide resolved
circuits/circom/utils/qv/messageValidator.circom Outdated Show resolved Hide resolved
circuits/circom/utils/qv/messageValidator.circom Outdated Show resolved Hide resolved
Copy link
Collaborator

@0xmad 0xmad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ctrlc03 thanks!

@ctrlc03 ctrlc03 force-pushed the refactor/optimize-circuits branch from a9300d0 to 69d31e2 Compare July 16, 2024 10:56
@ctrlc03 ctrlc03 merged commit cd7ea49 into dev Jul 16, 2024
22 checks passed
@ctrlc03 ctrlc03 deleted the refactor/optimize-circuits branch July 16, 2024 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
circuits Related to the Circom circuits
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Duplicate constraints on process message circuit
3 participants