Skip to content

chore: Remove pytket dependency from tket-py#1507

Open
aborgna-q wants to merge 2 commits intomainfrom
ab/drop-pytket-dep
Open

chore: Remove pytket dependency from tket-py#1507
aborgna-q wants to merge 2 commits intomainfrom
ab/drop-pytket-dep

Conversation

@aborgna-q
Copy link
Copy Markdown
Collaborator

@aborgna-q aborgna-q commented Apr 9, 2026

After #1482 we no longer use pytket from the python side other than for type-checking PytketHugrPass and when testing.
Moved the dependency to a pytket extra. This is only required to have proper typing for the pass.

Now that pytket is not required to build and test the wheels, we can produce free-threaded wheels for python 3.13t (pytket doesn't publish those).
Edit: Removed the free-threaded wheels build.
I'll leave it for later, once hugr publishes 3.14t releases (Quantinuum/hugr#3021).

@aborgna-q aborgna-q requested a review from a team as a code owner April 9, 2026 14:38
@aborgna-q aborgna-q requested a review from acl-cqc April 9, 2026 14:38
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.84%. Comparing base (042ba1d) to head (7477974).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1507   +/-   ##
=======================================
  Coverage   83.84%   83.84%           
=======================================
  Files         187      187           
  Lines       28959    28959           
  Branches    27880    27880           
=======================================
  Hits        24282    24282           
  Misses       3517     3517           
  Partials     1160     1160           
Flag Coverage Δ
python 92.33% <100.00%> (ø)
qis-compiler 91.66% <ø> (ø)

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.

@aborgna-q aborgna-q force-pushed the ab/drop-pytket-dep branch 4 times, most recently from 7477974 to a3d04e4 Compare April 10, 2026 09:05
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.

1 participant