Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

Conversation

@rajkk1
Copy link
Contributor

@rajkk1 rajkk1 commented Aug 4, 2022

Implementation of #607

@ghost
Copy link

ghost commented Aug 4, 2022

CLA assistant check
All CLA requirements met.

Copy link
Member

@msoeken msoeken 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 contribution. That looks really great! I left some comments.

rajkk1 and others added 15 commits August 19, 2022 15:56
Moved potential efficiency comments of fanning out control qubits from summary to inside the code
Split original summary to summary and remarks
Copy link
Member

@msoeken msoeken 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 incorporating the comments. I have left one more comment that includes an example for the docs.

@msoeken
Copy link
Member

msoeken commented Aug 22, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@msoeken
Copy link
Member

msoeken commented Aug 22, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@msoeken
Copy link
Member

msoeken commented Aug 22, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@msoeken msoeken merged commit 2991f67 into microsoft:main Aug 23, 2022
@rajkk1
Copy link
Contributor Author

rajkk1 commented Aug 23, 2022 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants