Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Copy/paste errors in docstrings #6225

Closed
daxfohl opened this issue Aug 1, 2023 · 0 comments · Fixed by #6537
Closed

Copy/paste errors in docstrings #6225

daxfohl opened this issue Aug 1, 2023 · 0 comments · Fixed by #6537
Assignees
@dstrain115
Labels
area/docstrings good first issue This issue can be resolved by someone who is not familiar with the codebase. A good starting issue. kind/bug-report Something doesn't seem to work. kind/docs Documentation related problems, ideas, requests triage/accepted A consensus emerged that this bug report, feature request, or other action should be worked on

Comments

@daxfohl
Copy link
Collaborator

daxfohl commented Aug 1, 2023

Description of the issue

Density matrix is mentioned in docstrings for _BufferedStateVector, SimulationState._perform_measurement, and Simulator. Should be changed to "state vector" and QuantumStateRepresentation respectively.
@daxfohl daxfohl added the kind/bug-report Something doesn't seem to work. label Aug 1, 2023
@senecameeks senecameeks self-assigned this Aug 2, 2023
@senecameeks senecameeks added the triage/accepted A consensus emerged that this bug report, feature request, or other action should be worked on label Aug 2, 2023
@dstrain115 dstrain115 added good first issue This issue can be resolved by someone who is not familiar with the codebase. A good starting issue. kind/docs Documentation related problems, ideas, requests area/docstrings labels Feb 13, 2024
@dstrain115 dstrain115 assigned dstrain115 and unassigned senecameeks Mar 21, 2024
dstrain115 added a commit to dstrain115/Cirq-1 that referenced this issue Mar 30, 2024
@dstrain115
Fix density matrix references in other simulators
135d7da 
- These simulators do not use density matrices.

Fixes: quantumlib#6225
@dstrain115 dstrain115 mentioned this issue Mar 30, 2024
Merged
dstrain115 added a commit that referenced this issue Mar 30, 2024
@dstrain115
Fix density matrix references in other simulators (#6537)
7b97ead 
- These simulators do not use density matrices.

Fixes: #6225
jselig-rigetti pushed a commit to jselig-rigetti/Cirq that referenced this issue May 28, 2024
@dstrain115 @jselig-rigetti
Fix density matrix references in other simulators (quantumlib#6537)
902c66d 
- These simulators do not use density matrices.

Fixes: quantumlib#6225
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this issue Oct 31, 2024
@dstrain115
Fix density matrix references in other simulators (quantumlib#6537)
3a61f45 
- These simulators do not use density matrices.

Fixes: quantumlib#6225
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dstrain115 dstrain115

Labels
area/docstrings good first issue This issue can be resolved by someone who is not familiar with the codebase. A good starting issue. kind/bug-report Something doesn't seem to work. kind/docs Documentation related problems, ideas, requests triage/accepted A consensus emerged that this bug report, feature request, or other action should be worked on
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix density matrix references in other simulators dstrain115/Cirq-1
3 participants
@daxfohl @senecameeks @dstrain115