remove id gates hack from simulator dev

[cvqmm]

use qubit_order to remove identity gates hack from cirq simulator device

[antalszava]

Hi @cvqmm, thanks for the contribution! We'll be leaving a review. It seems that the affected files may have to be formatted using black to pass the continuous integration check for formatting.

[codecov]

Codecov Report

Merging #111 (75bd9bf) into master (34325cf) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #111      +/-   ##
==========================================
- Coverage   99.24%   99.23%   -0.01%     
==========================================
  Files           8        8              
  Lines         397      393       -4     
==========================================
- Hits          394      390       -4     
  Misses          3        3              

Impacted Files	Coverage Δ
pennylane_cirq/simulator_device.py	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[Jaybsoni]

Hi @cvqmm,

Thanks for the contribution! Glad we were able to get rid of that hack in the code. The contribution looks good to go.

Could you add an entry under the subheading "Improvements" in the CHANGELOG.md file, with a short description of the fix and a link to this PR. Don't forget to add your name to the contributors list at the bottom!

Once that's done I am happy to approve.
Cheers,

[cvqmm]

Hi @Jaybsoni,

thanks for the review! the fix's description is added to the change log in the last commit 75bd9bf

Regards,

[Jaybsoni]

Looks good 💯! @antalszava I think we will need an override for failing codecov.

[antalszava]

Sure 👍 I can merge this once there's a 2nd review in.

[rmoyard]

It looks good to me, thanks for the contribution!