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.
Summary
Fix a minor bug where the complex part of the
LinCombQFI
was assumed to be symmetric, when it is actually hermitian.Details and comments
For a quantum state
\phi
, the complex parts of the QFI is 4 times the following equationwhich is hermitian, i.e.
G_{ij} = conj(G_{ji})
. TheLinCombQFI
however implementedG_{ij} = G_{ji}
which led to wrong results forderivative_type
s being any other thanDerivativeType.REAL
.This does not include a reno since the QFIs are not yet released.