Small tweaks

Author: pavoljuhas

cirq-core/cirq/contrib/paulistring/pauli_string_measurement_with_readout_mitigation.py

@@ -588,15 +588,14 @@ def measure_pauli_strings(
     normalized_circuits_to_pauli = _normalize_input_paulis(circuits_to_pauli)
 
     # Extract unique qubit tuples from input pauli strings
-    unique_qubit_tuples = set()
     if measure_on_full_support:
         full_support: set[ops.Qid] = set()
         for pauli_string_groups in normalized_circuits_to_pauli.values():
             for pauli_strings in pauli_string_groups:
                 for pauli_string in pauli_strings:
                     full_support.update(pauli_string.qubits)
         # One calibration group
-        unique_qubit_tuples.add(tuple(sorted(full_support)))
+        unique_qubit_tuples = {tuple(sorted(full_support))}
     else:
         unique_qubit_tuples = {
             tuple(_extract_readout_qubits(pauli_strings))

cirq-core/cirq/ops/phased_x_gate_test.py

@@ -261,7 +261,7 @@ def test_exponent_consistency(exponent, phase_exponent) -> None:
     """Verifies that instances of PhasedX gate expose consistent exponents."""
     g = cirq.PhasedXPowGate(exponent=exponent, phase_exponent=phase_exponent)
     assert g.exponent in [exponent, -exponent]
-    assert g.phase_exponent == [cirq.value.canonicalize_half_turns(g.phase_exponent)]
+    assert g.phase_exponent == cirq.value.canonicalize_half_turns(g.phase_exponent)
 
     g2 = cirq.PhasedXPowGate(exponent=g.exponent, phase_exponent=g.phase_exponent)
     assert g == g2

cirq-web/cirq_web/widget.py

@@ -117,6 +117,6 @@ def _to_script_tag(bundle_filename: str) -> str:
     Returns:
         The bundle file as string (readable by browser) wrapped in HTML script tags.
     """
-    bundle_file_path = os.path.join(_DIST_PATH, bundle_filename)
-    bundle_file_contents = Path(bundle_file_path).read_text(encoding='utf-8')
+    bundle_file_path = _DIST_PATH.joinpath(bundle_filename)
+    bundle_file_contents = bundle_file_path.read_text(encoding='utf-8')
     return f'<script>{bundle_file_contents}</script>'

dev_tools/codeowners_test.py

@@ -88,7 +88,8 @@ def test_codeowners(filepath, expected) -> None:
     # will be skipped
     codeowners = pytest.importorskip("codeowners")
 
-    owners = codeowners.CodeOwners(pathlib.Path(".github/CODEOWNERS").read_text(encoding="utf8"))
+    owners_text = pathlib.Path(".github/CODEOWNERS").read_text(encoding="utf8")
+    owners = codeowners.CodeOwners(owners_text)
     assert os.path.exists(
         filepath
     ), f"{filepath} should exist to avoid creating/maintaining meaningless codeowners rules."

dev_tools/conftest.py

@@ -113,7 +113,7 @@ def _check_for_reuse_or_recreate(env_dir: Path):
     def _create_base_env(base_dir: Path, pip_install_args: tuple[str, ...]):
         try:
             create_virtual_env(str(base_dir), [], sys.executable, True)
-            Path(base_dir / "testrun.uid").write_text(testrun_uid, encoding="utf8")
+            base_dir.joinpath("testrun.uid").write_text(testrun_uid, encoding="utf8")
             if pip_install_args:
                 shell_tools.run([f"{base_dir}/bin/pip", "install", *pip_install_args])
         except BaseException as ex: