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

chore: Noir.js interface accepts a stuct of public inputs and proof #2992

Merged
merged 4 commits into from
Oct 5, 2023

Conversation

kevaundray
Copy link
Contributor

Description

This PR splits the public inputs and proof into separate components and modifies the barretenberg implementation to account for this.

This doesn't require a lot of change on the API because from typescripts perspective, we've only done a type change.

Problem*

Resolves

Summary*

Documentation

  • This PR requires documentation updates when merged.

    • I will submit a noir-lang/docs PR.
    • I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

@kevaundray kevaundray changed the title chore: Interface accepts a stuct of public inputs and proof chore: Noir.js interface accepts a stuct of public inputs and proof Oct 5, 2023
@kevaundray kevaundray force-pushed the kw/noir-js-public-inputs branch from 8b03e9a to fe32f77 Compare October 5, 2023 11:39
Copy link
Contributor

@kobyhallx kobyhallx left a comment

Choose a reason for hiding this comment

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

👍

@kobyhallx kobyhallx added this pull request to the merge queue Oct 5, 2023
Merged via the queue into master with commit 6796bec Oct 5, 2023
26 checks passed
@kobyhallx kobyhallx deleted the kw/noir-js-public-inputs branch October 5, 2023 13:55
TomAFrench added a commit that referenced this pull request Oct 6, 2023
* master: (31 commits)
  chore: run tests for `noir_wasm` in node under default resolver (#3020)
  chore: align versions for js packages (#3019)
  fix: Fix aztec library after nargo fmt (#3014)
  chore: simplify ES workspace build and publish (#2989)
  fix: fix method `program_counter`, change method signature (#3012)
  fix: ACIR optimizer should update assertion messages (#3010)
  fix: check for overflow with hexadecimal inputs (#3004)
  feat(traits): Add trait impl for buildin types (#2964)
  feat(traits): added checks for duplicated trait associated items (types, consts, functions) (#2927)
  chore(formatter): trim spaces after opening brace (#3005)
  feat: format infix expressions (#3001)
  feat: provide formatting subcommand  (#2640)
  chore: Noir.js interface accepts a stuct of public inputs and proof (#2992)
  fix: Transform hir before type checks  (#2994)
  fix!: Make for loops a statement (#2975)
  feat: Add aztec selectors for event structs (#2983)
  fix(traits)!: trait functions with a default implementation must not be followed by a semicolon (#2987)
  chore: strip initialization of unused memory blocks from ACIR (#2985)
  chore: clippy fix (#2986)
  chore(ci): replace webdriver with playwright (#2988)
  ...
@Savio-Sou Savio-Sou added js Noir's JavaScript packages enhancement New feature or request labels Oct 12, 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 js Noir's JavaScript packages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants