Skip to content

Fix split_layers description in CollectCliffords (backport #16837) - #16854

Merged
jakelishman merged 1 commit into
stable/2.5from
mergify/bp/stable/2.5/pr-16837
Aug 28, 2026
Merged

Fix split_layers description in CollectCliffords (backport #16837)#16854
jakelishman merged 1 commit into
stable/2.5from
mergify/bp/stable/2.5/pr-16837

Conversation

@mergify

@mergify mergify Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Fix #16833

It uupdates the split_layers argument description in CollectCliffords to clarify that collected blocks are split into layers of non-overlapping instructions, in other words, depth-1 sub-blocks.

Validation:

  • python -m compileall qiskit/transpiler/passes/optimization/collect_cliffords.py

AI/LLM disclosure

  • No part of this submission is LLM generated.
  • Some written text was generated by: Claude
  • Some submitted code was generated by:

This is an automatic backport of pull request #16837 done by [Mergify](https://mergify.com).

@mergify
mergify Bot requested a review from a team as a code owner August 28, 2026 11:24
@mergify
mergify Bot requested a review from raynelfss August 28, 2026 11:24
@github-actions github-actions Bot added documentation Something is not clear or an error documentation Changelog: None Do not include in the GitHub Release changelog. labels Aug 28, 2026
@jakelishman
jakelishman enabled auto-merge August 28, 2026 11:26
@jakelishman
jakelishman added this pull request to the merge queue Aug 28, 2026
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 33167027746

Coverage increased (+0.01%) to 87.76%

Details

  • Coverage increased (+0.01%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 3 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

3 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
crates/qasm2/src/lex.rs 3 92.03%

Coverage Stats

Coverage Status
Relevant Lines: 128925
Covered Lines: 113145
Line Coverage: 87.76%
Coverage Strength: 997037.34 hits per line

💛 - Coveralls

@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 28, 2026
@jakelishman
jakelishman added this pull request to the merge queue Aug 28, 2026
Merged via the queue into stable/2.5 with commit c1c03ab Aug 28, 2026
32 checks passed
@jakelishman
jakelishman deleted the mergify/bp/stable/2.5/pr-16837 branch August 28, 2026 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changelog: None Do not include in the GitHub Release changelog. documentation Something is not clear or an error documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants