PennyLaneAI / catalyst

A JIT compiler for hybrid quantum programs in PennyLane
https://docs.pennylane.ai/projects/catalyst
Apache License 2.0
109 stars 28 forks source link

`MidCircuitMeasure` inherits `MidMeasureMP` #832

Open mudit2812 opened 4 weeks ago

mudit2812 commented 4 weeks ago

Context: MidCircuitMeasure inherits HybridOp but only relies on its tracing functionality and nothing else, but also contains a lot of attributes that make it more appropriate for it to be a child of MidMeasureMP instead of HybridOp.

Description of the Change:

Benefits:

Possible Drawbacks:

Related GitHub Issues:

mudit2812 commented 4 weeks ago

Tests won't pass until https://github.com/PennyLaneAI/pennylane/pull/5888 is merged. Otherwise ready for review.

dime10 commented 6 days ago

@mudit2812 Any chance we will see this improvement land or is it considered abandoned? 😅

mudit2812 commented 6 days ago

@dime10 I wouldn't call it abandoned. Definitely think it's possible to get this in for 0.8, but it's just not a priority at the moment.