Skip to content

Add snapshot support for default mixed #6659

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

Merged
merged 154 commits into from
Dec 10, 2024
Merged

Conversation

JerryChen97
Copy link
Contributor

@JerryChen97 JerryChen97 commented Dec 2, 2024

Context:
add a missing-for-long operation, snapshot. It was missing because it is necessarily based on the implmentation of measrue and sampling

refs: the counterparts in qml.devices.qubit.apply_operation, qml.devices.qutrit_mixed.apply_operation

Description of the Change:

Benefits:

Possible Drawbacks:

Related GitHub Issues:
[sc-73321]

JerryChen97 and others added 30 commits November 19, 2024 10:00
Mimicing the qutrit mixed one.
Copy link
Contributor

@astralcai astralcai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@JerryChen97 JerryChen97 enabled auto-merge (squash) December 10, 2024 20:02
Copy link
Contributor

@andrijapau andrijapau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@JerryChen97 JerryChen97 merged commit 42e6cba into master Dec 10, 2024
46 checks passed
@JerryChen97 JerryChen97 deleted the add-qubit_mixed/shadow branch December 10, 2024 20:20
mudit2812 added a commit that referenced this pull request Dec 13, 2024
**Context:**
add a missing-for-long operation, snapshot. It was missing because it is
necessarily based on the implmentation of measrue and sampling

refs: the counterparts in `qml.devices.qubit.apply_operation`,
`qml.devices.qutrit_mixed.apply_operation`

**Description of the Change:**

**Benefits:**

**Possible Drawbacks:**

**Related GitHub Issues:**
[sc-73321]

---------

Co-authored-by: Astral Cai <[email protected]>
Co-authored-by: Christina Lee <[email protected]>
Co-authored-by: Mudit Pandey <[email protected]>
Co-authored-by: lillian542 <[email protected]>
Co-authored-by: Andrija Paurevic <[email protected]>
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.

3 participants