Closed apkille closed 2 months ago
I left a couple of minor comments, nothing to interesting, and you might have already been thinking about some of these things.
I will convert this to a draft. When it is ready for review and it has all tests pass, just convert it back and request a review.
Attention: Patch coverage is 63.09524%
with 62 lines
in your changes missing coverage. Please review.
Project coverage is 62.78%. Comparing base (
8a75466
) to head (af3c073
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I updated functionalities of (anti)commutator and dagger. Also added operator properties (hermitian, unitary, and commutative) and wrote corresponding simplification rules, which are all tested. I added doctests as well for basic operations.
Woohoo!
Excellent! 😁
Defined constructors for bra, product of operators, commutator, anti commutator, and dagger. Also added scaling and a few simplification rules.