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.

Sign up for GitHub

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

Jump to bottom

Fix mutation on `conj`, `adjoint` #197

Merged

mofeing merged 1 commit into master from fix/mutation-on-conj

Sep 6, 2024

Member

mofeing commented Sep 6, 2024 •

edited

Loading

conj (and thus adjoint because it calls conj) was mutating the arrays of the original TensorNetwork when it shouldn't.

The reason is that we were doing a shallow copy (i.e. copy) instead of a deep copy (i.e. deepcopy).


          Fix mutation on conj, adjoint

744acc4

mofeing merged commit 03d2627 into master

6 checks passed

mofeing deleted the fix/mutation-on-conj branch

September 6, 2024 07:40

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet