Skip to content

fix: update deprecated QNode.transform_program code #2385

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
andrijapau merged 18 commits into from
Jan 21, 2026
Merged

fix: update deprecated QNode.transform_program code #2385

andrijapau merged 18 commits into from
Jan 21, 2026

Conversation

@andrijapau
Copy link
Contributor

@andrijapau andrijapau commented Jan 12, 2026

See associated deprecation PR in PennyLane: PennyLaneAI/pennylane#8906

[sc-105439]

@andrijapau andrijapau changed the title update deprecated QNode.transform_program code fix: update deprecated QNode.transform_program code Jan 12, 2026
@andrijapau andrijapau mentioned this pull request Jan 12, 2026
Merged
kipawaa
kipawaa requested changes Jan 16, 2026
View reviewed changes
Copy link
Contributor

@kipawaa kipawaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there's also a reference to transform_program in jax_extras/tracing, transform_program = getattr(qnode, "transform_program", []). I think that's all though, looks good!

@github-actions
Copy link
Contributor

github-actions bot commented Jan 16, 2026

Hello. You may have forgotten to update the changelog!
Please edit doc/releases/changelog-dev.md on your branch with:

  • A one-to-two sentence description of the change. You may include a small working example for new features.
  • A link back to this PR.
  • Your name (or GitHub username) in the contributors section.

@andrijapau
Copy link
Contributor Author

andrijapau commented Jan 16, 2026

I think there's also a reference to transform_program in jax_extras/tracing, transform_program = getattr(qnode, "transform_program", []). I think that's all though, looks good!

Nice catch!

@andrijapau
Copy link
Contributor Author

andrijapau commented Jan 16, 2026

I think there's also a reference to transform_program in jax_extras/tracing, transform_program = getattr(qnode, "transform_program", []). I think that's all though, looks good!

Nice catch!

Done: a46e508

kipawaa
kipawaa approved these changes Jan 16, 2026
View reviewed changes
Copy link
Contributor

@kipawaa kipawaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 💯

@andrijapau
Copy link
Contributor Author

andrijapau commented Jan 19, 2026

Hey @paul0403 and @kipawaa , just did a minor change here to update some internal naming.

kipawaa
kipawaa approved these changes Jan 19, 2026
View reviewed changes
Copy link
Contributor

@kipawaa kipawaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

github-merge-queue bot pushed a commit to PennyLaneAI/pennylane that referenced this pull request Jan 20, 2026
@andrijapau
deprecate: rename QNode.transform_program to `QNode.compile_pipelin…
5aa2710 
…e` (#8906)

Catalyst accesses the internal `QNode._transform_program` which makes
this merge process a bit annoying. ⬇️

This private access results in failing documentation tests in this PR
and will only be fixed once
PennyLaneAI/catalyst#2385 goes in. After that
goes in, we can merge #8945
which adds the deprecation warning and test coverage and unskips the
documentation tests. This is done so I don't have to `xFAIL` a million
things in the external-libraries-tests in this PR 😅.

Therefore, the merge order will need to be,

1. #8906 (this PR)
2. PennyLaneAI/catalyst#2385
3. #8945


-------

**Eco-system**

- Catalyst: PennyLaneAI/catalyst#2385
- Lightning: N/A

**Demos**

- QML: N/A

**Plug-ins**

- Qiskit: N/A
- Ion-Q: N/A
- Qulacs: N/A
- AQT: N/A
- Cirq: N/A

[sc-105439]
andrijapau
andrijapau commented Jan 20, 2026
View reviewed changes
andrijapau
andrijapau commented Jan 20, 2026
View reviewed changes
github-merge-queue bot pushed a commit to PennyLaneAI/pennylane that referenced this pull request Jan 20, 2026
@andrijapau
deprecate: rename QNode.transform_program to `QNode.compile_pipelin…
a13e990 
…e` (#8906)

Catalyst accesses the internal `QNode._transform_program` which makes
this merge process a bit annoying. ⬇️

This private access results in failing documentation tests in this PR
and will only be fixed once
PennyLaneAI/catalyst#2385 goes in. After that
goes in, we can merge #8945
which adds the deprecation warning and test coverage and unskips the
documentation tests. This is done so I don't have to `xFAIL` a million
things in the external-libraries-tests in this PR 😅.

Therefore, the merge order will need to be,

1. #8906 (this PR)
2. PennyLaneAI/catalyst#2385
3. #8945


-------

**Eco-system**

- Catalyst: PennyLaneAI/catalyst#2385
- Lightning: N/A

**Demos**

- QML: N/A

**Plug-ins**

- Qiskit: N/A
- Ion-Q: N/A
- Qulacs: N/A
- AQT: N/A
- Cirq: N/A

[sc-105439]
@andrijapau
Copy link
Contributor Author

andrijapau commented Jan 21, 2026

@kipawaa @paul0403 any idea why CI is failing? 😅

@kipawaa
Copy link
Contributor

kipawaa commented Jan 21, 2026

@andrijapau looking into it! Reproduced locally but haven't solved it yet...

@codecov
Copy link

codecov bot commented Jan 21, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.31%. Comparing base (458027d) to head (cfe7659).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #2385      +/-   ##
==========================================
+ Coverage   97.17%   97.31%   +0.13%     
==========================================
  Files         107      107              
  Lines       12951    12951              
  Branches     1075     1075              
==========================================
+ Hits        12585    12603      +18     
+ Misses        301      288      -13     
+ Partials       65       60       -5

☔ 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.

@andrijapau andrijapau merged commit 39a012a into main Jan 21, 2026
38 checks passed
@andrijapau andrijapau deleted the deprecate-transform-program-property branch January 21, 2026 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kipawaa kipawaa kipawaa approved these changes

@paul0403 paul0403 paul0403 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@andrijapau @kipawaa @paul0403