Quantum Computer symmetrize_readout argument is unused.

Open
#1,013 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
35/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Stale
Tech stack
python

Research direction

Start by locating the QuantumComputer constructor and the run and run_symmetrized_readout methods referenced in the issue. Determine whether symmetrize_readout should be removed or connected to execution, including the readout discrepancy mentioned with #1003. Done means the argument no longer appears unused and the chosen behavior is covered by the relevant tests.

Written by the indexing model from the issue text.

Description

bug :bug:

Pre-Report Checklist

  • I am running the latest versions of pyQuil and the Forest SDK
  • I checked to make sure that this bug has not already been reported

Issue Description

QuantumComputer accepts an argument symmetrize_readout which appears to do nothing, currently. We should either

  1. remove it
  2. check if it is set in run, and if so automatically use run_symmetrized_readout with some default symm_type instead. This will probably require dealing with the discrepancy between the two functions in how readout is handled (which maybe needs to be addressed separately anyway? e.g. #1003)
  3. changed from a bool to record symm_type

I think 1) is most appropriate. This seems like something that should be specified at run time rather than during creation of the QC object.

How to Reproduce

I just noticed that the parameter doesn't seem to be used anywhere.

Dominant language
Python
Stars
1.5k
Forks
358
Avg merge
1d 58m
Merged PRs (30d)
4

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from rigetti/pyquil

All issues in rigetti/pyquil

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.