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

Nargo should cache CRS instead of leaving to backends #1302

Closed
1 task done
phated opened this issue May 5, 2023 · 0 comments · Fixed by #1339
Closed
1 task done

Nargo should cache CRS instead of leaving to backends #1302

phated opened this issue May 5, 2023 · 0 comments · Fixed by #1339
Assignees
Labels
enhancement New feature or request

Comments

@phated
Copy link
Contributor

phated commented May 5, 2023

Problem

Currently, backends are expected to implement caching, but aztec_backend puts artifacts into nargo-controlled directories. Nargo CLI should handle the caching of CRS files.

Proposed solution

Implement a backend trait such as noir-lang/acvm#231 and implement it in aztec_backend and then consume the new API in Nargo and handle caching (via filesystem, or whatever) inside Nargo CLI.

Alternatives considered

No response

Additional context

No response

Submission Checklist

  • Once I hit submit, I will assign this issue to the Project Board with the appropriate tags.
@phated phated added the enhancement New feature or request label May 5, 2023
@phated phated self-assigned this May 5, 2023
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Noir May 5, 2023
@phated phated moved this from 📋 Backlog to 🤔 Grooming in Noir May 5, 2023
@phated phated moved this from 🤔 Grooming to 🏗 In progress in Noir May 12, 2023
@phated phated linked a pull request May 22, 2023 that will close this issue
6 tasks
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Noir May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant