Set stacklevel

setup.cfg

@@ -1,4 +1,4 @@
 [flake8]
 # See https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#line-length
 max-line-length = 88
-extend-ignore = E203, B028
+extend-ignore = E203

src/scippnexus/_hdf5_nexus.py

@@ -39,7 +39,8 @@ def _warn_latin1_decode(obj, decoded, error):
         f"extended ASCII (latin-1), but this behavior is not "
         f"specified by the HDF5 or nexus standards and may therefore "
         f"be incorrect. Decoded string using latin-1 is '{decoded}'. "
-        f"Error was '{error}'."
+        f"Error was '{error}'.",
+        stacklevel=2,
     )
 
 

src/scippnexus/base.py

@@ -378,7 +378,7 @@ def _warn_fallback(self, e: Exception) -> None:
             f"Failed to load {self.name} as {type(self._nexus).__name__}: {e} "
             "Falling back to loading HDF5 group children as scipp.DataGroup."
         )
-        warnings.warn(msg)
+        warnings.warn(msg, stacklevel=2)
 
     def __setitem__(self, key, value):
         """Set a child group or child dataset.

src/scippnexus/field.py

@@ -239,7 +239,8 @@ def unit(self) -> Union[sc.Unit, None]:
             except sc.UnitError:
                 warnings.warn(
                     f"Unrecognized unit '{unit}' for value dataset "
-                    f"in '{self.name}'; setting unit as 'dimensionless'"
+                    f"in '{self.name}'; setting unit as 'dimensionless'",
+                    stacklevel=2,
                 )
                 return sc.units.one
         return None

src/scippnexus/nxtransformations.py

@@ -403,7 +403,8 @@ def _with_positions(
                 out[store_transform] = transform
         except TransformationChainResolver.ChainError as e:
             warnings.warn(
-                UserWarning(f'depends_on chain references missing node:\n{e}')
+                UserWarning(f'depends_on chain references missing node:\n{e}'),
+                stacklevel=2,
             )
     for name, value in dg.items():
         if isinstance(value, sc.DataGroup):

src/scippnexus/v1/file.py

@@ -16,6 +16,7 @@ def __init__(self, *args, definition=None, **kwargs):
             "The scippnexus.v1 API is deprecated and will be removed in release 24.01. "
             "Switch to 'import scippnexus as snx' to use API version 2.",
             VisibleDeprecationWarning,
+            stacklevel=2,
         )
         self._file = h5py.File(*args, **kwargs)
         NXroot.__init__(self, self._file, definition=definition)

src/scippnexus/v1/nxobject.py

@@ -317,7 +317,8 @@ def unit(self) -> Union[sc.Unit, None]:
             except sc.UnitError:
                 warnings.warn(
                     f"Unrecognized unit '{unit}' for value dataset "
-                    f"in '{self.name}'; setting unit as 'dimensionless'"
+                    f"in '{self.name}'; setting unit as 'dimensionless'",
+                    stacklevel=2,
                 )
                 return sc.units.one
         return None
@@ -389,7 +390,7 @@ def _get_child(
                         f"Failed to determine axis names of {item.name}: {e}. "
                         "Falling back to default dimension labels."
                     )
-                    warnings.warn(msg)
+                    warnings.warn(msg, stacklevel=2)
                     dims = None
                 dtype = self._get_field_dtype(name)
                 return Field(
@@ -415,7 +416,7 @@ def _get_child(
                     f"Failed to load {self.name} as {type(self).__name__}: {e} "
                     "Falling back to loading HDF5 group children as scipp.DataGroup."
                 )
-                warnings.warn(msg)
+                warnings.warn(msg, stacklevel=2)
             da = NXobject._getitem(self, name)
         return da
 
@@ -582,7 +583,8 @@ def depends_on(self) -> Union[sc.Variable, sc.DataArray, None]:
             except (NexusStructureError, TransformationError) as e:
                 warnings.warn(
                     f"Failed to load transformation {self.name}/{depends_on}:\n{e}\n"
-                    "Falling back to returning the path to the transformation."
+                    "Falling back to returning the path to the transformation.",
+                    stacklevel=2,
                 )
                 return depends_on[()]
         return None