Skip to content

[CQT-430]: Make the interaction graph a property of the circuit expressed in terms of simple types#670

Merged
rares1609 merged 10 commits intodevelopfrom
CQT-430-Make-the-interaction-graph-a-property-of-the-Circuit-expressed-in-terms-of-simple-types
Mar 2, 2026
Merged

[CQT-430]: Make the interaction graph a property of the circuit expressed in terms of simple types#670
rares1609 merged 10 commits intodevelopfrom
CQT-430-Make-the-interaction-graph-a-property-of-the-Circuit-expressed-in-terms-of-simple-types

Conversation

@rares1609
Copy link
Contributor

No description provided.

Rares Oancea added 6 commits February 11, 2026 10:55
…nteraction-graph-a-property-of-the-Circuit-expressed-in-terms-of-simple-types
…nteraction-graph-a-property-of-the-Circuit-expressed-in-terms-of-simple-types
@rares1609 rares1609 requested a review from elenbaasc February 16, 2026 14:06
Copy link
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 the effort!

I think we should pass the circuit/self to the Circuit.map method, such that the passes themselves can pick up the interaction graph where needed.

Copy link
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.

You are very close, just some minor comments!

I fixed the test, so that the python version is considered.

…d docstring and slight modification to qgym_mapper.py
Copy link
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.

Ready to go!

@rares1609 rares1609 merged commit 49197bf into develop Mar 2, 2026
16 checks passed
@rares1609 rares1609 deleted the CQT-430-Make-the-interaction-graph-a-property-of-the-Circuit-expressed-in-terms-of-simple-types branch March 2, 2026 09:01
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