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.

Sign up for GitHub

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

Jump to bottom

Show failing constraints in public functions #1813

Closed
Tracked by #1224
sirasistant opened this issue Aug 25, 2023 · 1 comment · Fixed by #1853
Closed
Tracked by #1224

Show failing constraints in public functions #1813

sirasistant opened this issue Aug 25, 2023 · 1 comment · Fixed by #1853
Assignees
@sirasistant
Labels
A-ux/devex Area: relates to external ux / devex. (Users typically are devs) (See also A-internal-devex)

Comments

@sirasistant
Copy link
Collaborator

sirasistant commented Aug 25, 2023 •
edited
Loading

Public functions are executed by the sequencer who doesn't have any debug metadata/abi about the function. We can, however, simulate locally the public part of the functions if the user requests so, which will be a necessary feature for gas metering with fees. When executing locally we do have access to the debug metadata.
@sirasistant sirasistant mentioned this issue Aug 25, 2023
Closed
@github-project-automation github-project-automation bot moved this to Todo in A3 Aug 25, 2023
@sirasistant sirasistant self-assigned this Aug 25, 2023
@iAmMichaelConnor iAmMichaelConnor added A-ux/devex Area: relates to external ux / devex. (Users typically are devs) (See also A-internal-devex) debugging labels Aug 25, 2023
@iAmMichaelConnor iAmMichaelConnor added this to the 📢 Initial Public Sandbox Release milestone Aug 25, 2023
@sirasistant
Copy link
Collaborator Author

Related to #1377

@sirasistant sirasistant mentioned this issue Aug 29, 2023
Merged
4 tasks
sirasistant added a commit that referenced this issue Aug 30, 2023
@sirasistant
feat: Simulate enqueued public functions and locate failing constrain…
a065fd5 
…ts on them (#1853)

Resolves #1813 and
#1377
- Simulates the public part of transactions before submission to the
node.
- Introduces a SimulationError to show complete call stacks (aztec
function calls and noir function calls)
- Reconstructs public function errors to show noir call stacks on public
functions after simulation
- Fixes a bug where the public tx processor was mutating Tx objects in
place.
@github-project-automation github-project-automation bot moved this from Todo to Done in A3 Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sirasistant sirasistant

Labels
A-ux/devex Area: relates to external ux / devex. (Users typically are devs) (See also A-internal-devex)
Projects
A3
Archived in project
Milestone
No milestone
2 participants
@sirasistant @iAmMichaelConnor