Warn on duplicate processor registration

Change `register_processor` to log a warning instead of raising on duplicate `PROCESSOR_ID` keys, allowing later registrations to override earlier ones without import-time failures. Update subclass save tests to load processor classes from `dlclivegui.processors.examples` via a dedicated fixture, so the parametrized tests validate the concrete example processors against the correct module data path.

Author: C-Achard

dlclivegui/processors/dlc_processor_socket.py

@@ -30,10 +30,11 @@
 def register_processor(cls):
     registry_key = getattr(cls, "PROCESSOR_ID", cls.__name__)
     if registry_key in PROCESSOR_REGISTRY:
-        raise ValueError(
+        msg = (
             f"Duplicate processor registration key '{registry_key}': "
             f"{PROCESSOR_REGISTRY[registry_key].__name__} vs {cls.__name__}"
         )
+        logger.warning(msg)
     PROCESSOR_REGISTRY[registry_key] = cls
     return cls
 

tests/custom_processors/test_base_processor.py

@@ -37,6 +37,19 @@ def socket_mod(monkeypatch):
     return importlib.import_module(mod_name)
 
 
+@pytest.fixture
+def example_processor_mod(monkeypatch):
+    """
+    Import the example processor module with dlclive mocked.
+    Adjust module name if your file lives elsewhere.
+    """
+    _mock_dlclive(monkeypatch)
+    mod_name = "dlclivegui.processors.examples"
+    if mod_name in sys.modules:
+        del sys.modules[mod_name]
+    return importlib.import_module(mod_name)
+
+
 def _module_data_dir(socket_mod) -> Path:
     """Compute the data/ directory where save() writes artifacts."""
     return Path(socket_mod.__file__).parent.parent.parent / "data"
@@ -233,12 +246,14 @@ def test_save_ignores_pre_recording_original_pose_frames(socket_mod):
         ("ExampleProcessorSocketFilterKeypoints", 10),
     ],
 )
-def test_subclass_save_ignores_pre_recording_original_pose_frames(socket_mod, class_name, n_keypoints):
+def test_subclass_save_ignores_pre_recording_original_pose_frames(
+    socket_mod, example_processor_mod, class_name, n_keypoints
+):
     """
     Concrete processors must keep original_pose aligned with recorded metadata
     even when process() is called before recording starts.
     """
-    processor_class = getattr(socket_mod, class_name)
+    processor_class = getattr(example_processor_mod, class_name)
     proc = processor_class(bind=("127.0.0.1", 0), save_original=True)
 
     try: