generate_cutting_experiments
returns coefficients, not weights
#412
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.
I noticed that
generate_cutting_experiments
really returns coefficients, not weights. While weights (e.g., as returned bygenerate_qpd_weights
) are all positive and can be rescaled arbitrarily, coefficients can be either positive or negative and cannot be rescaled arbitrarily. (Actually, the difference between weights and coefficients deserves some explanation, see #411.)The misleading jump from "coefficients" to
weights
is in the following snippet:https://github.com/Qiskit-Extensions/circuit-knitting-toolbox/blob/f33317fde7de9f2c6a5c8916ff301a5388e3eedb/circuit_knitting/cutting/cutting_experiments.py#L144-L145
Interestingly, this was already labeled
coefficients
at the call site incutting_experiments.py
.https://github.com/Qiskit-Extensions/circuit-knitting-toolbox/blob/f33317fde7de9f2c6a5c8916ff301a5388e3eedb/circuit_knitting/cutting/cutting_experiments.py#L144-L145