Skip to content

New Squin statements. #200

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Apr 25, 2025
Merged

New Squin statements. #200

merged 6 commits into from
Apr 25, 2025

Conversation

weinbe58
Copy link
Member

@weinbe58 weinbe58 commented Apr 24, 2025

This PR is addressing the new squin statements. However, after discussion with @kaihsin and @Roger-luo we have slightly modified the semantics to make things a bit cleaner:

  1. Boradcast is now an appliy-like statement, see Broadcast should be an Apply-like statement. #206
  2. PualiString is now CliffordString to enable better rewrites to STIM.

@weinbe58 weinbe58 linked an issue Apr 24, 2025 that may be closed by this pull request
@weinbe58 weinbe58 marked this pull request as draft April 24, 2025 20:36
Copy link

codecov bot commented Apr 24, 2025

Codecov Report

Attention: Patch coverage is 88.88889% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/bloqade/squin/op/stmts.py 84.61% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

Copy link
Contributor

github-actions bot commented Apr 24, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
6220 5290 85% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
src/bloqade/squin/op/stmts.py 98% 🟢
src/bloqade/squin/qubit.py 90% 🟢
TOTAL 94% 🟢

updated for commit: d9f62f2 by action🐍

@weinbe58 weinbe58 linked an issue Apr 25, 2025 that may be closed by this pull request
@weinbe58 weinbe58 marked this pull request as ready for review April 25, 2025 14:56
@weinbe58 weinbe58 merged commit 5f2cead into main Apr 25, 2025
11 checks passed
@johnzl-777
Copy link
Contributor

@weinbe58 is there a reason we don't want broadcast in the wire dialect as well? Or is the idea that there will be some lowering from a qubit broadcast to a bunch of wire applications?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants