unitaryfund / research

A repository for archived research code
GNU General Public License v3.0
16 stars 9 forks source link

Update text comment on BQSKit notebook #12

Closed andreamari closed 2 years ago

andreamari commented 2 years ago

Hi @natestemen, I guess this comment should be removed since we don't use a volume circuit anymore:

https://github.com/unitaryfund/research/blob/d1e4448e9cf572455fddcf4fa6416406645c441f/ieee-quantum-week/compilation-with-error-mitigation-tutorial/bqskit.ipynb?short_path=5531e7e#L290

natestemen commented 2 years ago

Ah good catch. I'll get that patched up here asap.

nathanshammah commented 2 years ago

I was also wondering if in the next version, we could explain briefly that the quantum volume function is used just as a useful ready-to-use way to have random circuits, but we do not really use it to do a quantum volume test. This could be not immediately clear to a user.

natestemen commented 2 years ago

In the latest version we don't use quantum volume circuits. Do you still think it's worth mentioning?