Skip to content

perf: Avoid cloning signatures for simple port checks - #3149

Merged
aborgna-q merged 8 commits into
mainfrom
ab/envelope-perf-2
Sep 2, 2026
Merged

perf: Avoid cloning signatures for simple port checks#3149
aborgna-q merged 8 commits into
mainfrom
ab/envelope-perf-2

Review comment

238ba09
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 2, 2026 in 0s

81.61% (+0.06%) compared to e609c66

View this Pull Request on Codecov

81.61% (+0.06%) compared to e609c66

Details

Codecov Report

❌ Patch coverage is 93.50000% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.61%. Comparing base (e609c66) to head (238ba09).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
hugr-core/src/ops.rs 95.18% 3 Missing and 1 partial ⚠️
hugr-core/src/ops/sum.rs 81.25% 2 Missing and 1 partial ⚠️
hugr-core/src/ops/controlflow.rs 93.54% 2 Missing ⚠️
hugr-core/src/ops/dataflow.rs 96.55% 2 Missing ⚠️
hugr-core/src/hugr/views.rs 66.66% 1 Missing ⚠️
hugr-core/src/hugr/views/sibling_subgraph.rs 88.88% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3149      +/-   ##
==========================================
+ Coverage   81.55%   81.61%   +0.06%     
==========================================
  Files         242      242              
  Lines       47373    47649     +276     
  Branches    40970    41246     +276     
==========================================
+ Hits        38633    38888     +255     
- Misses       6721     6737      +16     
- Partials     2019     2024       +5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.