Skip to content

Fixing bugs related to kirin 0.17.3 version. #214

New issue

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

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

Already on GitHub? # to your account

Merged
merged 4 commits into from
Apr 29, 2025

Conversation

weinbe58
Copy link
Member

@weinbe58 weinbe58 commented Apr 28, 2025

fixes:

  • ParallelToUop I just added a setdefault so that it would register the dictionary value and then overwrite it later
  • For RewriteNoiseModel: I split up the rewrites to be correct when having the new traversal order.

@weinbe58 weinbe58 requested a review from Roger-luo April 29, 2025 15:06
Copy link
Contributor

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
6234 5344 86% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
src/bloqade/qasm2/passes/noise.py 100% 🟢
src/bloqade/qasm2/rewrite/heuristic_noise.py 95% 🟢
src/bloqade/qasm2/rewrite/uop_to_parallel.py 68% 🟢
TOTAL 88% 🟢

updated for commit: a5e5cec by action🐍

Copy link

codecov bot commented Apr 29, 2025

Codecov Report

Attention: Patch coverage is 96.42857% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/bloqade/qasm2/rewrite/heuristic_noise.py 95.45% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@weinbe58 weinbe58 merged commit 009b73d into main Apr 29, 2025
11 checks passed
@weinbe58 weinbe58 deleted the phil/fix-kirin-patch-issues branch April 29, 2025 15:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant