FIX: determine numerical algorithm correctly (#501)

* DX: add `poe upgrade` task
* DX: group Poe the Poet tasks
* MAINT: add type hints for `Printer`
* MAINT: implement updates from auto-formatters
* MAINT: upgrade lock files

Author: redeboer

.pre-commit-config.yaml

@@ -14,7 +14,7 @@ repos:
       - id: check-useless-excludes
 
   - repo: https://github.com/ComPWA/policy
-    rev: 0.8.9
+    rev: 0.8.13
     hooks:
       - id: check-dev-files
         args:
@@ -59,7 +59,7 @@ repos:
             metadata.vscode
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.15.10
+    rev: v0.15.16
     hooks:
       - id: ruff-check
         args: [--fix]
@@ -113,7 +113,7 @@ repos:
           - --in-place
 
   - repo: https://github.com/python-jsonschema/check-jsonschema
-    rev: 0.37.1
+    rev: 0.37.2
     hooks:
       - id: check-jsonschema
         name: Check CITATION.cff
@@ -153,6 +153,6 @@ repos:
         types_or: [python, pyi, jupyter]
 
   - repo: https://github.com/astral-sh/uv-pre-commit
-    rev: 0.11.6
+    rev: 0.11.19
     hooks:
       - id: uv-lock

.taplo.toml

@@ -13,7 +13,10 @@ reorder_keys = true
 
 [[rule]]
 include = ["**/pyproject.toml"]
-keys = ["tool.poe.tasks"]
+keys = [
+    "tool.poe.groups",
+    "tool.poe.tasks",
+]
 
 [rule.formatting]
 reorder_arrays = false

docs/analyticity/integration-algorithms.ipynb

@@ -467,7 +467,7 @@
     "\n",
     "w.jslink((sliders[\"disable_limit\"], \"value\"), (sliders[\"limit\"], \"disabled\"))\n",
     "sliders[\"algorithm_name\"].observe(on_algorithm_change, names=\"value\")\n",
-    "sliders[\"algorithm_name\"].value = \"romberg\"  # trigger changes\n",
+    "sliders[\"algorithm_name\"].value = \"romberg\"  # trigger changes  # ty:ignore[invalid-assignment]\n",
     "timer_box = cast(\"w.ValueWidget\", w.HTML())\n",
     "ui = w.VBox([\n",
     "    tabs := w.Tab([\n",

docs/dynamics.ipynb

@@ -509,7 +509,7 @@
     "    m0_val = kwargs[\"m0\"]\n",
     "    thr = kwargs[\"m1\"] + kwargs[\"m2\"]\n",
     "    if LINES is None:\n",
-    "        LINES = [\n",
+    "        LINES = (\n",
     "            ax1.plot(x, z.real, label=\"real\")[0],\n",
     "            ax1.plot(x, z.imag, label=\"imag\")[0],\n",
     "            ax1.plot(x, np.abs(z) ** 2, label=\"absolute\", c=\"black\", ls=\"dotted\")[0],\n",
@@ -520,7 +520,7 @@
     "            ax2.axvline(m0_val, c=\"red\", label=f\"${sp.latex(m0)}$\", **style),\n",
     "            ax1.axvline(thr, c=\"black\", label=f\"${sp.latex(m1 + m2)}$\", **style),\n",
     "            ax2.axvline(thr, c=\"black\", label=f\"${sp.latex(m1 + m2)}$\", **style),\n",
-    "        ]\n",
+    "        )\n",
     "    else:\n",
     "        LINES[0].set_ydata(z.real)\n",
     "        LINES[1].set_ydata(z.imag)\n",
@@ -585,7 +585,7 @@
    "name": "python",
    "nbconvert_exporter": "python",
    "pygments_lexer": "ipython3",
-   "version": "3.13.12"
+   "version": "3.13.13"
   }
  },
  "nbformat": 4,

docs/dynamics/k-matrix.ipynb

@@ -1294,7 +1294,7 @@
     "sliders[\"complex_rendering\"] = w.RadioButtons(\n",
     "    description=R\"\\(s\\)-plane plot\",\n",
     "    options=[\"imag\", \"real\", \"abs\"],\n",
-    ")\n",
+    ")  # ty:ignore[invalid-assignment]\n",
     "\n",
     "# Create cut-off slider for z-direction\n",
     "sliders[\"z_cutoff\"] = w.FloatSlider(\n",
@@ -1622,7 +1622,7 @@
     "    sliders[\"complex_rendering\"] = w.RadioButtons(\n",
     "        description=R\"\\(s\\)-plane plot\",\n",
     "        options=[\"imag\", \"real\", \"abs\"],\n",
-    "    )\n",
+    "    )  # ty:ignore[invalid-assignment]\n",
     "\n",
     "    # Create cut-off slider for z-direction\n",
     "    sliders[\"z_cutoff\"] = w.FloatSlider(\n",
@@ -1733,7 +1733,7 @@
    "name": "python",
    "nbconvert_exporter": "python",
    "pygments_lexer": "ipython3",
-   "version": "3.13.12"
+   "version": "3.13.13"
   }
  },
  "nbformat": 4,

pyproject.toml

@@ -165,49 +165,10 @@ isolated = true
 no-group = "dev"
 type = "uv"
 
-[tool.poe.tasks.all]
-help = "Run all continuous integration (CI) tasks locally"
-ignore_fail = "return_non_zero"
-sequence = [
-    "benchmark",
-    "cov",
-    "docnb-force",
-    "linkcheck",
-    "style",
-    "test-all",
-]
-
-[tool.poe.tasks.benchmark]
-cmd = """
-pytest benchmarks \
-    --benchmark-autosave \
-    --benchmark-json benchmarks/output.json \
-    --durations=0 \
-    -k benchmark
-"""
-executor = {group = "test"}
-help = "Run benchmark tests and visualize performance"
-
-[tool.poe.tasks.cov]
-cmd = """
-pytest \
-    --cov=ampform \
-    --cov-fail-under=75 \
-    --cov-report=html \
-    --cov-report=xml \
-    --numprocesses=auto \
-    ${paths}
-"""
-executor = {group = "test"}
-help = "Compute how much of the source code is covered by tests"
-
-[[tool.poe.tasks.cov.args]]
-default = "src tests"
-multiple = true
-name = "paths"
-positional = true
+[tool.poe.groups.doc]
+heading = "Documentation"
 
-[tool.poe.tasks.doc]
+[tool.poe.groups.doc.tasks.doc]
 cmd = """
 sphinx-build \
     --builder=html \
@@ -221,7 +182,7 @@ env = {PYTHONWARNINGS = ""}
 executor = {group = "doc"}
 help = "Build documentation and API"
 
-[tool.poe.tasks.doclive]
+[tool.poe.groups.doc.tasks.doclive]
 cmd = """
 sphinx-autobuild \
     --builder=html \
@@ -249,28 +210,17 @@ sphinx-autobuild \
 executor = {group = "doc", with = "sphinx-autobuild"}
 help = "Set up a server to directly preview changes to the HTML pages"
 
-[tool.poe.tasks.docnb]
+[tool.poe.groups.doc.tasks.docnb]
 env = {EXECUTE_NB = "yes"}
 help = "Build documentation and API with cached notebook execution"
 sequence = ["doc"]
 
-[tool.poe.tasks.docnb-force]
-env = {FORCE_EXECUTE_NB = "yes"}
-help = "Build documentation and API with notebook execution (no cache)"
-sequence = ["doc"]
-
-[tool.poe.tasks.docnblive]
+[tool.poe.groups.doc.tasks.docnblive]
 env = {EXECUTE_NB = "yes"}
 help = "Set up a server to directly preview changes to the HTML pages with cached notebook execution"
 sequence = ["doclive"]
 
-[tool.poe.tasks.lab]
-args = [{name = "paths", default = "", positional = true}]
-cmd = "jupyter lab ${paths}"
-executor = {group = ["jupyter", "notebooks"]}
-help = "Launch Jupyter Lab"
-
-[tool.poe.tasks.linkcheck]
+[tool.poe.groups.doc.tasks.linkcheck]
 cmd = """
 sphinx-build \
     --builder=linkcheck \
@@ -283,29 +233,60 @@ env = {PYTHONWARNINGS = ""}
 executor = {group = "doc"}
 help = "Check external links in the documentation (requires internet connection)"
 
-[tool.poe.tasks.nb]
+[tool.poe.groups.notebook]
+heading = "Notebooks"
+
+[tool.poe.groups.notebook.tasks.lab]
+args = [{name = "paths", default = "", positional = true}]
+cmd = "jupyter lab ${paths}"
+executor = {group = ["jupyter", "notebooks"]}
+help = "Launch Jupyter Lab"
+
+[tool.poe.groups.notebook.tasks.nb]
 args = [{name = "paths", default = "docs", multiple = true, positional = true}]
 cmd = "pytest --nbmake --nbmake-timeout=0 ${paths}"
 executor = {group = "notebooks", with = "nbmake"}
 help = "Run all notebooks"
 
-[tool.poe.tasks.style]
-cmd = "pre-commit run --all-files"
-executor = {extra = "scipy", group = "style"}
-help = "Perform all linting, formatting, and spelling checks"
+[tool.poe.groups.test]
+heading = "Testing"
+
+[tool.poe.groups.test.tasks.cov]
+cmd = """
+pytest \
+    --cov=ampform \
+    --cov-fail-under=75 \
+    --cov-report=html \
+    --cov-report=xml \
+    --numprocesses=auto \
+    ${paths}
+"""
+executor = {group = "test"}
+help = "Compute how much of the source code is covered by tests"
 
-[tool.poe.tasks.test]
+[[tool.poe.groups.test.tasks.cov.args]]
+default = "src tests"
+multiple = true
+name = "paths"
+positional = true
+
+[tool.poe.groups.test.tasks.docnb-force]
+env = {FORCE_EXECUTE_NB = "yes"}
+help = "Build documentation and API with notebook execution (no cache)"
+sequence = ["doc"]
+
+[tool.poe.groups.test.tasks.test]
 cmd = "pytest -m 'slow or not slow' --numprocesses=auto ${paths}"
 executor = {group = "test"}
 help = "Run all unit tests"
 
-[[tool.poe.tasks.test.args]]
+[[tool.poe.groups.test.tasks.test.args]]
 default = "src tests"
 multiple = true
 name = "paths"
 positional = true
 
-[tool.poe.tasks.test-all]
+[tool.poe.groups.test.tasks.test-all]
 help = "Run all tests on each supported Python version"
 sequence = [
     {ref = "test-py310 ${paths}"},
@@ -315,32 +296,68 @@ sequence = [
     {ref = "test-py314 ${paths}"},
 ]
 
-[[tool.poe.tasks.test-all.args]]
+[[tool.poe.groups.test.tasks.test-all.args]]
 default = ""
 multiple = true
 name = "paths"
 positional = true
 
-[tool.poe.tasks.test-py310]
+[tool.poe.groups.test.tasks.test-py310]
 env = {UV_PYTHON = "3.10"}
 ref = "test"
 
-[tool.poe.tasks.test-py311]
+[tool.poe.groups.test.tasks.test-py311]
 env = {UV_PYTHON = "3.11"}
 ref = "test"
 
-[tool.poe.tasks.test-py312]
+[tool.poe.groups.test.tasks.test-py312]
 env = {UV_PYTHON = "3.12"}
 ref = "test"
 
-[tool.poe.tasks.test-py313]
+[tool.poe.groups.test.tasks.test-py313]
 env = {UV_PYTHON = "3.13"}
 ref = "test"
 
-[tool.poe.tasks.test-py314]
+[tool.poe.groups.test.tasks.test-py314]
 env = {UV_PYTHON = "3.14"}
 ref = "test"
 
+[tool.poe.tasks.all]
+help = "Run all continuous integration (CI) tasks locally"
+ignore_fail = "return_non_zero"
+sequence = [
+    "benchmark",
+    "cov",
+    "docnb-force",
+    "linkcheck",
+    "style",
+    "test-all",
+]
+
+[tool.poe.tasks.benchmark]
+cmd = """
+pytest benchmarks \
+    --benchmark-autosave \
+    --benchmark-json benchmarks/output.json \
+    --durations=0 \
+    -k benchmark
+"""
+executor = {group = "test"}
+help = "Run benchmark tests and visualize performance"
+
+[tool.poe.tasks.style]
+cmd = "pre-commit run --all-files"
+executor = {extra = "scipy", group = "style"}
+help = "Perform all linting, formatting, and spelling checks"
+
+[tool.poe.tasks.upgrade]
+executor = {type = "simple"}
+help = "Upgrade lock files"
+parallel = [
+    {cmd = "pre-commit autoupdate -j8"},
+    {cmd = "uv lock --upgrade"},
+]
+
 [tool.pytest]
 addopts = [
     "--color=yes",
@@ -573,6 +590,10 @@ rules.unresolved-attribute = "ignore"
 include = ["docs/amplitude/interactive.ipynb"]
 rules.too-many-positional-arguments = "ignore"
 
+[[tool.ty.overrides]]
+include = ["docs/dynamics.ipynb"]
+rules.index-out-of-bounds = "ignore"
+
 [[tool.ty.overrides]]
 include = ["tests/external/test_sympy.py"]
 rules.call-non-callable = "ignore"

src/ampform/helicity/__init__.py

@@ -607,7 +607,7 @@ class BuilderConfiguration:
     `~.HelicityModel.kinematic_variables` (which are expressions to compute an event-
     wise array of invariant masses). This is useful if final state particles are stable.
     """
-    use_helicity_couplings: bool = field(validator=instance_of(bool))
+    use_helicity_couplings: bool = field(validator=instance_of(bool))  # ty:ignore[dataclass-field-order]
     """Use helicity couplings instead of amplitude coefficients.
 
     Helicity couplings are a measure for the strength of each partial two-body decay.