Add Ruff check stage to CI

.github/workflows/checks.yml

@@ -0,0 +1,30 @@
+# Runs linting on the codebase with Ruff
+name: Ruff linting
+
+defaults:
+  run:
+    shell: bash
+
+# Runs on the following conditions:
+# - Any push to a branch linked to a pull request
+#   -> if the pushed commit changes files within the codebase (in pyhdtoolkit/ or tests/ directories)
+#   -> or if the pushed commit changes the pyproject.toml configuration file
+on:
+  pull_request:
+    paths:
+      - 'pyhdtoolkit/**'
+      - 'tests/**'
+      - 'pyproject.toml'
+
+jobs:
+  linter:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v4
+
+      # This runs ruff check by default, our pyproject.toml has the config
+      - uses: astral-sh/ruff-action@v3
+        with:
+          version: "latest"
+          src: "pyhdtoolkit/"

docs/releases/v1.8.0.rst

@@ -21,6 +21,7 @@ Enhancements
 
 * The regex parsing in the `HTCondor` utilities has been made more robust.
 * The whole package now consistently makes use of postponed evaluations of annotations (see PEP 563).
+* Some operations have been fixed to adapt to the default behaviors of `pandas 3.x`.
 
 Documentation
 ~~~~~~~~~~~~~

pyhdtoolkit/cpymadtools/lhc/_routines.py

@@ -102,9 +102,9 @@ def do_kmodulation(
         logger.trace(f"Modulation of '{element}' - Setting '{powering_variable}' to {powering}")
         madx.globals[powering_variable] = powering
         twiss_df = get_ir_twiss(madx, ir=ir, centre=True, columns=["k1l", "l"], **kwargs)
-        results.loc[powering].K = twiss_df.loc[element.lower()].k1l / twiss_df.loc[element.lower()].l  # Store K
-        results.loc[powering].TUNEX = madx.table.summ.q1[0]  # Store Qx
-        results.loc[powering].TUNEY = madx.table.summ.q2[0]  # Store Qy
+        results.at[powering, "K"] = twiss_df.loc[element.lower()].k1l / twiss_df.loc[element.lower()].l  # Store K
+        results.at[powering, "TUNEX"] = madx.table.summ.q1[0]  # Store Qx
+        results.at[powering, "TUNEY"] = madx.table.summ.q2[0]  # Store Qy
 
     logger.debug(f"Resetting '{element}' powering")
     madx.globals[powering_variable] = old_powering