-
Notifications
You must be signed in to change notification settings - Fork 39
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
Added pickle support for lightning.gpu device's DevPool
attribute
#772
Conversation
[sc-66035] |
pennylane_lightning/core/src/simulators/lightning_gpu/bindings/LGPUBindings.hpp
Outdated
Show resolved
Hide resolved
Hello. You may have forgotten to update the changelog!
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #772 +/- ##
==========================================
+ Coverage 90.30% 98.48% +8.17%
==========================================
Files 81 109 +28
Lines 11360 15801 +4441
==========================================
+ Hits 10259 15562 +5303
+ Misses 1101 239 -862 ☔ View full report in Codecov by Sentry. |
DevPool
attributeDevPool
attribute
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.
No blockers from me --- thanks @Jaybsoni
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.
Thanks @Jaybsoni! I have a few suggestions and a question. I will approve when you address them and update the changelog.
pennylane_lightning/core/src/simulators/lightning_gpu/bindings/LGPUBindings.hpp
Show resolved
Hide resolved
Co-authored-by: Ali Asadi <10773383+maliasadi@users.noreply.github.com>
Co-authored-by: Ali Asadi <10773383+maliasadi@users.noreply.github.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.
One more suggestion with the latest changes. Looks good to me otherwise 🎉
Co-authored-by: Ali Asadi <10773383+maliasadi@users.noreply.github.com>
…772) **Context:** Allowing the `qml.device("lightning.gpu", wires=...)._dp` argument, which is a python `DevPools()` object to be pickle-able through the C++ layer. **Description of the Change:** - Updated Devpools object to have a `refresh()` method - Updated TSQueue to have a clear method - Add bindings for pickle to DevPool --------- Co-authored-by: ringo-but-quantum <github-ringo-but-quantum@xanadu.ai> Co-authored-by: Ali Asadi <10773383+maliasadi@users.noreply.github.com>
Context:
Allowing the
qml.device("lightning.gpu", wires=...)._dp
argument, which is a pythonDevPools()
object to be pickle-able through the C++ layer.Description of the Change:
refresh()
method