Merge pull request #7850 from jenshnielsen/ruff_0_15

Upgrade ruff to 0.15.0

Author: jenshnielsen

.pre-commit-config.yaml

@@ -1,7 +1,7 @@
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
     # Ruff version.
-    rev: v0.14.14
+    rev: v0.15.0
     hooks:
     - id: ruff-check
       types_or: [python, pyi, jupyter, toml]

docs/examples/DataSet/Measuring X as a function of time.ipynb

@@ -77,9 +77,7 @@
    "metadata": {},
    "outputs": [],
    "source": [
-    "noise = Parameter(\n",
-    "    \"noise\", label=\"Position\", unit=\"m\", get_cmd=lambda: np.random.randn()\n",
-    ")\n",
+    "noise = Parameter(\"noise\", label=\"Position\", unit=\"m\", get_cmd=np.random.randn)\n",
     "time = ElapsedTimeParameter(\"time\")"
    ]
   },

src/qcodes/dataset/data_set_protocol.py

@@ -4,7 +4,7 @@
 import os
 import warnings
 from collections.abc import Callable, Mapping, Sequence
-from enum import Enum
+from enum import StrEnum
 from importlib.metadata import entry_points
 from pathlib import Path
 from typing import (
@@ -555,6 +555,6 @@ def dependent_parameters(self) -> tuple[ParamSpecBase, ...]:
         return tuple(self.description.interdeps.dependencies.keys())
 
 
-class DataSetType(str, Enum):
+class DataSetType(StrEnum):
     DataSet = "DataSet"
     DataSetInMem = "DataSetInMem"

src/qcodes/instrument/delegate/instrument_group.py

@@ -44,7 +44,7 @@ def __init__(
         super().__init__(name=name, **kwargs)
 
         module_name = ".".join(submodules_type.split(".")[:-1])
-        instr_class_name = submodules_type.split(".")[-1]
+        instr_class_name = submodules_type.rsplit(".", maxsplit=1)[-1]
         module = importlib.import_module(module_name)
         instr_class = getattr(module, instr_class_name)
 

src/qcodes/instrument_drivers/QDev/QDac_channels.py

@@ -450,7 +450,7 @@ def _num_verbose(self, s: str) -> float:
         value descriptor.
         """
         if self.verbose.get_latest():
-            s = s.split(": ")[-1]
+            s = s.rsplit(": ", maxsplit=1)[-1]
         return float(s)
 
     def _current_parser(self, s: str) -> float:

src/qcodes/instrument_drivers/american_magnetics/AMI430_visa.py

@@ -72,15 +72,15 @@ def __init__(
             "enabled",
             label="Switch Heater Enabled",
             get_cmd=self._check_enabled,
-            set_cmd=lambda x: (self._enable() if x else self._disable()),
+            set_cmd=lambda x: self._enable() if x else self._disable(),
             vals=Bool(),
         )
         """Parameter enabled"""
         self.state: Parameter = self.add_parameter(
             "state",
             label="Switch Heater On",
             get_cmd=self._check_state,
-            set_cmd=lambda x: (self._on() if x else self._off()),
+            set_cmd=lambda x: self._on() if x else self._off(),
             vals=Bool(),
         )
         """Parameter state. Always False is the switch heater is not enabled"""
@@ -262,8 +262,8 @@ def __init__(
         """Parameter current_ramp_limit"""
         self.field_ramp_limit: Parameter = self.add_parameter(
             "field_ramp_limit",
-            get_cmd=lambda: self.current_ramp_limit(),
-            set_cmd=lambda x: self.current_ramp_limit(x),
+            get_cmd=self.current_ramp_limit,
+            set_cmd=self.current_ramp_limit,
             scale=1 / float(self.ask("COIL?")),
             unit="T/s",
         )

src/qcodes/instrument_drivers/oxford/MercuryiPS_VISA.py

@@ -73,7 +73,7 @@ def _temp_parser(response: str) -> float:
         response: What comes back from instrument.ask
 
     """
-    return float(response.split(":")[-1][:-1])
+    return float(response.rsplit(":", maxsplit=1)[-1][:-1])
 
 
 class OxfordMercuryWorkerPS(InstrumentChannel):

src/qcodes/instrument_drivers/oxford/triton.py

@@ -306,7 +306,7 @@ def _get_control_Bcomp_param(self, param: str) -> float | str | list[float] | No
         return self._get_response_value(self.ask(cmd[:-2]) + cmd[-2:])
 
     def _get_response(self, msg: str) -> str:
-        return msg.split(":")[-1]
+        return msg.rsplit(":", maxsplit=1)[-1]
 
     def _get_response_value(self, msg: str) -> float | str | list[float] | None:
         msg = self._get_response(msg)
@@ -551,12 +551,12 @@ def _parse_time(self, msg: str) -> str:
     def _parse_temp(self, msg: str) -> float | None:
         if "NOT_FOUND" in msg:
             return None
-        return float(msg.split("SIG:TEMP:")[-1].strip("K"))
+        return float(msg.rsplit("SIG:TEMP:", maxsplit=1)[-1].strip("K"))
 
     def _parse_pres(self, msg: str) -> float | None:
         if "NOT_FOUND" in msg:
             return None
-        return float(msg.split("SIG:PRES:")[-1].strip("mB")) * 1e3
+        return float(msg.rsplit("SIG:PRES:", maxsplit=1)[-1].strip("mB")) * 1e3
 
     def _recv(self) -> str:
         return super()._recv().rstrip()
@@ -579,7 +579,7 @@ def _set_pump_state(self, pump: str, state: str) -> None:
     def _get_parser_pump_speed(self, msg: str) -> float | None:
         if "NOT_FOUND" in msg:
             return None
-        return float(msg.split("SPD:")[-1].strip("Hz"))
+        return float(msg.rsplit("SPD:", maxsplit=1)[-1].strip("Hz"))
 
     def _add_temp_state(self) -> None:
         for i in range(1, 17):
@@ -599,7 +599,7 @@ def _set_temp_state(self, chan: str, state: str) -> None:
     def _get_parser_state(self, key: str, msg: str) -> str | None:
         if "NOT_FOUND" in msg:
             return None
-        return msg.split(f"{key}:")[-1]
+        return msg.rsplit(f"{key}:", maxsplit=1)[-1]
 
 
 Triton = OxfordTriton

src/qcodes/instrument_drivers/rigol/Rigol_DG1062.py

@@ -209,9 +209,11 @@ def __init__(
                 vals.Enum("INF", "MIN", "MAX", "HighZ"),
             ),
             get_parser=(
-                lambda value: "HighZ"
-                if float(value) > RigolDG1062Channel.max_impedance
-                else float(value)
+                lambda value: (
+                    "HighZ"
+                    if float(value) > RigolDG1062Channel.max_impedance
+                    else float(value)
+                )
             ),
             set_parser=lambda value: "INF" if value == "HighZ" else value,
         )

src/qcodes/instrument_drivers/tektronix/AWG5014.py

@@ -275,7 +275,7 @@ def __init__(
             get_cmd="AWGControl:SEQuencer:POSition?",
             set_cmd="SEQuence:JUMP:IMMediate {}",
             vals=vals.PermissiveInts(1),
-            set_parser=lambda x: round(x),
+            set_parser=round,
         )
         """Parameter sequence_pos"""