Skip to content

[CQT-422] Add implementation of U, Z90 and MinusZ90 gate#636

Merged
elenbaasc merged 12 commits intodevelopfrom
ugate_v3
Nov 11, 2025
Merged

[CQT-422] Add implementation of U, Z90 and MinusZ90 gate#636
elenbaasc merged 12 commits intodevelopfrom
ugate_v3

Conversation

@eendebakpt
Copy link
Copy Markdown
Contributor

@eendebakpt eendebakpt commented Oct 31, 2025 •
edited
Loading

  • Z90 and MinusZ90 are a direct copy of the corresponding X and Y rotation implementations
  • U gate is implemented via Rn gates

@eendebakpt eendebakpt changed the title Add implementation of U gate Add implementation of U and Z90 gate Oct 31, 2025
davidbakker123
davidbakker123 reviewed Oct 31, 2025
View reviewed changes
Comment thread opensquirrel/default_instructions.py Outdated
@eendebakpt eendebakpt changed the title Add implementation of U and Z90 gate Add implementation of U, Z90 and MinusZ90 gate Nov 1, 2025
@elenbaasc
Copy link
Copy Markdown
Collaborator

elenbaasc commented Nov 3, 2025

Thanks for adding these changes!

Note that the Z90 and MinusZ90 gates are, semantically, already represented respectively by the S and Sdagger gates. Also adding gates to OpenSquirrel can only then be accessed through the CircuitBuilder (i.e., they are not part of the cQASM language), the OpenSquirrel reader will not be able to parse cQASM files with these gates.

Would you mind if I take over this PR and add some changes (without changing the goal of the PR)?

@eendebakpt
Copy link
Copy Markdown
Contributor Author

eendebakpt commented Nov 3, 2025

@elenbaasc Feel free to take over!

@elenbaasc elenbaasc changed the title Add implementation of U, Z90 and MinusZ90 gate [CQT-422] Add implementation of U, Z90 and MinusZ90 gate Nov 4, 2025
@elenbaasc
Refactor and move BSR composition functionality from merger pass inte…
203f766 
…rface to BSR module.

Add U gate exporting to cQASM v1 exporter.
Add tests for new gates.
Add entry to Changelog.
@elenbaasc
Restore utils __init__.py.
d32391d
@elenbaasc
Move check_mapper to general mapper pass.
fdbd69f
@elenbaasc
Fix Changelog.
8a2f7f0 
Use dict instead of Mapping for default gate sets.
@elenbaasc elenbaasc merged commit ce24647 into develop Nov 11, 2025
16 checks passed
@elenbaasc elenbaasc deleted the ugate_v3 branch November 11, 2025 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@davidbakker123 davidbakker123 davidbakker123 left review comments

@elenbaasc elenbaasc elenbaasc approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@eendebakpt @elenbaasc @davidbakker123