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

qml.AmplitudeAmplification #5160

Merged
merged 156 commits into from
Mar 15, 2024
Merged

qml.AmplitudeAmplification #5160

merged 156 commits into from
Mar 15, 2024

Conversation

KetpuntoG
Copy link
Contributor

@KetpuntoG KetpuntoG commented Feb 5, 2024

Second block of the Amplitude amplification algorithm toolkit project

Some use-examples:
https://colab.research.google.com/drive/1EUy9aRPAll7plaIBopbDkTQVo4hCGFs_?authuser=1#scrollTo=Bh0r7LMOiyCt

Copy link
Contributor

github-actions bot commented Feb 5, 2024

Hello. You may have forgotten to update the changelog!
Please edit doc/releases/changelog-dev.md with:

  • A one-to-two sentence description of the change. You may include a small working example for new features.
  • A link back to this PR.
  • Your name (or GitHub username) in the contributors section.

@KetpuntoG
Copy link
Contributor Author

[sc-55960]

Copy link
Contributor

@Jaybsoni Jaybsoni left a comment

Choose a reason for hiding this comment

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

Great work! Left a few comments. Happy to approve once they are addressed!

Copy link
Contributor

@Jaybsoni Jaybsoni left a comment

Choose a reason for hiding this comment

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

Nice 💯

Copy link
Contributor

@soranjh soranjh left a comment

Choose a reason for hiding this comment

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

Thanks @KetpuntoG, left a few final minor comments.

doc/releases/changelog-dev.md Outdated Show resolved Hide resolved
doc/releases/changelog-dev.md Show resolved Hide resolved
Copy link
Contributor

@trbromley trbromley left a comment

Choose a reason for hiding this comment

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

Thanks @KetpuntoG!

doc/_static/templates/subroutines/ampamp.png Outdated Show resolved Hide resolved
doc/releases/changelog-dev.md Outdated Show resolved Hide resolved
doc/releases/changelog-dev.md Outdated Show resolved Hide resolved
doc/releases/changelog-dev.md Outdated Show resolved Hide resolved
doc/releases/changelog-dev.md Outdated Show resolved Hide resolved
pennylane/templates/subroutines/amplitude_amplification.py Outdated Show resolved Hide resolved
@soranjh soranjh merged commit c2f9670 into master Mar 15, 2024
40 checks passed
@soranjh soranjh deleted the AmplitudeAmplification branch March 15, 2024 14:03
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.

5 participants