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

updating pennylane version #55

Merged
merged 21 commits into from
Jul 25, 2024
Merged

updating pennylane version #55

merged 21 commits into from
Jul 25, 2024

Conversation

bdoolittle
Copy link
Member

Updating Pennylane dependencies and associated code.

@bdoolittle bdoolittle self-assigned this Jul 25, 2024
@bdoolittle
Copy link
Member Author

@trbromley Do you want to take a quick review before merge? There is nothing breaking in here, but I did need to make some changes to accommodate various dependency updates.

@trbromley
Copy link
Collaborator

@trbromley Do you want to take a quick review before merge? There is nothing breaking in here, but I did need to make some changes to accommodate various dependency updates.

No problem, I'll take a look hopefully today 👍

@trbromley
Copy link
Collaborator

Ok I might not have capacity to give a full review, but it looks like there were no challenges with working with the latest PL outside of dependency/version/environment issues? I.e., nothing from the latest PL UI caused an issue 👍

@bdoolittle
Copy link
Member Author

Ok I might not have capacity to give a full review, but it looks like there were no challenges with working with the latest PL outside of dependency/version/environment issues? I.e., nothing from the latest PL UI caused an issue 👍

No worries, I'll proceed with the merge. I noted two things from the PennyLane update:

  1. The syntax/API for tensorflow optimization using the Keras solver changed. This was easy to resolve because the PennyLane docs showed the correct syntax.
  2. I saw a strange behavior in a test that might suggest that PennyLane fixed a bug involving midcircuit measurements. More specifically, in the LOCC (midcircuit measurement) tests, operations got added to the Quantum Tape that weren't there in previous tests. As far as I can tell, the current behavior is correct and the old behavior was wrong. The high-level functionality seems to work in both cases, so I'm not to concerned about digging into what changed here.

@bdoolittle bdoolittle merged commit b516495 into main Jul 25, 2024
3 checks passed
@bdoolittle bdoolittle deleted the update-qnetvo-version branch July 25, 2024 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants