Remove TODO markers for fixed issues (quantumlib#5871)

pavoljuhas · rht · commit 03d1a447b2db · 2023-04-30T23:41:45.000-04:00
Clean up TODO-s for fixed issues quantumlib#4328 and quantumlib#4532
diff --git a/cirq-core/cirq/ops/boolean_hamiltonian.py b/cirq-core/cirq/ops/boolean_hamiltonian.py
@@ -266,7 +266,6 @@ def _simplify_cnots_triplets(
             )
             # Since we removed the pivot, the length should be one fewer.
             cnots = [cnots[idx] for idx in new_idx]
-            # TODO(#4532): Speed up code by not returning early.
             return True, cnots
 
     return False, cnots
diff --git a/cirq-core/cirq/ops/clifford_gate_test.py b/cirq-core/cirq/ops/clifford_gate_test.py
@@ -480,7 +480,6 @@ def test_parses_single_qubit_gate(gate):
     itertools.product(_all_clifford_gates(), _paulis, (1.0, 0.25, 0.5, -0.5)),
 )
 def test_commutes_pauli(gate, pauli, half_turns):
-    # TODO(#4328) cirq.X**1 should be _PauliX instead of XPowGate
     pauli_gate = pauli if half_turns == 1 else pauli**half_turns
     q0 = cirq.NamedQubit('q0')
     mat = cirq.Circuit(gate(q0), pauli_gate(q0)).unitary()

Original file line number	Diff line number	Diff line change
`@@ -266,7 +266,6 @@ def _simplify_cnots_triplets(`
`266`	`266`	`)`
`267`	`267`	`# Since we removed the pivot, the length should be one fewer.`
`268`	`268`	`cnots = [cnots[idx] for idx in new_idx]`
`269`		`- # TODO(#4532): Speed up code by not returning early.`
`270`	`269`	`return True, cnots`
`271`	`270`
`272`	`271`	`return False, cnots`
Original file line number	Diff line number	Diff line change
`@@ -480,7 +480,6 @@ def test_parses_single_qubit_gate(gate):`
`480`	`480`	`itertools.product(_all_clifford_gates(), _paulis, (1.0, 0.25, 0.5, -0.5)),`
`481`	`481`	`)`
`482`	`482`	`def test_commutes_pauli(gate, pauli, half_turns):`
`483`		`- # TODO(#4328) cirq.X**1 should be _PauliX instead of XPowGate`
`484`	`483`	`pauli_gate = pauli if half_turns == 1 else pauli**half_turns`
`485`	`484`	`q0 = cirq.NamedQubit('q0')`
`486`	`485`	`mat = cirq.Circuit(gate(q0), pauli_gate(q0)).unitary()`