qiskit-experiments icon indicating copy to clipboard operation
qiskit-experiments copied to clipboard

Published 20 hours ago • verified publisher icon Qiskit-Extensions

Add Mirror RB experiment

Open paco-ri opened this issue 2 years ago • 7 comments

Summary

Added mirror randomized benchmarking experiment and analysis classes

Details and comments

TODO

Accommodate mirror circuits of zero length

paco-ri avatar Jul 11 '22 19:07 paco-ri

Would it be better to report error per layer instead of EPC? Especially since there will be non-Clifford and Clifford versions.

coruscating avatar Aug 19 '22 15:08 coruscating

@coruscating Yeah that would be more accurate

albertzhu01 avatar Aug 22 '22 17:08 albertzhu01

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Aug 31 '22 21:08 CLAassistant

I think this can be merged after addressing Naoki's comments and changing EPC to EPL. If issues come up later, those can be addressed in future PRs.

coruscating avatar Sep 08 '22 18:09 coruscating

The term "EPC" is currently used in standard and interleaved RB, so it would have to be replaced in those implementations too. Or maybe it would be better to replace "EPC" with "EPL" in the PR with universal-gate-set MRB. This PR implements MRB with circuits generated by the "old" sampling method which, to my knowledge, has only been implemented with Clifford gates.

paco-ri avatar Sep 13 '22 04:09 paco-ri

Agreed, let's leave it as EPC for this PR then.

coruscating avatar Sep 14 '22 14:09 coruscating

Assuming the branch https://github.com/Qiskit/qiskit-experiments/tree/feature/rb_speedup is merged before this PR, note two things:

  1. The methods CliffordUtils::random_clifford_circuits and CliffordUtils::random_cliffords will be deprecated. Use StandardRB::__sample_sequence instead.
  2. Consider whether using the mapping of Cliffords to integers is worthwhile for computing the inverses. See CliffordUtils::_CLIFFORD_INVERSE_1Q

merav-aharoni avatar Nov 13 '22 11:11 merav-aharoni