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

Avoid redundant computation in Protogalaxy #1349

Conversation

jeanmon
Copy link

@jeanmon jeanmon commented May 6, 2024

This is porting changes made in aztec-package PR #5844

Maddiaa0 and others added 30 commits October 10, 2023 10:25
* feat: just pil

* fix: do not generate trace builder .cpp
* feat: just pil

* fix: do not generate trace builder .cpp

* hack: allow specifying output name in pil cli

* merge fix
* feat: smaller bberg binary

* fix: missed assignment
* fix: remove brillig to powdr

* chore: refactor file writer / circuit builder
* fix: refactor to use traits

* fix
* chore: merge upstream

* fix: merge clean
* fix: remove first requirement

* fix: remove last requirement

* fix

* fix
* fix: resync master

* fix: reenable opt
* fix: resync master

* fix: reenable opt

* fix: dynamic naming
Maddiaa0 and others added 28 commits February 9, 2024 14:37
Any generated file names use a snake case version of the prefix
…folder

Use snake case for the folder containing permutation and lookup
* Align namespaces with simplification performed in Barretenberg

* fix: use lhs + rhs in perm

* todos

---------

Co-authored-by: jeanmon <[email protected]>
…ilename

Generated permutation and lookup filenames are lower-cased
Comment related to ProverPolynomials
feat: support large literals in FF-limb decomp
* feat: circuit builder does not need to compute inverses, prover does so

* fix: integration cleanup

* chore: sweep

* chore: clippy and fmt
@jeanmon jeanmon closed this May 6, 2024
@jeanmon
Copy link
Author

jeanmon commented May 6, 2024

Sorry for the noise. This PR was intended for a forked powdr repo of Aztec and not the upstream project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants