Qutrit mixed apply operation #5032
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
…nylane_qutrit_mixed into qutrit_mixed_initialization
Co-authored-by: Mudit Pandey <mudit.pandey@xanadu.ai>
Co-authored-by: Mudit Pandey <mudit.pandey@xanadu.ai>
Co-authored-by: Mudit Pandey <mudit.pandey@xanadu.ai>
Co-authored-by: Mudit Pandey <mudit.pandey@xanadu.ai>
glassnotes
reviewed
Jan 17, 2024
tests/devices/qutrit_mixed/test_qutrit_mixed_apply_operation.py
Outdated
Show resolved
Hide resolved
tests/devices/qutrit_mixed/test_qutrit_mixed_apply_operation.py
Outdated
Show resolved
Hide resolved
tests/devices/qutrit_mixed/test_qutrit_mixed_apply_operation.py
Outdated
Show resolved
Hide resolved
tests/devices/qutrit_mixed/test_qutrit_mixed_apply_operation.py
Outdated
Show resolved
Hide resolved
tests/devices/qutrit_mixed/test_qutrit_mixed_apply_operation.py
Outdated
Show resolved
Hide resolved
albi3ro
reviewed
Jan 22, 2024
glassnotes
approved these changes
Jan 22, 2024
tests/devices/qutrit_mixed/test_qutrit_mixed_apply_operation.py
Outdated
Show resolved
Hide resolved
mudit2812
approved these changes
Jan 22, 2024
There was a problem hiding this comment.
Looks good! Reminder to resolve pending comments. Nice work @Gabriel-Bottrill
tests/devices/qutrit_mixed/test_qutrit_mixed_apply_operation.py
Outdated
Show resolved
Hide resolved
Co-authored-by: Olivia Di Matteo <2068515+glassnotes@users.noreply.github.com>
Co-authored-by: Olivia Di Matteo <2068515+glassnotes@users.noreply.github.com>
Co-authored-by: Mudit Pandey <mudit.pandey@xanadu.ai>
|
@albi3ro, I had to use your suggestion to input |
|
@Gabriel-Bottrill, we're good to go to merge this! Would you mind updating your branch as it's a blocker for merging. |
|
Thanks @trbromley - it looks like there are some test failures that I don't think are related to this PR - in the details I'm seeing errors thrown by tests related to Catalyst and |
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.
Context:
Currently the qutrit_mixed device is being developed this is a necessary addition to the overall project as this applies operations to a qutrit mixed state. This is a prerequisite for other functionality relating to the qutrit mixed device for noisy qutrit simulation.
Description of the Change:
Added functionality for applying operations to a qutrit mixed-state. The new
apply_operationfunction can be used to apply gates and Channels to a qutrit mixed-state.Benefits:
Allows for Channels and operations to be applied to a mixed state, will be used to add execute functionality to qutrit mixed-state device allowing for noisy qutrit simulation
Possible Drawbacks:
Abstracting for qubits and more generally qutrits may have added challenges and will require a reforctor or copied code code-smell.
Related GitHub Issues:
N/A