Malik Williams

Results 9 comments of Malik Williams

Hi @AshwinKul28, I updated qiskit via 'pip install update qiskit==0.25.2', and I still got the error. ![get-unitary-error](https://user-images.githubusercontent.com/33338744/116039878-241d5b00-a620-11eb-9bf8-4fe616c69152.PNG) #Update: Here's the same circuit in the conda terminal ![get-unitary-error-2](https://user-images.githubusercontent.com/33338744/116041600-3d270b80-a622-11eb-8587-ccd4d1de3a78.PNG)

I was able to fix the error, but only by having 'circ.save_unitary()' called after 'circ = transpile(circ, simulator)'. ![get-unitary-error-fix](https://user-images.githubusercontent.com/33338744/116165266-57a7c600-a6b0-11eb-8f5c-3c0dd74150ce.PNG)

@AshwinKul28 I think the main issue is that it needs to be called after the simulator. Because 'save_unitary()' still works with the transpile commented out: ![get-unitary-error-fix-2](https://user-images.githubusercontent.com/33338744/116296711-65f8ef00-a74f-11eb-8ee0-e44c10d85c75.PNG)

@AshwinKul28 I'm a little confused by your wording. But it seems like the fix works. So is the next step to create a pull request for the fix we found?

@AshwinKul28 I saw the previous Pull Request needed to be closed. This Pull Request has been created as a resolve #1176.

@SooluThomas @nonhermitian could you review this Pull Request? This Pull Request also fixes issue #1172.

I'm going to be updating this commit to issue a resolve found in https://github.com/Qiskit/qiskit-terra/issues/6346. And reverting the changes made previously where the lines were moved (to where the save states...

@mtreinish could you review the changes I made regarding qiskit.providers.aer?

@mtreinish @nonhermitian. Could I get a review on the changes made?