Skip to content

feat!: Replace Subcircuit // take 3#1223

Closed
lmondada wants to merge 3 commits intoQuantinuum:lm/interval-prfrom
lmondada:lm/subcirc-pr
Closed

feat!: Replace Subcircuit // take 3#1223
lmondada wants to merge 3 commits intoQuantinuum:lm/interval-prfrom
lmondada:lm/subcirc-pr

Conversation

@lmondada
Copy link
Copy Markdown
Contributor

@lmondada lmondada commented Nov 5, 2025

Hey Alan,

As promised, here is the subcircuit PR, rewritten to use CopyableExpr. This way subcircuits can be used to represent any valid SiblingSubgraph (indeed, a superset of SiblingSubgraph, as certain non-convex subgraphs are expressible too).

The convexity check will come in a separate PR.

BREAKING CHANGE: New API for Subcircuit, see docs. The Rewrite trait now takes a generic node argument.

@lmondada lmondada requested a review from a team as a code owner November 5, 2025 08:03
@lmondada lmondada requested review from cqc-alec and removed request for a team November 5, 2025 08:03
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 5, 2025

Codecov Report

❌ Patch coverage is 70.19790% with 256 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.08%. Comparing base (d6d3324) to head (1eef17b).

Files with missing lines Patch % Lines
tket/src/subcircuit.rs 74.47% 168 Missing and 14 partials ⚠️
tket/src/resource.rs 2.43% 40 Missing ⚠️
tket/src/rewrite/strategy.rs 5.55% 17 Missing ⚠️
tket/src/resource/scope.rs 85.71% 5 Missing and 2 partials ⚠️
tket/src/portmatching/matcher.rs 33.33% 4 Missing ⚠️
tket/src/resource/types.rs 0.00% 2 Missing ⚠️
tket/src/rewrite.rs 83.33% 1 Missing and 1 partial ⚠️
tket/src/rewrite/trace.rs 0.00% 1 Missing ⚠️
tket/src/subcircuit/interval.rs 88.88% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@                Coverage Diff                 @@
##           lm/interval-pr    #1223      +/-   ##
==================================================
- Coverage           78.98%   78.08%   -0.90%     
==================================================
  Files                 154      155       +1     
  Lines               19244    19994     +750     
  Branches            18142    18892     +750     
==================================================
+ Hits                15199    15612     +413     
- Misses               3091     3419     +328     
- Partials              954      963       +9     
Flag Coverage Δ
python 92.65% <ø> (ø)
qis-compiler 68.40% <ø> (ø)
rust 77.66% <70.19%> (-0.94%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.

@cqc-alec cqc-alec requested review from acl-cqc and removed request for cqc-alec November 5, 2025 09:30
@doug-q doug-q deleted the branch Quantinuum:lm/interval-pr November 6, 2025 09:32
@doug-q doug-q closed this Nov 6, 2025
@lmondada
Copy link
Copy Markdown
Contributor Author

lmondada commented Nov 7, 2025

Hi, is there a specific reason why this was closed? Or maybe this was automatic with the deletion of the lm/interval-pr...

In any case, @doug-q are you able to re-open this PR (I can't)? Otherwise I'll create a new one :)

@cqc-alec
Copy link
Copy Markdown
Collaborator

cqc-alec commented Nov 7, 2025

Hi, is there a specific reason why this was closed? Or maybe this was automatic with the deletion of the lm/interval-pr...

Probably the latter. It isn't possible to reopen this PR because the target branch has gone. I suggest creating a new one!

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.

3 participants