Make Prod compatible with qutrit operators
#5400
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
albi3ro
commented
Mar 18, 2024
mudit2812
approved these changes
Mar 18, 2024
This was referenced Mar 20, 2024
mudit2812
added a commit
that referenced
this pull request
Apr 12, 2024
**Context:** Measuring and applying operation functionality have been added to the `qutrit_mixed` module, these tests ensure that these operations are differentiable. Since support for qml.prod for qutrits has been implemented from [PR](#5400). It will be possible to change the method of state measurement without issue. The way expectation value measurement was done may have been overly complicated so this reducing that complication, while still allowing for product type observables. **Description of the Change:** Tests have been added that ensure that the `apply_operation` and `measure` functions retain differentiability for the new qutrit mixed-state device. Expectation value measurements have been changed to use diagonalizing gates and eigenvalues instead of trace. **Benefits:** Shows that the `apply_operation` and `measure` functions are differentiable. **Possible Drawbacks:** This PR does not remove the abstractions made in apply_operations to make the trace based measure expectation value work so there is some code left that may be overly abstract. **Related GitHub Issues:** N/A --------- Co-authored-by: Gabe PC <bottrill@student.ubc.ca> Co-authored-by: Mudit Pandey <mudit.pandey@xanadu.ai> Co-authored-by: Thomas R. Bromley <49409390+trbromley@users.noreply.github.com> Co-authored-by: Christina Lee <chrissie.c.l@gmail.com> Co-authored-by: Olivia Di Matteo <2068515+glassnotes@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[sc-59010]
The
tests/devices/test_default_qutrit.pytests were failing due toProd.eigvals(). The eigenvalues were not defined forTensor, but nowProdgives it its best effort. Unfortunately,Prod's best effort was not sufficient.This change improves
qml.utils.expand_vectorso it is compatible with qudit operators.