remove internal `PassPipelineWrapper/QNodeWrapper` in favour of PennyLane's new unified transforms API by andrijapau · Pull Request #2525 · PennyLaneAI/catalyst

andrijapau · 2026-02-26T17:47:38Z

Copies some of the changes made here: #2487.

The goal of this PR is to get rid of PassPipelineWrapper and QNodeWrapper and use the new unified transforms API.

Complimentary PR in PennyLane that follows up on the removal: PennyLaneAI/pennylane#9123

[sc-112140][sc-104261]

Co-authored-by: Christina Lee <chrissie.c.l@gmail.com>

frontend/catalyst/passes/pass_api.py

frontend/catalyst/qfunc.py

frontend/catalyst/passes/pass_api.py

Co-authored-by: Christina Lee <chrissie.c.l@gmail.com>

albi3ro

👍

andrijapau · 2026-03-11T18:09:51Z

Do not merge until someone from @PennyLaneAI/compilation has time to take a look 🙏🏼

paul0403

Music to my ears 🤩

.github/workflows/check-catalyst.yaml

doc/releases/changelog-dev.md

frontend/catalyst/passes/pass_api.py

frontend/test/pytest/python_interface/pass_api/test_pass_api.py

andrijapau · 2026-03-11T19:14:21Z

@paul0403 are you planning on doing a full review? I.e., providing an approval if you're happy with the changes?

paul0403 · 2026-03-11T19:42:21Z

@paul0403 are you planning on doing a full review? I.e., providing an approval if you're happy with the changes?

Oh it was already a full review, everything looks good!

I'll click the approve button once the hardcoded branches in CI is removed!

andrijapau · 2026-03-11T19:45:32Z

@paul0403 are you planning on doing a full review? I.e., providing an approval if you're happy with the changes?

Oh it was already a full review, everything looks good!

I'll click the approve button once the hardcoded branches in CI is removed!

Ok, that won't happen until PennyLaneAI/pennylane#9123 goes in 👍🏼. I'll re-tag you when it's ready.

Must be merged *before* PennyLaneAI/catalyst#2525 (the PR that removes `PassPipelineWrapper`. [sc-104261] --------- Co-authored-by: Christina Lee <christina@xanadu.ai>

andrijapau · 2026-03-12T13:15:59Z

@paul0403 are you planning on doing a full review? I.e., providing an approval if you're happy with the changes?

Oh it was already a full review, everything looks good!
I'll click the approve button once the hardcoded branches in CI is removed!

Ok, that won't happen until PennyLaneAI/pennylane#9123 goes in 👍🏼. I'll re-tag you when it's ready.

Ready to go @paul0403! Just waiting on your approval. 🫡

lillian542 and others added 13 commits February 18, 2026 16:18

raise an error in apply_pass if program capture is enabled

7a22c10

Update frontend/catalyst/passes/pass_api.py

a6bb0e3

Co-authored-by: Christina Lee <chrissie.c.l@gmail.com>

dispatch to qml.transform instead

52e360a

update builtin_passes to use qml.transform

ab7bb9a

use setup_inputs instead of kwargs

1bf27b0

setup_inputs is a function

ea0fd53

switch - and _ where needed

f14eb7b

Update frontend/catalyst/passes/pass_api.py

fa547da

Co-authored-by: Christina Lee <chrissie.c.l@gmail.com>

update out of date test name

0fcc633

update outdated test

5b46757

black formatting

0576ad2

push fix

f2490bc

format

0ea4b36

andrijapau changed the title ~~push fix~~ fix: #2487 Feb 26, 2026

fix lit test

657fe66

andrijapau commented Feb 26, 2026

View reviewed changes

frontend/catalyst/passes/pass_api.py Outdated Show resolved Hide resolved

andrijapau commented Feb 26, 2026

View reviewed changes

frontend/catalyst/passes/pass_api.py Outdated Show resolved Hide resolved

andrijapau and others added 2 commits February 26, 2026 13:21

Apply suggestions from code review

4844921

update logic for filtering the pass_options

cc29d2a

andrijapau commented Feb 26, 2026

View reviewed changes

frontend/catalyst/passes/pass_api.py Show resolved Hide resolved

andrijapau commented Feb 26, 2026

View reviewed changes

frontend/catalyst/passes/pass_api.py Outdated Show resolved Hide resolved

andrijapau added 4 commits February 26, 2026 13:26

format

5703d3d

completely stop using PassPipelineWrapper

e2be3fe

remove test that no longer holds

629ec23

try fix

6e556ba

andrijapau commented Feb 27, 2026

View reviewed changes

frontend/catalyst/qfunc.py Outdated Show resolved Hide resolved

andrijapau added 4 commits February 27, 2026 14:37

fix

7e0699f

fix

9ea8816

fix

4050e33

fix

706d4a1

JerryChen97 reviewed Mar 11, 2026

View reviewed changes

frontend/catalyst/passes/pass_api.py Outdated Show resolved Hide resolved

JerryChen97 reviewed Mar 11, 2026

View reviewed changes

frontend/catalyst/passes/pass_api.py Outdated Show resolved Hide resolved

andrijapau and others added 2 commits March 11, 2026 10:53

dont use lambda use function def

7252907

Merge branch 'main' into andrija/apply_pass_raises_error

0034175

JerryChen97 approved these changes Mar 11, 2026

View reviewed changes

albi3ro reviewed Mar 11, 2026

View reviewed changes

frontend/catalyst/passes/pass_api.py Outdated Show resolved Hide resolved

Update frontend/catalyst/passes/pass_api.py

7df31fa

Co-authored-by: Christina Lee <chrissie.c.l@gmail.com>

albi3ro approved these changes Mar 11, 2026

View reviewed changes

andrijapau added the do-not-merge label Mar 11, 2026

Merge branch 'main' into andrija/apply_pass_raises_error

b9c63e6

andrijapau changed the title ~~remove PassPipelineWrapper/QNodeWrapper in favour of unified qml.transform/CompilePipeline API~~ remove internal PassPipelineWrapper/QNodeWrapper in favour of unified qml.transform/CompilePipeline API Mar 11, 2026

andrijapau changed the title ~~remove internal PassPipelineWrapper/QNodeWrapper in favour of unified qml.transform/CompilePipeline API~~ remove internal PassPipelineWrapper/QNodeWrapper in favour of PennyLane's new unified transforms API Mar 11, 2026

paul0403 reviewed Mar 11, 2026

View reviewed changes

move to internal changes

2e37628

andrijapau requested a review from paul0403 March 11, 2026 19:10

andrijapau and others added 3 commits March 12, 2026 08:52

restore check-catalyst

c38fd17

update PL version

036b8bf

Merge branch 'main' into andrija/apply_pass_raises_error

2983be2

andrijapau removed the do-not-merge label Mar 12, 2026

paul0403 approved these changes Mar 12, 2026

View reviewed changes

Merge branch 'main' into andrija/apply_pass_raises_error

e144592

andrijapau merged commit c6cf95d into main Mar 12, 2026
38 checks passed

andrijapau deleted the andrija/apply_pass_raises_error branch March 12, 2026 14:26

maliasadi mentioned this pull request Mar 17, 2026

[Base] The native-MLIR graph decomposition #2552

Open

Conversation

andrijapau commented Feb 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

albi3ro left a comment

Choose a reason for hiding this comment

Uh oh!

andrijapau commented Mar 11, 2026

Uh oh!

paul0403 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

andrijapau commented Mar 11, 2026

Uh oh!

paul0403 commented Mar 11, 2026

Uh oh!

andrijapau commented Mar 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

andrijapau commented Mar 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

andrijapau commented Feb 26, 2026 •

edited

Loading

andrijapau commented Mar 11, 2026 •

edited

Loading

andrijapau commented Mar 12, 2026 •

edited

Loading