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

Conversation

@adrianleh
Copy link
Contributor

Implementation of #602

Applies an operation windowing over input but with const target qubits
@ghost
Copy link

ghost commented Jul 7, 2022

CLA assistant check
All CLA requirements met.

@adrianleh adrianleh force-pushed the feature/apply-windowed branch from 2f0fefd to 3f87f06 Compare July 7, 2022 15:14
@adrianleh adrianleh force-pushed the feature/apply-windowed branch from 888d2bf to 0e6e244 Compare July 7, 2022 21:33
Copy link
Contributor

@tcNickolas tcNickolas left a comment

Choose a reason for hiding this comment

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

I left some comments for the main operation, the docs ones can be safely copied to the three other ones. For the sample, I would not copy it over, but rather add a reference from the variants to the first operation so that the example in it can be discovered:

    /// # See Also
    /// - Microsoft.Quantum.Arrays.ApplyWindowed

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 @adrianleh

Copy link
Contributor

@tcNickolas tcNickolas left a comment

Choose a reason for hiding this comment

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

Looks good, just one small formatting comment

Co-authored-by: Mariia Mykhailova <[email protected]>
@msoeken
Copy link
Member

msoeken commented Jul 20, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@msoeken msoeken merged commit 0ac7ac1 into microsoft:main Jul 20, 2022
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.

3 participants