Add argument check to move_pauli_strings_into_circuit#7985
Add argument check to move_pauli_strings_into_circuit#7985pavoljuhas merged 9 commits intoquantumlib:mainfrom
move_pauli_strings_into_circuit#7985Conversation
…_into_circuit` to only have PauliStringPhasor operations.
…t, and add a test case.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7985 +/- ##
=======================================
Coverage 99.63% 99.63%
=======================================
Files 1110 1110
Lines 100020 100028 +8
=======================================
+ Hits 99658 99666 +8
Misses 362 362 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Ensure error message cannot get too long for larget circuit_left. Simplify the test of the new argument validation. Just repeat example from the addressed issue quantumlib#7934.
contrib.paulistring.move_pauli_strings_into_circuit.move_pauli_strings_into_circuit
move_pauli_strings_into_circuitmove_pauli_strings_into_circuit
|
NB - I am moving the initial description here so the description can be reworded as a commit message
It is sufficient to verify that the new exception is raised. I have tweaked the test in 3aa9879 so it corresponds to the addressed issue
The function is in the |
pavoljuhas
left a comment
There was a problem hiding this comment.
LGTM after pushing in a couple of tweaks.
@Acciaccatura - please check if this is good to go from your POV.
The
circuit_leftargument ofmove_pauli_strings_into_circuitshould contain only a PauliStringPhasor-type operations.
Raise exception if that is not the case.
Fixes #7934