unitaryfund / mitiq

Mitiq is an open source toolkit for implementing error mitigation techniques on most current intermediate-scale quantum computers.
https://mitiq.readthedocs.io
GNU General Public License v3.0
363 stars 160 forks source link

RFC for Readout Error Mitigation #1387

Closed Misty-W closed 2 years ago

Misty-W commented 2 years ago

Pre-Request Checklist

Issue Description

Review and approve [RFC for Readout Error Mitigation (REM)] (https://docs.google.com/document/d/1buO5PrO5sS02VXjcaYf37RuR0rF6xpyr4J9H1tI4vN4/edit#heading=h.c072otyej9hw)

Feature request issue #815

Misty-W commented 2 years ago

Tried to assign @amirebrahimi but he doesn't appear in the list of possible assignees when I type in his username.

amirebrahimi commented 2 years ago

Thanks for the update. I have comments in the RFC that I'll be addressing soon.

amirebrahimi commented 2 years ago

I've updated the RFC doc and my branch to address some of the changes requested: https://github.com/amirebrahimi/mitiq/commit/464578b247c6c5a517b556e94f4541149528b26f https://github.com/amirebrahimi/mitiq/commit/5d7e51e878c3dd8a3e0c2f1e9a80b585505f4e0c

andreamari commented 2 years ago

Thanks @amirebrahimi, I left some comments on the RFC document. For my side, the current version of the RFC can be accepted! However a formal acceptance decision can only be taken after a proper discussion with other members of the Mitiq team. I'll keep you posted!

natestemen commented 2 years ago

Nice job Amir, this looks really great. I've made a comment about naming similar to Andrea's, but this is ready to move on to the next steps from me. I've marked it as accepted. Congrats! 🥳 🎉

I'll close this out now as this ticket was just for tracking the RFC, and we can use https://github.com/unitaryfund/mitiq/issues/815 for the implementation whenever we decide to do that.