Make kraus_to_choi
faster by using numpy
#2283
Merged
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.
np.tensordot
is faster than a triple loop. On my applications the function became up to 100 times faster.Also, the way to handle
dims
of the Choi matrix was incorrect (or I misunderstood something). It is explained in the comments.This was coded in collab with Rafael Haenel.
Also, please, could you make a release of the qutip 4 with all the new changes after this PR passes?