search

PennyLaneAI / qml

Introductions to key concepts in quantum programming, as well as tutorials and implementations from cutting-edge quantum computing research.
https://pennylane.ai/qml
Apache License 2.0
517 stars 183 forks source link

Add Catalyst demos #1076

Closed josh146 closed 3 months ago

github-actions[bot] commented 3 months ago

👋 Hey, looks like you've updated some demos!

🐘 Don't forget to update the dateOfLastModification in the associated metadata files so your changes show up on pennylane.ai.

Please hide this comment once the field(s) are updated. Thanks!

github-actions[bot] commented 3 months ago

Thank you for opening this pull request.

You can find the built site at this link.

Deployment Info:

Note: It may take several minutes for updates to this pull request to be reflected on the deployed site.

KetpuntoG commented 3 months ago

Also, to clarify, the reason I put the ci-run-all-demos label was to check that there were no problems with the other demos as the requirements are being modified in this PR. Everything went smoothly so I remove the tag 💪

josh146 commented 3 months ago

thanks @KetpuntoG! I took into account your suggestions

KetpuntoG commented 3 months ago

Thank you for working on the suggestions @josh146 , looks great! after @dime10 answers some open comments, we can merge 🪄 (remember to add it to the performance category if you see fit)

josh146 commented 3 months ago

Thanks @isaacdevlugt! Rather than hide the warning, I'm tempted to keep it there 🤔 for the sole reason that: I can't see a reason for this warning (I'm never using autograd) and so the warning is not actionable.

So it sounds like a bug we would want to fix in the library itself.

Also, if a user downloads this demo and sees the warning themselves, and it's not in the original demo, they might wonder what they are doing wrong.