Skip to content

Change str of ResultDict to print out repeated measurements#6468

Merged
dstrain115 merged 2 commits intoquantumlib:mainfrom
dstrain115:result_dict_reprs
Feb 16, 2024
Merged

Change str of ResultDict to print out repeated measurements#6468
dstrain115 merged 2 commits intoquantumlib:mainfrom
dstrain115:result_dict_reprs

Conversation

@dstrain115
Copy link
Copy Markdown
Collaborator

  • ResultDict will now print out repeated measurements on each line, as opposed to throwing ValueError

Fixes: #6447

- ResultDict will now print out repeated measurements on each
line, as opposed to throwing ValueError

Fixes: quantumlib#6447
@dstrain115 dstrain115 requested review from a team, cduck and vtomole as code owners February 16, 2024 21:59
@dstrain115 dstrain115 requested a review from viathor February 16, 2024 21:59
@CirqBot CirqBot added the size: S 10< lines changed <50 label Feb 16, 2024
@dstrain115 dstrain115 enabled auto-merge (squash) February 16, 2024 22:11
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (6a40da5) 97.82% compared to head (a5bcf12) 97.81%.
Report is 1 commits behind head on main.

❗ Current head a5bcf12 differs from pull request most recent head 159d324. Consider uploading reports for the commit 159d324 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6468      +/-   ##
==========================================
- Coverage   97.82%   97.81%   -0.01%     
==========================================
  Files        1115     1115              
  Lines       97448    97468      +20     
==========================================
+ Hits        95327    95342      +15     
- Misses       2121     2126       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dstrain115 dstrain115 merged commit a8bd9a5 into quantumlib:main Feb 16, 2024
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
…ib#6468)

- ResultDict will now print out repeated measurements on each
line, as opposed to throwing ValueError

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

Labels

size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can't print the ResultDict of an array with 2 values

4 participants