greptile round 1

Author: yaugenst-flex

docs/index.rst

@@ -75,12 +75,12 @@ Get Started
             pip install pipx
             pipx run tidy3d configure --apikey=XXX
 
-        If you're running into trouble, you may need to manually set the API key directly in the configuration file where Tidy3D looks for it.
-        You need to place the ``$HOME/.tidy3d/config`` file in your home directory such as ``C:\Users\username\`` (where ``username`` is your username).
-
-        The ``tidy3d configure`` command stores the API key for you. If you prefer
-        to manage the file yourself, see :doc:`configuration/index` for the current
-        location and format on each platform.
+        If you're running into trouble, you may need to manually set the API key
+        directly in the configuration file where Tidy3D looks for it. The ``tidy3d
+        configure`` command stores the API key for you. If you prefer to manage the
+        file yourself, place ``config.toml`` in the directory described in
+        :doc:`configuration/index` (for example ``~/.config/tidy3d`` on macOS/Linux
+        or ``%APPDATA%\\tidy3d`` on Windows).
 
     .. group-tab:: In the Cloud |:cloud:|
 
@@ -251,4 +251,3 @@ Contents
   changelog
   About our Solver <https://www.flexcompute.com/tidy3d/solver/>
 
-

tests/config/test_plugins.py

@@ -1,6 +1,5 @@
 from __future__ import annotations
 
-import toml
 from pydantic import Field
 
 from tidy3d.config.__init__ import get_manager, reload_config
@@ -37,7 +36,6 @@ def test_plugin_updates_persist(mock_config_dir):
     manager.save()
     config_path = manager.config_dir / "config.toml"
     assert config_path.exists()
-    contents = toml.load(config_path)
 
     reload_config(profile=manager.profile)
     new_manager = get_manager()

tidy3d/components/geometry/base.py

@@ -2549,7 +2549,7 @@ def _derivative_face(
     @staticmethod
     def _arr_at_face(arr: xr.DataArray, interp_point: float, dim_normal: str) -> xr.DataArray:
         """Interpolate the array at the given coordinate unless it only has a single value in the normal
-        dimnsion already in which case just return the array itself."""
+        dimension already in which case just return the array itself."""
         arr_at_face = (
             arr
             if (len(arr.coords[dim_normal]) == 1)

tidy3d/packaging.py

@@ -252,8 +252,9 @@ def _fn(*args, **kwargs):
         global _CONFIG_LOCAL_SUBPIXEL
         use_local_subpixel = _CONFIG_LOCAL_SUBPIXEL
         _CONFIG_LOCAL_SUBPIXEL = False
-        result = fn(*args, **kwargs)
-        _CONFIG_LOCAL_SUBPIXEL = use_local_subpixel
-        return result
+        try:
+            return fn(*args, **kwargs)
+        finally:
+            _CONFIG_LOCAL_SUBPIXEL = use_local_subpixel
 
     return _fn