Skip to content

feat!: Remove deprecated unpack tuple pass#1387

Merged
aborgna-q merged 2 commits intomainfrom
ab/remove-deprecated
Feb 2, 2026
Merged

feat!: Remove deprecated unpack tuple pass#1387
aborgna-q merged 2 commits intomainfrom
ab/remove-deprecated

Conversation

@aborgna-q
Copy link
Copy Markdown
Collaborator

@aborgna-q aborgna-q commented Feb 2, 2026

BREAKING CHANGE: Removed deprecated find_tuple_unpack_rewrites. Use hugr::algorithms::UntuplePass.

drive-by: Cleanup other uses of deprecated.

Not a python breaking change
BEGIN_COMMIT_OVERRIDE
END_COMMIT_OVERRIDE

@aborgna-q aborgna-q requested a review from a team as a code owner February 2, 2026 16:55
@aborgna-q aborgna-q requested a review from ss2165 February 2, 2026 16:55
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.66%. Comparing base (01ff98c) to head (b5fe0e5).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1387      +/-   ##
==========================================
+ Coverage   79.57%   79.66%   +0.09%     
==========================================
  Files         158      157       -1     
  Lines       20529    20434      -95     
  Branches    19540    19445      -95     
==========================================
- Hits        16335    16279      -56     
+ Misses       3211     3177      -34     
+ Partials      983      978       -5     
Flag Coverage Δ
python 92.99% <ø> (ø)
qis-compiler 100.00% <ø> (ø)
rust 78.97% <ø> (+0.09%) ⬆️

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.

@hugrbot
Copy link
Copy Markdown
Collaborator

hugrbot commented Feb 2, 2026

This PR contains breaking changes to the public Rust API.

cargo-semver-checks summary
    Building tket v0.16.0 (current)
     Built [  46.100s] (current)
   Parsing tket v0.16.0 (current)
    Parsed [   0.093s] (current)
  Building tket v0.16.0 (baseline)
     Built [  43.643s] (baseline)
   Parsing tket v0.16.0 (baseline)
    Parsed [   0.085s] (baseline)
  Checking tket v0.16.0 -> v0.16.0 (assume minor change)
   Checked [   0.095s] 196 checks: 194 pass, 2 fail, 0 warn, 49 skip

--- failure function_missing: pub fn removed or renamed ---

Description:
A publicly-visible function cannot be imported by its prior path. A `pub use` may have been removed, or the function itself may have been renamed or removed entirely.
      ref: https://doc.rust-lang.org/cargo/reference/semver.html#item-remove
     impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.46.0/src/lints/function_missing.ron

Failed in:
function tket::passes::tuple_unpack::find_tuple_unpack_rewrites, previously in file /home/runner/work/tket2/tket2/BASELINE_BRANCH/tket/src/passes/tuple_unpack.rs:21
function tket::passes::find_tuple_unpack_rewrites, previously in file /home/runner/work/tket2/tket2/BASELINE_BRANCH/tket/src/passes/tuple_unpack.rs:21

--- failure module_missing: pub module removed or renamed ---

Description:
A publicly-visible module cannot be imported by its prior path. A `pub use` may have been removed, or the module may have been renamed, removed, or made non-public.
      ref: https://doc.rust-lang.org/cargo/reference/semver.html#item-remove
     impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.46.0/src/lints/module_missing.ron

Failed in:
mod tket::passes::tuple_unpack, previously in file /home/runner/work/tket2/tket2/BASELINE_BRANCH/tket/src/passes/tuple_unpack.rs:1

   Summary semver requires new major version: 2 major and 0 minor checks failed
  Finished [  91.425s] tket
  Building tket-qsystem v0.22.0 (current)
     Built [  44.620s] (current)
   Parsing tket-qsystem v0.22.0 (current)
    Parsed [   0.027s] (current)
  Building tket-qsystem v0.22.0 (baseline)
     Built [  44.375s] (baseline)
   Parsing tket-qsystem v0.22.0 (baseline)
    Parsed [   0.025s] (baseline)
  Checking tket-qsystem v0.22.0 -> v0.22.0 (assume minor change)
   Checked [   0.058s] 196 checks: 196 pass, 49 skip
   Summary no semver update required
  Finished [  90.587s] tket-qsystem

@aborgna-q aborgna-q added this pull request to the merge queue Feb 2, 2026
Merged via the queue into main with commit 86ac8f9 Feb 2, 2026
24 checks passed
@aborgna-q aborgna-q deleted the ab/remove-deprecated branch February 2, 2026 17:16
This was referenced Feb 2, 2026
github-merge-queue bot pushed a commit that referenced this pull request Feb 3, 2026
## 🤖 New release

* `tket`: 0.16.0 -> 0.17.0 (✓ API compatible changes)
* `tket-qsystem`: 0.22.0 -> 0.23.0 (✓ API compatible changes)

<details><summary><i><b>Changelog</b></i></summary><p>

## `tket`

<blockquote>

##
[0.17.0](tket-v0.16.0...tket-v0.17.0)
- 2026-02-02

### Bug Fixes

- *(encoded-circ)* Track unsupported wires between input and output
([#1224](#1224))
- Multiple fixes to the pytket encoder
([#1226](#1226))
- Don't use opgroup in pytket barrier encoding
([#1251](#1251))
- guppy_to_circuit always returns num_operations = 0
([#1200](#1200))
- *(pytket-decoder)* Avoid QAllocating and immediately freeing qubits
([#1256](#1256))
- Encoding of opaque subgraphs with no associated qubit/bit
([#1295](#1295))
- [**breaking**] Don't rely on command params for pytket barriers
([#1298](#1298))
- Track output qubits in CircuitInfo
([#1304](#1304))
- Wrongly reused qubit IDs in pytket encoding
([#1358](#1358))

### New Features

- Deprecate local find_tuple_unpack rewrite
([#1188](#1188))
- Add CopyableExpressionAST
([#1209](#1209))
- `NormalizeGuppy` pass to simplify generated structure
([#1220](#1220))
- [**breaking**] pytket EncodedCircuit struct for in-place pytket
optimisation ([#1211](#1211))
- [**breaking**] Interval is independent of resource IDs and scope
position ([#1205](#1205))
- Don't translate usizes to pytket
([#1241](#1241))
- BorrowSquashPass to elide redundant borrow/return ops
([#1159](#1159))
- [**breaking**] Bump hugr to 0.25.0
([#1325](#1325))
- Remove order edges in NormalizeGuppy pass
([#1326](#1326))
- [**breaking**] Remove deprecated unpack tuple pass
([#1387](#1387))

### Refactor

- Remove contain_qubits, use TypeUnpacker
([#1283](#1283))
- [**breaking**] Replace Subcircuit with SiblingSubgraph
([#1288](#1288))
- *(metadata)* [**breaking**] Migrate all metadata keys onto the new
metadata traits ([#1328](#1328))
</blockquote>

## `tket-qsystem`

<blockquote>

##
[0.23.0](tket-qsystem-v0.22.0...tket-qsystem-v0.23.0)
- 2026-02-02

### Bug Fixes

- [**breaking**] Don't rely on command params for pytket barriers
([#1298](#1298))
- Wrongly reused qubit IDs in pytket encoding
([#1358](#1358))

### New Features

- `NormalizeGuppy` pass to simplify generated structure
([#1220](#1220))
- Allow running arbitrary serializable pytket passes on hugrs
([#1266](#1266))
- BorrowSquashPass to elide redundant borrow/return ops
([#1159](#1159))
- [**breaking**] Bump hugr to 0.25.0
([#1325](#1325))
- Remove order edges in NormalizeGuppy pass
([#1326](#1326))
- hide new public funcs introduced by linearization
([#1333](#1333))

### Testing

- regenerate guppy_opt examples, and count gates
([#1249](#1249))
- run pytket on guppy_opt tests, measure (very limited) success
([#1250](#1250))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).
@hugrbot hugrbot mentioned this pull request Feb 5, 2026
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