Skip to content

Commit

Permalink
chore: Reset noir-gates-diff report on master (#6003)
Browse files Browse the repository at this point in the history
Similarly to noir-lang/noir#4878 the gates diff
report got corrupted following changes to `nargo info`. This PR simply
resets the gates diff so it is expected to see really large gates
differences. You can look into the Noir PR for more info. There will be
a followup which sets back to the correct noir-gates-diff commit.
Without this reset we will not be able to get accurate gate diffs based
off of master.
  • Loading branch information
vezenovm authored and TomAFrench committed Apr 24, 2024
1 parent b366be4 commit b129602
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/protocol-circuits-gate-diff.yml
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ jobs:

- name: Compare gates reports
id: gates_diff
uses: TomAFrench/noir-gates-diff@df05f34e2ab275ddc4f2cac065df1c88f8a05e5d
uses: vezenovm/noir-gates-diff@45e9c9a21deb236fa7f38138b42b33ddaf7c0985
with:
report: protocol_circuits_report.json
summaryQuantile: 0 # Display any diff in gate count
Expand Down

0 comments on commit b129602

Please sign in to comment.