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

Supports struct inputs/outputs to Brillig #1641

Closed
vezenovm opened this issue Jun 12, 2023 · 2 comments
Closed

Supports struct inputs/outputs to Brillig #1641

vezenovm opened this issue Jun 12, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@vezenovm
Copy link
Contributor

Problem

Brillig inputs are currently limited to singular values for inputs/outputs and are being expanded to support arrays as well in this PR (#1630). However, the PR only supports singular and not nested arrays. Structures are flattened into arrays in SSA thus we need to be able to support nested arrays in order to enable struct input/outputs.

Happy Case

We can use a struct as input to unconstrained fns as well as output to unconstrained fns.

Alternatives Considered

N/A

Additional Context

No response

Would you like to submit a PR for this Issue?

No

Support Needs

No response

@kevaundray
Copy link
Contributor

@vezenovm what is the status of this?

@vezenovm
Copy link
Contributor Author

vezenovm commented Aug 2, 2023

@vezenovm vezenovm closed this as completed Aug 2, 2023
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Noir Aug 2, 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

No branches or pull requests

3 participants