feat: add PauliStringSum container#1270
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1270 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 170 171 +1
Lines 11011 11180 +169
Branches 1413 1439 +26
==========================================
+ Hits 11011 11180 +169 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
| @classmethod | ||
| def from_list(cls, terms: Iterable[tuple[numbers.Number, str | PauliString]]) -> PauliStringSum: | ||
| """Builds a ``PauliStringSum`` from a list of weighted Pauli strings.""" | ||
| return cls(terms) |
There was a problem hiding this comment.
Unnecessary if the constructor already does this
| for _, right in other_sum.terms | ||
| ) | ||
|
|
||
| def is_self_commuting(self) -> bool: |
There was a problem hiding this comment.
Can be a property, and the value determined on instantiation (instead of computing each time). Also, rename to all_terms_commute
| observables.append(coefficient * pauli.to_unsigned_observable(include_trivial=True)) | ||
| return Sum(observables) | ||
|
|
||
| def commutes_with(self, other: PauliStringSum | PauliString | str) -> bool: |
There was a problem hiding this comment.
And if all_terms_commute is determined at instantiation, this can be short circuited to False if all_terms_commute is False for either PauliSum.
| ) | ||
|
|
||
| @classmethod | ||
| def from_sum(cls, observable_sum: Sum) -> PauliStringSum: |
There was a problem hiding this comment.
nit: Move up right under __init__
| _OBSERVABLE_TO_FACTOR = {I: "I", X: "X", Y: "Y", Z: "Z"} | ||
|
|
||
|
|
||
| class PauliStringSum: |
There was a problem hiding this comment.
Should add utilities here to, so we can add PauliStrings together, check commutativity, etc.
CleanDev-Fix
force-pushed
the
codex/paulistring-sums-1256
branch
from
12f08c2 to
fdcfd25
Compare
speller26
left a comment
speller26
left a comment
There was a problem hiding this comment.
Thank you for adding the tests; can you also expand PauliString to allow addition and commutation checking?
2 participants
Summary
PauliStringSumcontainer for Pauli string arithmeticSumobservables, indexing, membership, scalar operations, and multiplication byPauliStringPauliStringCloses #1256.
Tests
uv run --extra test pytest test/unit_tests/braket/quantum_information test/unit_tests/braket/program_sets/test_circuit_binding.py -q -n 0uv run --extra test pytest test/unit_tests/braket/quantum_information/test_pauli_string.py test/unit_tests/braket/quantum_information/test_pauli_string_sum.py -q -n 0uv run --with tox tox -e linters_check