Fix failing test after API change in MultiControlledX #1472
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context: The input argument
control_wires
was recently removed fromqml.MultiControlledX
in PennyLaneAI/pennylane#6832. This API change resulted in a failure in the Catalyst testTestGateAliases.test_control_aliases
in the "Check CPL latest/latest/latest" workflow.For reference, the call signature of
qml.MultiControlledX
changed fromto
Description of the Change: Update the failing test to use the new call signature of
qml.MultiControlledX
. Also, prefer to explicitly pass in the wire index or list of wire indices by keyword argument withwires=...
, rather than as positional arguments, to hopefully avoid similar issues in the future.