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

MPS: Support snapshot_density_matrix #948

Merged
merged 7 commits into from
Sep 25, 2020

Conversation

merav-aharoni
Copy link
Contributor

Summary

Added support for snapshot_density_matrix in the MPS simulator.

Details and comments

This involved change in the density_matrix_internal function, because it previously did not support reordering of the qubits.
This also required a change in expectation_value that depends on density_matrix: We no longer require reordering of the qubits here, because this is now implemented in density_matrix.

@merav-aharoni merav-aharoni changed the title [WIP] MPS: Support snapshot_density_matrix MPS: Support snapshot_density_matrix Sep 23, 2020
Copy link
Member

@chriseclectic chriseclectic left a comment

Choose a reason for hiding this comment

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

Looks good. Can you add a release note

@chriseclectic chriseclectic added this to the Aer 0.7 milestone Sep 24, 2020
@chriseclectic chriseclectic added the Changelog: New Feature Include in the Added section of the changelog label Sep 24, 2020
@chriseclectic chriseclectic merged commit a8caf20 into Qiskit:master Sep 25, 2020
@chriseclectic
Copy link
Member

@merav-aharoni Since we're having trouble with the CI tests taking so long to run I've merged this, so can you add the release note for this PR to #915

@merav-aharoni merav-aharoni deleted the new-density-matrix branch November 4, 2020 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: New Feature Include in the Added section of the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants