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

Update lightning version for dynamic device shots #1359

Merged
merged 2 commits into from
Dec 6, 2024

Conversation

paul0403
Copy link
Contributor

@paul0403 paul0403 commented Dec 5, 2024

Context:
As part of the work to support dynamic shapes across catalyst, in #1310 the DeviceInitOp was changed to take in a proper SSA argument for shots, instead of having shots as just another entry in the DeviceInitOp's attribute dictionary.

Correspondingly, the backend devices' catalyst interfaces need to stop parsing the DeviceInitOp's attribute dictionary for shots. For the lightning devices, this was done in lightning repo at PennyLaneAI/pennylane-lightning#1017

We now update catalyst to track the lightning with that update.

Description of the Change:
Update lightning version to 0.40.0-dev30, which is the version of PennyLaneAI/pennylane-lightning#1017

@paul0403 paul0403 requested review from erick-xanadu and a team December 5, 2024 19:32
Copy link
Contributor

github-actions bot commented Dec 5, 2024

Hello. You may have forgotten to update the changelog!
Please edit doc/releases/changelog-dev.md on your branch 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.

@paul0403
Copy link
Contributor Author

paul0403 commented Dec 5, 2024

Wait for #451 for failures brought by version issues in pytest

@paul0403 paul0403 merged commit cb544fb into main Dec 6, 2024
41 checks passed
@paul0403 paul0403 deleted the paul0403/update_lightning_shots_parser branch December 6, 2024 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants