-
Notifications
You must be signed in to change notification settings - Fork 11
feat!: Add ReplaceBoolPass
#854
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
Changes from all commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
a836f12
chore!: BUmp to hugr-0.20
doug-q 8dee0ad
fmt
doug-q 764152a
update extensions
doug-q f7c7737
how do you deal with Infallible?
doug-q d7178ba
fix
doug-q 36ba773
Bump to latest rev
mark-koch 7880723
Regenerate json extensions
mark-koch c2b53c3
bump to latest rev
ss2165 2fac8ce
Add linearize arrays pass to qsystem pass
mark-koch 1dda29b
Bump to latest rev
mark-koch bf4c50b
gen extensions
ss2165 3eee1b5
bump commit
ss2165 0bf8515
bump rev
ss2165 733e772
feat!: bump msrv to 1.85
ss2165 22ce914
Merge branch 'ss/msrv' into update-hugr-202505
ss2165 beb408a
Merge branch 'main' into update-hugr-202505
ss2165 ec5090e
regenerate eccs
ss2165 6e082c2
expect deprecations
ss2165 ee6cc09
pin to hugr-py commit
ss2165 d6fa53f
wip bump to rc.1
aborgna-q d9097a5
fix infinite type recursion in CircuitHash
aborgna-q 120b290
more test fixes
aborgna-q 40209c7
recompile eccs
aborgna-q 5560e5d
squash remaining rust-side errors
aborgna-q a3a0817
✅
aborgna-q 702db23
Update hugr, failing SimpleReplacement::try_new
aborgna-q 8e5a635
Update hugr with funcdecl private fields
aborgna-q bb6ce4c
fix subgraph signature
lmondada c008735
use released hugr
aborgna-q 69840e5
Start lowering implementation
tatiana-s 7f29af7
Add further replacements
tatiana-s 7a38e6f
Rename bool ops
tatiana-s 3b51e36
Various pass improvements
tatiana-s 27f39c0
Change measure op signatures and replace
tatiana-s a00db20
Fix failing tests
tatiana-s 013e6f1
Improve tests
tatiana-s 1577e83
Minor fixes
tatiana-s cda725e
Add tag to measure replacements
tatiana-s e0bca55
Fixes after rebase
tatiana-s 2e0bffe
Actual bump
tatiana-s 121e7a0
More rebasing fixes
tatiana-s 645478c
Update
tatiana-s d4370d7
hugr=py dependency link
tatiana-s ee09996
fmt
ss2165 5597f79
update extension
ss2165 30acb3e
Add const lowering
tatiana-s 9e1bef3
fix: constant sum type
ss2165 7b089db
remove comment
ss2165 88ffa84
Change pass order
tatiana-s c18e683
Merge remote-tracking branch 'origin/main' into ts/replace-bool
tatiana-s 17cee70
Minor fixes
tatiana-s 3ecd0d5
More fixes
tatiana-s 8deef1b
Re-add xor
tatiana-s 848f256
Merge remote-tracking branch 'origin/main' into ts/replace-bool
tatiana-s File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bool_typeis not new or even touched in this PR but I have to wonder whetherbool_type(opaque) vsbool_t(Hugr prelude / Sum) is the best naming scheme ;-). Might this be a good time to consider renamingbool_typeto e.g.opaque_bool(_t),tket2_bool, something like that?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, this is going to be annoying. I think my preference is
opaque_bool_t