chore: update ruff -> 1.14.1 and update rules for python 3.10

Author: yaugenst-flex

.github/workflows/tidy3d-python-client-tests.yml

@@ -166,7 +166,7 @@ jobs:
           persist-credentials: false
       - uses: astral-sh/ruff-action@57714a7c8a2e59f32539362ba31877a1957dded1 # v3.5.1
         with:
-          version: 0.11.11
+          version: 0.14.1
       - name: Run ruff format
         run: ruff format --check --diff
       - name: Run ruff check

.pre-commit-config.yaml

@@ -4,7 +4,7 @@ default_install_hook_types:
   - commit-msg
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.11.11"
+    rev: "v0.14.1"
     hooks:
      - id: ruff-check
        args: [ --fix ]

pyproject.toml

@@ -52,7 +52,7 @@ typing-extensions = "*"
 
 ### Optional dependencies ###
 # development core
-ruff = { version = "0.11.11", optional = true }
+ruff = { version = "0.14.1", optional = true }
 coverage = { version = "*", optional = true }
 dill = { version = "*", optional = true }
 ipython = { version = "*", optional = true }
@@ -236,38 +236,38 @@ requires = ["poetry-core>=1.0.0"]
 build-backend = "poetry.core.masonry.api"
 
 [tool.ruff]
-target-version = "py39"
+target-version = "py310"
 line-length = 100
 extend-exclude = ["docs/faq/", "docs/notebooks/"]
 
 [tool.ruff.lint]
 extend-select = [
-  "E",      # pycodestyle errors
-  "F",      # pyflakes
-  "B",      # bugbear
-  "I",      # isort
-  "UP",     # pyupgrade
-  "W",      # pycodestyle
-  "C4",     # flake8-comprehensions
-  "NPY",    # numpy-specific rules
-  "RUF",    # ruff builtins
-  "ISC",    # implicit string concatenation
-  "PIE",    # flake8-pie
-  "RSE",    # unnecessary parantheses on raised exceptions
-  "TID",    # no relative imports from parent modules
-  "PLE",    # pylint errors
-  "PLC",    # pylint conventions
+  "E",        # pycodestyle errors
+  "F",        # pyflakes
+  "B",        # bugbear
+  "I",        # isort
+  "UP",       # pyupgrade
+  "W",        # pycodestyle
+  "C4",       # flake8-comprehensions
+  "NPY",      # numpy-specific rules
+  "RUF",      # ruff builtins
+  "ISC",      # implicit string concatenation
+  "PIE",      # flake8-pie
+  "RSE",      # unnecessary parantheses on raised exceptions
+  "TID",      # no relative imports from parent modules
+  "PLE",      # pylint errors
+  "PLC",      # pylint conventions
 ]
 extend-ignore = [
-  "RUF001", # ambiguous unicode characters
-  "RUF002", # ambiguous unicode characters
-  "RUF003", # ambiguous unicode characters
-  "RUF012", # type hints for mutable defaults
-  "RUF015", # next(iter(...)) instead of list(...)[0]
-  "E501",   # line too long
-  "B905",   # `zip()` without an explicit `strict=` parameter
-  "UP007",  # TODO: Remove once Python >= 3.10
-  "NPY002", # TODO: Revisit RNG handling
+  "RUF001",   # ambiguous unicode characters
+  "RUF002",   # ambiguous unicode characters
+  "RUF003",   # ambiguous unicode characters
+  "RUF012",   # type hints for mutable defaults
+  "RUF015",   # next(iter(...)) instead of list(...)[0]
+  "E501",     # line too long
+  "B905",     # `zip()` without an explicit `strict=` parameter
+  "NPY002",   # TODO: Revisit RNG handling
+  "PLC0415",  # allow imports not at top level
 ]
 
 [tool.ruff.lint.isort]

tests/_test_local/_test_fit_web.py

@@ -3,7 +3,6 @@
 from math import isclose
 
 import numpy as np
-
 from tidy3d.plugins.fitter import AdvancedFitterParam, StableDispersionFitter
 
 np.random.seed(4)
@@ -37,7 +36,7 @@ def test_dispersion_lossless():
     num_poles = 3
     num_tries = 10
     tolerance_rms = 1e-3
-    best_medium, best_rms = fitter.fit(
+    best_medium, _best_rms = fitter.fit(
         num_tries=num_tries, num_poles=num_poles, tolerance_rms=tolerance_rms
     )
 

tests/_test_local/_test_plugins_web.py

@@ -1,7 +1,6 @@
 from __future__ import annotations
 
 import numpy as np
-
 from tidy3d.plugins.fitter import DispersionFitter, StableDispersionFitter
 
 

tests/_test_local/_test_web.py

@@ -5,9 +5,10 @@
 import os
 from unittest import TestCase, mock
 
-import tidy3d.web as web
 from tidy3d.web.auth import encode_password, get_credentials
 
+import tidy3d.web as web
+
 from ..utils import SIM_FULL as sim_original
 
 CALLBACK_URL = "https://callbackurl"

tests/test_components/autograd/numerical/test_autograd_box_polyslab_numerical.py

@@ -368,8 +368,8 @@ def test_box_and_polyslab_gradients_match(is_3d, infinite_dim_2d, shift_box_cent
         local_gradient=False,
     )
 
-    box_value, box_grad = value_and_grad(box_objective)([initial_params])
-    polyslab_value, polyslab_grad = value_and_grad(polyslab_objective)([initial_params])
+    _box_value, box_grad = value_and_grad(box_objective)([initial_params])
+    _polyslab_value, polyslab_grad = value_and_grad(polyslab_objective)([initial_params])
 
     box_grad_np = box_grad
     polyslab_grad_np = polyslab_grad

tests/test_components/autograd/numerical/test_autograd_conductivity_numerical.py

@@ -114,7 +114,7 @@ def make_base_sim(
         ]
     )
 
-    src_size = sim_size_um[0:2] + (0,)
+    src_size = (*sim_size_um[0:2], 0)
 
     wl_min_src_um = 0.9 * adj_wvl_um
     wl_max_src_um = 1.1 * adj_wvl_um
@@ -409,10 +409,10 @@ def test_finite_difference_conductivity_data(
     sim_geometry = get_sim_geometry(mesh_wvl_um)
 
     box_for_override = td.Box(
-        center=(0, 0, 0), size=sim_geometry.size[0:2] + (thickness_um + mesh_wvl_um,)
+        center=(0, 0, 0), size=(*sim_geometry.size[0:2], thickness_um + mesh_wvl_um)
     )
 
-    eval_fns, eval_fn_names = make_eval_fns(monitor_size_wvl)
+    eval_fns, _eval_fn_names = make_eval_fns(monitor_size_wvl)
 
     sim_path_dir = tmp_path / f"test{test_number}"
     sim_path_dir.mkdir()
@@ -443,7 +443,7 @@ def test_finite_difference_conductivity_data(
     )
     conductivity_init = overall_conductivity_scaling * np.ones((dim, dim, Nz))
 
-    obj, adj_grad = obj_val_and_grad([conductivity_init])
+    _obj, adj_grad = obj_val_and_grad([conductivity_init])
 
     # empirical step size for finite difference set as 1/10 the size of the conductivity seed value
     fd_step = 0.1 * overall_conductivity_scaling

tests/test_components/autograd/numerical/test_autograd_mode_polyslab_numerical.py

@@ -72,7 +72,7 @@ def make_base_sim(
         ]
     )
 
-    src_size = sim_size_um[0:2] + (0,)
+    src_size = (*sim_size_um[0:2], 0)
 
     wl_min_src_um = 0.9 * adj_wvl_um
     wl_max_src_um = 1.1 * adj_wvl_um
@@ -425,7 +425,7 @@ def eval_fn(sim_data):
     vertex_centers_x = 1.1 * mesh_wvl_um * np.cos(angles)
     vertex_centers_y = 0.8 * mesh_wvl_um * np.sin(angles)
 
-    obj, adj_grad = obj_val_and_grad([list(vertex_centers_x) + list(vertex_centers_y)])
+    _obj, adj_grad = obj_val_and_grad([list(vertex_centers_x) + list(vertex_centers_y)])
 
     for fd_idx in range(NUM_FINITE_DIFFERENCE):
         # Create random perturbation of vertices to check against the computed adjoint gradient.

tests/test_components/autograd/numerical/test_autograd_numerical.py

@@ -72,7 +72,7 @@ def make_base_sim(
         ]
     )
 
-    src_size = sim_size_um[0:2] + (0,)
+    src_size = (*sim_size_um[0:2], 0)
 
     wl_min_src_um = 0.9 * adj_wvl_um
     wl_max_src_um = 1.1 * adj_wvl_um
@@ -269,10 +269,10 @@ def test_finite_difference_field_data(field_data_test_parameters, rng, tmp_path,
     sim_geometry = get_sim_geometry(mesh_wvl_um)
 
     box_for_override = td.Box(
-        center=(0, 0, 0), size=sim_geometry.size[0:2] + (thickness_um + mesh_wvl_um,)
+        center=(0, 0, 0), size=(*sim_geometry.size[0:2], thickness_um + mesh_wvl_um)
     )
 
-    eval_fns, eval_fn_names = make_eval_fns(monitor_size_wvl)
+    eval_fns, _eval_fn_names = make_eval_fns(monitor_size_wvl)
 
     sim_path_dir = tmp_path / f"test{test_number}"
     sim_path_dir.mkdir()
@@ -298,7 +298,7 @@ def test_finite_difference_field_data(field_data_test_parameters, rng, tmp_path,
 
     perm_init = FINITE_DIFF_PERM_SEED * np.ones((dim, dim, Nz))
 
-    obj, adj_grad = obj_val_and_grad([perm_init])
+    _obj, adj_grad = obj_val_and_grad([perm_init])
 
     # empirical step size from running other finite difference tests for field
     # cases with permittivity