Closed apkille closed 2 months ago
Attention: Patch coverage is 80.30303%
with 26 lines
in your changes missing coverage. Please review.
Project coverage is 70.10%. Comparing base (
079ea5e
) to head (0d66026
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
looks good! Feel free to update the changelog / merge / release (or to postpone some of these steps if it is more convenient for your near term plans)
This PR addresses several discussions in #55:
prefactorscalings
evolved from its original purpose in a rather messy way. Now it straightforwardly pulls out scalar factors in multiplication and tensor products as well as commutators and anticommutators.flattenop
function that parses repeated addition, multiplication, and tensor products as a single expression. This prevents any ugly nested expressions for these operations.qexpand
function that manually expands expressions containing quantum objects. It is built on top ofSymbolicUtils
utilities for rule-rewriting in a way similar toqsimplify
.utils.jl
andlinalg.jl
and did some rearranging of the source code. For instance, it didn't make sense fordagger
to be inpredefined.jl
where we define things such as Pauli operators and harmonic oscillator operators, sodagger
was moved tolinalg.jl
.express.md
(which was merged in #57) to docs.