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

Covalent Cloud GPU tutorial #1101

Closed kessler-frost closed 2 months ago

kessler-frost commented 2 months ago

Title: Running GPU-accelerated quantum circuit simulations on Covalent Cloud using PennyLane

Summary:

Relevant references:

Possible Drawbacks:

Related GitHub Issues:


If you are writing a demonstration, please answer these questions to facilitate the marketing process.

kessler-frost commented 2 months ago

CC: @araghukas @santoshkumarradha

kessler-frost commented 2 months ago

One issue I had when converting this from a jupyter notebook to a .py file using notebook_to_demo.py was this error:

❯ python notebook_converter/notebook_to_demo.py covalent_cloud_gpu.ipynb --is-executable=False
Traceback (most recent call last):
  File "/Users/neptune/dev/qml/notebook_converter/notebook_to_demo.py", line 360, in <module>
    nb_py = convert_notebook_to_python(
  File "/Users/neptune/dev/qml/notebook_converter/notebook_to_demo.py", line 262, in convert_notebook_to_python
    cell_id = cell["id"]
KeyError: 'id'

I circumvented it by editing Line 262 to cell_id = i instead.

ikurecic commented 2 months ago

Hi @kessler-frost , thank you so much for making the updates and marking the pull request as ready for review! I can't push directly to your branch because it's in an organization-owned fork

Could you please add the following thumbnails to the pull request?

You can also transfer the included plot into the demonstrations_assets folder by following the instructions here with the syntax below.

##############################################################################
#.. figure:: ../_static/demonstrations_assets/<demo name>/image.png
#    :align: center
#    :width: 90%

It would be great if you could also complete the metadata file by following the readme instructions, so we can include everything in the review. (You can also look at a recently merged demo to check if anything might be missing, e.g., this one.) If you're not sure about the dates, put down estimates that we can change later.

For the visuals in the metadata file, you can include this:

    "previewImages": [  
        {
            "type": "thumbnail",
            "uri": "/_static/demonstration_assets/covalent_cloud_gpu/thumbnail_covalent_cloud_gpu.png"
        },
        {
            "type": "large_thumbnail",
            "uri": "/_static/large_demo_thumbnails/thumbnail_large_covalent_cloud_gpu.png"
        }
    ],

Once metadata can be validated, we can also check out the deployment preview as part of the review.

Please let me know if you have any questions or concerns.

kessler-frost commented 2 months ago

Thanks a lot @ikurecic ! Updated the PR ✅ . Hopefully I haven't missed anything else 🤞 .

You can also transfer the included plot into the demonstrations_assets folder by following the instructions here with the syntax below.

My bad actually I must've missed the right dir when converting it from .ipynb, updated ✅ .

github-actions[bot] commented 2 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.

kessler-frost commented 2 months ago

Thanks @josh146 !

josh146 commented 2 months ago

Hey @kessler-frost, would you be able to give us permissions to push to this branch :) That way we can help out with some of the operations here.

wjcunningham7 commented 2 months ago

Hey @kessler-frost, would you be able to give us permissions to push to this branch :) That way we can help out with some of the operations here.

You should have access now @josh146

josh146 commented 2 months ago

@wjcunningham7 thank you!

josh146 commented 2 months ago

Hey all! There was a slight accident when I tried to merge the master branch of the QML repo into this PR --- I think because the forked repo did not create a new branch (and instead used master as the branch), the sync caused this PR to be accidentally merged.

I've reverted the merge, and everything is find, but it doesn't look like I am able to re-open this PR. To try and fix this, I create a new PR, this time with a new branch name (not master), here: https://github.com/PennyLaneAI/qml/pull/1115/

@kessler-frost you should have full access to push to this PR.

Apologies!

kessler-frost commented 2 months ago

Thanks @josh146 !