Skip to content

Coverage - avoid skipping code lines on different random seeds#6256

Merged
pavoljuhas merged 2 commits intoquantumlib:masterfrom
pavoljuhas:avoid-random-coverage-deltas
Aug 23, 2023
Merged

Coverage - avoid skipping code lines on different random seeds#6256
pavoljuhas merged 2 commits intoquantumlib:masterfrom
pavoljuhas:avoid-random-coverage-deltas

Conversation

@pavoljuhas
Copy link
Copy Markdown
Collaborator

Eliminate random decreases of coverage.
Make codecov coverage check happy.

@pavoljuhas pavoljuhas requested review from a team, cduck and vtomole as code owners August 23, 2023 05:24
@pavoljuhas pavoljuhas requested a review from dabacon August 23, 2023 05:24
@CirqBot CirqBot added the Size: XS <10 lines changed label Aug 23, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 23, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.01% ⚠️

Comparison is base (6e38a27) 97.60% compared to head (634a1fc) 97.60%.
Report is 1 commits behind head on master.

❗ Current head 634a1fc differs from pull request most recent head 6247057. Consider uploading reports for the commit 6247057 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6256      +/-   ##
==========================================
- Coverage   97.60%   97.60%   -0.01%     
==========================================
  Files        1116     1116              
  Lines       95772    95770       -2     
==========================================
- Hits        93481    93479       -2     
  Misses       2291     2291              
Files Changed Coverage Δ
cirq-core/cirq/vis/heatmap_test.py 98.00% <100.00%> (-0.02%) ⬇️

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

@pavoljuhas pavoljuhas merged commit 999fd94 into quantumlib:master Aug 23, 2023
@pavoljuhas pavoljuhas deleted the avoid-random-coverage-deltas branch August 23, 2023 20:21
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
Eliminate random decreases of coverage.
Make codecov coverage check happy.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Size: XS <10 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants