Skip to content

Conversation

Maddiaa0
Copy link
Member

@Maddiaa0 Maddiaa0 commented Jun 17, 2025

Overview

The first part of this pr includes making infra to store the output of the nargo info command, so that it can be compared against main when making changes to the circuits

@Maddiaa0 Maddiaa0 changed the title feat(ci): bench gate counts wip: feat(ci): bench gate counts Jun 17, 2025
@Maddiaa0 Maddiaa0 force-pushed the md/bench-opcode-counts branch 2 times, most recently from 9f22aaa to 7a548ee Compare June 17, 2025 15:45
@Maddiaa0 Maddiaa0 force-pushed the md/bench-opcode-counts branch from 7a548ee to a9c4b06 Compare June 17, 2025 15:46
@Maddiaa0 Maddiaa0 force-pushed the md/bench-opcode-counts branch from c14237d to bf0805e Compare June 17, 2025 16:11
@michaelelliot
Copy link
Contributor

Love this idea! Although unfortunately it ended up tripling the CI run time. It's also currently only checking the circuits used in integration tests, so we're not seeing gate counts for the full set, which would take even longer to process.

One thing that could help: caching gate counts to disk using a filename based on a hash of the flattened Noir code, along with the nargo and bb versions. That way, if nothing has changed we can skip recalculating the gate count and hopefully speed up CI quite a bit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants