Skip to content

[CQT-434] Implementation of transitions between different two-qubit gate semantics#678

Merged
davidbakker123 merged 15 commits into
developfrom
CQT-434-Implementation-of-transitions-between-different-two-qubit-gate-semantics
Apr 20, 2026
Merged

[CQT-434] Implementation of transitions between different two-qubit gate semantics#678
davidbakker123 merged 15 commits into
developfrom
CQT-434-Implementation-of-transitions-between-different-two-qubit-gate-semantics

Conversation

@davidbakker123
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Collaborator

@elenbaasc elenbaasc left a comment

Choose a reason for hiding this comment

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

Looks awesome, thanks for the effort!

Can't wait to add this to develop 😁

Comment thread opensquirrel/ir/two_qubit_gate.py Outdated
Comment thread opensquirrel/ir/two_qubit_gate.py Outdated
Comment thread opensquirrel/ir/two_qubit_gate.py Outdated
Comment thread opensquirrel/ir/two_qubit_gate.py Outdated
Comment thread opensquirrel/utils/matrix_expander.py Outdated
Comment thread opensquirrel/utils/matrix_expander.py Outdated
Comment thread tests/ir/semantics/test_canonical_gate_semantic.py
Comment thread tests/ir/semantics/test_canonical_gate_semantic.py
Comment thread tests/utils/test_matrix_expander.py
Comment thread tests/utils/test_matrix_expander.py
Copy link
Copy Markdown
Collaborator

@elenbaasc elenbaasc left a comment

Choose a reason for hiding this comment

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

Thanks for resolving the comments!

Just one very minor comment/suggestion left :)

... and be sure to resolve the merge conflicts (Rares just merged his changes, so I guess that has changed some things on develop)

Comment thread opensquirrel/ir/two_qubit_gate.py Outdated
@elenbaasc elenbaasc self-requested a review April 17, 2026 09:01
@davidbakker123 davidbakker123 merged commit 7e455d5 into develop Apr 20, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants