feat(ci): Publish binaries for noir-profiler #7443

vezenovm · 2025-02-19T16:04:51Z

Description

Problem*

Resolves #7162

Working towards #7438 but just for the noir-profiler.

Summary*

I added the noir-profiler to the publish-nargo job. I build the noir-profiler along with nargo, package its artifacts, and upload it. This is all done in a similar way to the existing nargo job.

Additional Context

Documentation*

Check one:

No documentation needed.
Documentation included in this PR.
[For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

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

TomAFrench · 2025-02-19T17:03:46Z

I was thinking that we can publish all of the binaries for a given OS/architecture in single tarball. This makes it easier for us to just unzip them all into the relevant directory when installing rather than needing to make N different network requests. (this is how foundry does it https://github.com/foundry-rs/foundry/releases/tag/stable)

We can keep a copy of the nargo binary separate as well for backwards compatibility.

vezenovm · 2025-02-19T17:06:39Z

I was thinking that we can publish all of the binaries for a given OS/architecture in single tarball

I will switch to that. I wasn't sure if we wanted to keep a separate nargo binary for now.

github-actions

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Compilation Time'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite	Current: `9407d13`	Previous: `d0d04e8`	Ratio
`rollup-block-root-single-tx`	`113` s	`93.6` s	`1.21`

This comment was automatically generated by workflow using github-action-benchmark.

CC: @TomAFrench

github-actions

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Test Suite Duration'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite	Current: `9407d13`	Previous: `d0d04e8`	Ratio
`AztecProtocol_aztec-packages_noir-projects_noir-protocol-circuits_crates_blob`	`62` s	`51` s	`1.22`

This comment was automatically generated by workflow using github-action-benchmark.

CC: @TomAFrench

.github/workflows/publish-nargo.yml

TomAFrench · 2025-02-20T15:48:32Z

The main switch is that I moved away to using tar -czf for packaging the artifacts.

Why did you made this switch?

vezenovm · 2025-02-20T16:08:42Z

Why did you made this switch?

Mainly that I found it much more readable and simple. I was thinking that while 7z can provide more compression we are still using gzip so the performance should be similar.

I did a comparison just now for only the nargo binary:
tar -czf resulted in 14948270 bytes.
7z resulted in 14383145 bytes.

As 7z does result in a small binary (although not by much), I'll just switch it back.

.github/workflows/publish-nargo.yml

TomAFrench

LGTM

chore: remove `disable_macros` compile option (noir-lang/noir#7468) chore(ci): add workflow to automate bumping aztec-packages commit (noir-lang/noir#7465) chore: Release Noir(1.0.0-beta.3) (noir-lang/noir#7346) chore(ci): Missing dash in profiler command argument (noir-lang/noir#7467) feat(experimental): show macro errors where they happen (noir-lang/noir#7333) feat: optimize FieldElement::num_bits (noir-lang/noir#7147) chore(profiler): Docs on profiler command and more complete error reporting (noir-lang/noir#7436) feat(ci): Release noir-inspector in binaries (noir-lang/noir#7464) chore(docs): Noir Profiler external documentation (noir-lang/noir#7457) feat(ci): Publish binaries for noir-profiler (noir-lang/noir#7443) chore: Copy #7387 docs into v1.0.0-beta.2 versioned_docs (noir-lang/noir#7458) fix: prevent incorrect ACIRgen caused by noop truncations (noir-lang/noir#7456) feat: add native `u128` type (noir-lang/noir#7301) chore: standardize that doc comments on top of statements and expression are allowed but warn (noir-lang/noir#7450) fix: don't let nargo fmt produce multiple trailing newlines (noir-lang/noir#7444)

* master: (89 commits) chore: bump external pinned commits (#7472) chore: remove `disable_macros` compile option (#7468) chore(ci): add workflow to automate bumping aztec-packages commit (#7465) chore: Release Noir(1.0.0-beta.3) (#7346) chore(ci): Missing dash in profiler command argument (#7467) feat(experimental): show macro errors where they happen (#7333) feat: optimize FieldElement::num_bits (#7147) chore(profiler): Docs on profiler command and more complete error reporting (#7436) feat(ci): Release noir-inspector in binaries (#7464) chore(docs): Noir Profiler external documentation (#7457) feat(ci): Publish binaries for noir-profiler (#7443) chore: Copy #7387 docs into v1.0.0-beta.2 versioned_docs (#7458) fix: prevent incorrect ACIRgen caused by noop truncations (#7456) feat: add native `u128` type (#7301) chore: standardize that doc comments on top of statements and expression are allowed but warn (#7450) fix: don't let nargo fmt produce multiple trailing newlines (#7444) feat(cli): add noir-execute binary (#7384) chore!: make `ResolverError::OracleMarkedAsConstrained` into a full error (#7426) chore: simplify reports (#7421) fix: do not discard negative sign from field literals in comptime interpreter (#7439) ...

fix: don't panic when shifting too much (noir-lang/noir#7429) chore: bump external pinned commits (noir-lang/noir#7472) chore: remove `disable_macros` compile option (noir-lang/noir#7468) chore(ci): add workflow to automate bumping aztec-packages commit (noir-lang/noir#7465) chore: Release Noir(1.0.0-beta.3) (noir-lang/noir#7346) chore(ci): Missing dash in profiler command argument (noir-lang/noir#7467) feat(experimental): show macro errors where they happen (noir-lang/noir#7333) feat: optimize FieldElement::num_bits (noir-lang/noir#7147) chore(profiler): Docs on profiler command and more complete error reporting (noir-lang/noir#7436) feat(ci): Release noir-inspector in binaries (noir-lang/noir#7464) chore(docs): Noir Profiler external documentation (noir-lang/noir#7457) feat(ci): Publish binaries for noir-profiler (noir-lang/noir#7443) chore: Copy #7387 docs into v1.0.0-beta.2 versioned_docs (noir-lang/noir#7458) fix: prevent incorrect ACIRgen caused by noop truncations (noir-lang/noir#7456) feat: add native `u128` type (noir-lang/noir#7301) chore: standardize that doc comments on top of statements and expression are allowed but warn (noir-lang/noir#7450) fix: don't let nargo fmt produce multiple trailing newlines (noir-lang/noir#7444)

Automated pull of development from the [noir](https://github.com/noir-lang/noir) programming language, a dependency of Aztec. BEGIN_COMMIT_OVERRIDE feat: Sync from aztec-packages (noir-lang/noir#7474) fix: don't panic when shifting too much (noir-lang/noir#7429) chore: bump external pinned commits (noir-lang/noir#7472) chore: remove `disable_macros` compile option (noir-lang/noir#7468) chore(ci): add workflow to automate bumping aztec-packages commit (noir-lang/noir#7465) chore: Release Noir(1.0.0-beta.3) (noir-lang/noir#7346) chore(ci): Missing dash in profiler command argument (noir-lang/noir#7467) feat(experimental): show macro errors where they happen (noir-lang/noir#7333) feat: optimize FieldElement::num_bits (noir-lang/noir#7147) chore(profiler): Docs on profiler command and more complete error reporting (noir-lang/noir#7436) feat(ci): Release noir-inspector in binaries (noir-lang/noir#7464) chore(docs): Noir Profiler external documentation (noir-lang/noir#7457) feat(ci): Publish binaries for noir-profiler (noir-lang/noir#7443) chore: Copy #7387 docs into v1.0.0-beta.2 versioned_docs (noir-lang/noir#7458) fix: prevent incorrect ACIRgen caused by noop truncations (noir-lang/noir#7456) feat: add native `u128` type (noir-lang/noir#7301) chore: standardize that doc comments on top of statements and expression are allowed but warn (noir-lang/noir#7450) fix: don't let nargo fmt produce multiple trailing newlines (noir-lang/noir#7444) END_COMMIT_OVERRIDE --------- Co-authored-by: Tom French <[email protected]>

vezenovm added 2 commits February 19, 2025 16:03

publish binaries for noir-profiler

13e6668

use correct package name for noir_profiler in linux build

0baf8f8

vezenovm marked this pull request as ready for review February 19, 2025 16:17

vezenovm requested a review from TomAFrench February 19, 2025 16:17

vezenovm mentioned this pull request Feb 19, 2025

Remove separate nargo binary after noirup is updated #7445

Open

put all binaries in a single tarball

9407d13

github-actions bot reviewed Feb 19, 2025

View reviewed changes

Merge branch 'master' into mv/publish-profiler-binaries

b921fdf

vezenovm mentioned this pull request Feb 19, 2025

feat: Install noir-profiler and noir-inspector noir-lang/noirup#46

Merged

2 tasks

TomAFrench reviewed Feb 20, 2025

View reviewed changes

.github/workflows/publish-nargo.yml Outdated Show resolved Hide resolved

TomAFrench reviewed Feb 20, 2025

View reviewed changes

.github/workflows/publish-nargo.yml Outdated Show resolved Hide resolved

.github/workflows/publish-nargo.yml Outdated Show resolved Hide resolved

.github/workflows/publish-nargo.yml Outdated Show resolved Hide resolved

.github/workflows/publish-nargo.yml Outdated Show resolved Hide resolved

Apply suggestions from code review

9b1e3be

vezenovm commented Feb 20, 2025

View reviewed changes

.github/workflows/publish-nargo.yml Outdated Show resolved Hide resolved

vezenovm added 3 commits February 20, 2025 11:09

Update .github/workflows/publish-nargo.yml

7232bc9

do not upload nargo artifacts

cd870e2

Merge branch 'master' into mv/publish-profiler-binaries

8f68fdf

TomAFrench approved these changes Feb 20, 2025

View reviewed changes

TomAFrench added this pull request to the merge queue Feb 20, 2025

Merged via the queue into master with commit af5e4cd Feb 20, 2025
102 checks passed

TomAFrench deleted the mv/publish-profiler-binaries branch February 20, 2025 17:55

noirwhal mentioned this pull request Feb 20, 2025

chore: Release Noir(1.0.0-beta.3) #7346

Merged

AztecBot mentioned this pull request Feb 20, 2025

feat: Sync from noir AztecProtocol/aztec-packages#12129

Closed

AztecBot mentioned this pull request Feb 20, 2025

feat: Sync from noir AztecProtocol/aztec-packages#12176

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(ci): Publish binaries for noir-profiler #7443

feat(ci): Publish binaries for noir-profiler #7443

vezenovm commented Feb 19, 2025 •

edited

Loading

TomAFrench commented Feb 19, 2025

vezenovm commented Feb 19, 2025

github-actions bot left a comment

github-actions bot left a comment

TomAFrench commented Feb 20, 2025

vezenovm commented Feb 20, 2025

TomAFrench left a comment

feat(ci): Publish binaries for noir-profiler #7443

feat(ci): Publish binaries for noir-profiler #7443

Conversation

vezenovm commented Feb 19, 2025 • edited Loading

Description

Problem*

Summary*

Additional Context

Documentation*

PR Checklist*

TomAFrench commented Feb 19, 2025

vezenovm commented Feb 19, 2025

github-actions bot left a comment

Choose a reason for hiding this comment

⚠️ Performance Alert ⚠️

github-actions bot left a comment

Choose a reason for hiding this comment

⚠️ Performance Alert ⚠️

TomAFrench commented Feb 20, 2025

vezenovm commented Feb 20, 2025

TomAFrench left a comment

Choose a reason for hiding this comment

vezenovm commented Feb 19, 2025 •

edited

Loading