-
Notifications
You must be signed in to change notification settings - Fork 92
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 hello-world and QAOA samples from Pasqal #607
Conversation
@microsoft-github-policy-service agree company="Pasqal" |
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Outdated
Show resolved
Hide resolved
Co-authored-by: Nicolas Ménétrier <nicolas.menetrier@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good ! I am just letting a couple of comments on syntax and QAOA :)
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Outdated
Show resolved
Hide resolved
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Show resolved
Hide resolved
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Show resolved
Hide resolved
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Show resolved
Hide resolved
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Show resolved
Hide resolved
Co-authored-by: Antoine Cornillot <61453516+a-corni@users.noreply.github.com>
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Outdated
Show resolved
Hide resolved
FYI while re-testing the HW notebook this morning, I faced a new error which I was not seeing ~15h ago:
We are investigating this internally. |
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Outdated
Show resolved
Hide resolved
This was due to an incorrect target being used, following the application of one of the suggestions. Now fixed by reinstating
Thanks @nmenetrier ! |
@kikomiss I think that all threads have been addressed, we might be good to go 😀 🤞 |
Hello ! Sending a little reminder to see if we can merge this PR 😀 |
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Outdated
Show resolved
Hide resolved
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Outdated
Show resolved
Hide resolved
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Outdated
Show resolved
Hide resolved
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Outdated
Show resolved
Hide resolved
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Outdated
Show resolved
Hide resolved
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Outdated
Show resolved
Hide resolved
...uantum-approximation-optimization/qaoa-for-quadratic-unconstrained-binary-optimization.ipynb
Show resolved
Hide resolved
@ArnaudPel I apologize, for the delay with review. I've left a couple of |
One of the final tasks required to complete the integration of Pasqal as an Azure Quantum provider was to provide sample notebooks, which is what this PR attempts to do.