Skip to content
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

Add first draft of ZXTransformer to contrib. #6647

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dlyongemallo
Copy link
Contributor

This is a custom transformer which uses ZX-calculus through the PyZX library to perform circuit optimisation.

See issue #6585.

@dlyongemallo dlyongemallo requested review from vtomole, cduck and a team as code owners June 13, 2024 18:45
@dlyongemallo dlyongemallo requested a review from dstrain115 June 13, 2024 18:45
@CirqBot CirqBot added the size: L 250< lines changed <1000 label Jun 13, 2024
@NoureldinYosri NoureldinYosri self-assigned this Jun 13, 2024
@NoureldinYosri NoureldinYosri self-requested a review June 13, 2024 19:56
@dlyongemallo dlyongemallo marked this pull request as draft July 9, 2024 09:05
@dlyongemallo dlyongemallo force-pushed the zxtransformer branch 3 times, most recently from ef6bcf2 to 443aae0 Compare July 24, 2024 03:23
This is a custom transformer which uses ZX-calculus through the PyZX library to perform circuit optimisation.
@github-actions github-actions bot added the Stale label Aug 30, 2024
@github-actions github-actions bot closed this Sep 29, 2024
@mhucka mhucka added the status/stale This has been closed due to inactivity for an extended period of time. label Nov 18, 2024
@mhucka mhucka reopened this Dec 20, 2024
@NoureldinYosri
Copy link
Collaborator

@mhucka the contributer has indicated in the issue that they don't have time to work on this #6585 (comment)

@mhucka
Copy link
Contributor

mhucka commented Dec 20, 2024

That's correct, but they asked if someone else could take it over; they didn't say to close it. Moreover, the associated issue is still open. Since this PR was auto-closed by bot due to staleness, it seems like the right action is to reopen it and at least have a discussion about whether both the issue and the PR should be closed deliberately.

@github-actions github-actions bot removed the status/stale This has been closed due to inactivity for an extended period of time. label Dec 21, 2024
@dstrain115
Copy link
Collaborator

@naerabati Are you still interested in doing work on the ZX calculus transformer? Want to pick this up?

@naerabati
Copy link
Contributor

Sorry I couldn't answer earlier! I don't think right now I will have the time to finish the transformer either

@naerabati
Copy link
Contributor

@dstrain115 @dlyongemallo would you be able to tell me what else needs to implemented to complete this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: L 250< lines changed <1000
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants