search

qiskit-community / qiskit-ignis

Ignis (deprecated) provides tools for quantum hardware verification, noise characterization, and error correction.
Apache License 2.0
168 stars 162 forks source link

Add possibility of x base encoding #581

Closed quantumjim closed 2 years ago

quantumjim commented 3 years ago

Summary

A keyword argument xbasis is added to RepetitionCodes. This allows an alternate encoding to be used, based on the repetitions of the X basis states rather than the Z basis.

Details and comments

This does not impact any existing features and adds no new methods. It basically just injects a bunch of hadamards and reverses cx gates if xbasis=True.

quantumjim commented 3 years ago

I snuck in an extra little opt-in feature too: analysing error probs for the results from a logical '1'.

quantumjim commented 3 years ago

Thanks for the comments. What do you mean by a reno?

mtreinish commented 3 years ago

Oh, just a release note file using the reno tool. The process is documented in the contributing guide here: https://github.com/Qiskit/qiskit-ignis/blob/master/CONTRIBUTING.md#release-notes

quantumjim commented 3 years ago

I added the release note file. The failing test is just due to the random timeout of the tutorials, and might get fixed on a rerun.

quantumjim commented 3 years ago

It seems there was something making the tutorial take longer. All fixed now and ready to merge!

quantumjim commented 3 years ago

I inadvertently pushed something to this branch in error. It's undone now so everything is back to what it should be.

quantumjim commented 2 years ago

Done! And also included the missing ')', which made the linter :(