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

Handle Honk recursive aggregation in NoirJS #5661

Open
vezenovm opened this issue Aug 1, 2024 · 3 comments
Open

Handle Honk recursive aggregation in NoirJS #5661

vezenovm opened this issue Aug 1, 2024 · 3 comments
Labels
enhancement New feature or request js Noir's JavaScript packages

Comments

@vezenovm
Copy link
Contributor

vezenovm commented Aug 1, 2024

Problem

After #5656 NoirJS will be compatible with UltraHonk. However, only proving and verification was included in that PR. We should add support for recursive aggregation as that is supported in UltraHonk.

Happy Case

We should have the same test that we have in onchain_recursive_verification.test.ts for UltraPlonk for UltraHonk.

Workaround

None

Workaround Description

No response

Additional Context

No response

Project Impact

Nice-to-have

Blocker Context

No response

Would you like to submit a PR for this Issue?

None

Support Needs

No response

@vezenovm vezenovm added enhancement New feature or request js Noir's JavaScript packages labels Aug 1, 2024
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Noir Aug 1, 2024
@signorecello
Copy link
Contributor

Does this close this issue @TomAFrench? Have a dev looking into this and can't proceed but I'm not sure

@signorecello
Copy link
Contributor

Seems like it's not, not for the recursive artifacts at least. I'm giving it a shot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request js Noir's JavaScript packages
Projects
Status: 📋 Backlog
Development

No branches or pull requests

4 participants