Julien Gacon
Julien Gacon
The problem seems to be that the `QFT` circuit, which is called "QFT", already contains a gate called "QFT". So once you call `append` on the circuit, it is again...
To fix the last errors you should run black, e.g. by calling `make black` in the terminal in the folder `qiskit-terra`. 🙂
@daniel-fry are you still pursuing this? 🙂
It would probably be good to hold this off until #7666 is merged, since this here is the smaller change and will likely be easier to integrate 🙂
This is not blocked anymore, so we can proceed if you're still interested @daniel-fry! Since `Initialize` was split partially into `StatePreparation`, the functionality would probably have to be moved there...
That's a great extension! Does this also collect cases where other single qubit gates are in between Pauli rotations, like `RY(a) H RY(b)`? And can we extend this to cover...
Thanks for the PR @epelaaez! I'm not quite sure if this is the right solution though, as the `qbit_argument_conversion` is still called in `append`. Isn't the error comming from calling...
Looks good! Could you update the documentation to reflect this function accepts qubits and add a bugfix releasenote?
Looks good thanks! Actually, could you add the same to the `state_preparation` method (in `state_preparation.py`)? Since that's the same initialize just without the reset it would be good if they...
@MarcDrudis yes to both your comments above 🙂 Also, I think the learning rate can be an array which doesn't seem to be documented.