Skip to content

Conversation

@Roger-luo
Copy link
Collaborator

@Roger-luo Roger-luo commented Apr 8, 2025

┆Issue is synchronized with this Jira Task by Unito

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 25 out of 25 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/bloqade/qasm2/dialects/uop/stmts.py:239

  • The CSwap class is currently using 'ccx' as its name, which appears to be a typo. Consider renaming it to 'cswap' to properly reflect the controlled swap operation.
name = "ccx"

@Roger-luo Roger-luo added the priority: high high priority, blocking milestones, time sensitive label Apr 9, 2025
@Roger-luo Roger-luo added this to the v0.1.0 milestone Apr 9, 2025
@weinbe58 weinbe58 force-pushed the roger/upgrade-kirin-0-16 branch 2 times, most recently from d7c8b3a to 37ec08f Compare April 12, 2025 14:00
@codecov
Copy link

codecov bot commented Apr 12, 2025

@github-actions
Copy link
Contributor

github-actions bot commented Apr 12, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
6142 5262 86% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
src/bloqade/noise/native/stmts.py 100% 🟢
src/bloqade/pyqrack/noise/native.py 100% 🟢
src/bloqade/qasm2/dialects/core/stmts.py 100% 🟢
src/bloqade/qasm2/dialects/expr/_init_.py 100% 🟢
src/bloqade/qasm2/dialects/expr/stmts.py 100% 🟢
src/bloqade/qasm2/dialects/glob.py 100% 🟢
src/bloqade/qasm2/dialects/indexing.py 84% 🟢
src/bloqade/qasm2/dialects/inline.py 75% 🟢
src/bloqade/qasm2/dialects/noise.py 100% 🟢
src/bloqade/qasm2/dialects/parallel.py 86% 🟢
src/bloqade/qasm2/dialects/uop/stmts.py 100% 🟢
src/bloqade/qasm2/groups.py 98% 🟢
src/bloqade/qasm2/parse/lowering.py 58% 🟢
src/bloqade/qasm2/passes/py2qasm.py 92% 🟢
src/bloqade/qasm2/passes/qasm2py.py 94% 🟢
src/bloqade/qbraid/lowering.py 95% 🟢
src/bloqade/squin/op/stmts.py 100% 🟢
src/bloqade/squin/qubit.py 89% 🟢
src/bloqade/squin/wire.py 100% 🟢
src/bloqade/stim/dialects/aux/lowering.py 93% 🟢
src/bloqade/stim/dialects/aux/stmts/annotate.py 100% 🟢
src/bloqade/stim/dialects/aux/stmts/const.py 100% 🟢
src/bloqade/stim/dialects/collapse/stmts/measure.py 100% 🟢
src/bloqade/stim/dialects/collapse/stmts/pp_measure.py 100% 🟢
src/bloqade/stim/dialects/collapse/stmts/reset.py 100% 🟢
src/bloqade/stim/dialects/gate/stmts/base.py 100% 🟢
src/bloqade/stim/dialects/gate/stmts/pp.py 100% 🟢
src/bloqade/stim/dialects/noise/stmts.py 100% 🟢
TOTAL 95% 🟢

updated for commit: c6320c5 by action🐍

@github-actions
Copy link
Contributor

github-actions bot commented Apr 12, 2025

PR Preview Action v1.6.1
Preview removed because the pull request was closed.
2025-04-12 16:45 UTC

Copy link
Member

@weinbe58 weinbe58 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Roger-luo Roger-luo merged commit 16ec8ec into main Apr 12, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: high high priority, blocking milestones, time sensitive

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants