Switch from warnings to logger.warning (#362)

Author: peterhollender

src/openlifu/seg/seg_method.py

@@ -2,7 +2,7 @@
 
 import copy
 import inspect
-import warnings
+import logging
 from abc import ABC, abstractmethod
 from dataclasses import dataclass, field
 from typing import Annotated, Any, Literal
@@ -71,7 +71,7 @@ def from_dict(d: dict, on_keyword_mismatch: Literal['warn', 'raise', 'ignore'] =
             if on_keyword_mismatch == 'raise':
                 raise TypeError(f"Unexpected keyword arguments for {short_classname}: {unexpected_keywords}")
             elif on_keyword_mismatch == 'warn':
-                warnings.warn(f"Ignoring unexpected keyword arguments for {short_classname}: {unexpected_keywords}", stacklevel=2)
+                logging.warning(f"Ignoring unexpected keyword arguments for {short_classname}: {unexpected_keywords}")
             for k in unexpected_keywords:
                 d.pop(k)
 

tests/test_seg_method.py

@@ -87,9 +87,6 @@ def test_from_dict_on_keyword_mismatch():
         "ref_material": "water"
     }
 
-    with pytest.warns(UserWarning):
-        SegmentationMethod.from_dict(d, on_keyword_mismatch='warn')
-
     with pytest.raises(TypeError, match=r"Unexpected keyword arguments for UniformWater: \['ref_material'\]"):
         SegmentationMethod.from_dict(d, on_keyword_mismatch='raise')