Refactor artifacts detection + add sturation detection

conftest.py

@@ -8,6 +8,21 @@ def create_cache_folder(tmp_path_factory):
     return cache_folder
 
 
+@pytest.fixture(scope="module")
+def debug_plots(request):
+    """Return True if debug plots should be shown."""
+    return request.config.getoption("--debug-plots")
+
+
+def pytest_addoption(parser):
+    parser.addoption(
+        "--debug-plots",
+        action="store_true",
+        default=False,
+        help="Enable debug plots during tests",
+    )
+
+
 def pytest_collection_modifyitems(config, items):
     """
     This function marks (in the pytest sense) the tests according to their name and file_path location

src/spikeinterface/core/node_pipeline.py

@@ -500,11 +500,13 @@ def check_graph(nodes):
     Check that node list is orderd in a good (parents are before children)
     """
 
-    node0 = nodes[0]
-    if not isinstance(node0, PeakSource):
-        raise ValueError(
-            "Peak pipeline graph must have as first element a PeakSource (PeakDetector or PeakRetriever or SpikeRetriever"
-        )
+    # Do not remove this, this is to remenber that in previous version the first node needed to be
+    # a detectot but not anymore
+    # node0 = nodes[0]
+    # if not isinstance(node0, PeakSource):
+    #     raise ValueError(
+    #         "Peak pipeline graph must have as first element a PeakSource (PeakDetector or PeakRetriever or SpikeRetriever"
+    #     )
 
     for i, node in enumerate(nodes):
         assert isinstance(node, PipelineNode), f"Node {node} is not an instance of PipelineNode"

src/spikeinterface/preprocessing/__init__.py

@@ -20,6 +20,8 @@
     PreprocessingPipeline,
 )
 
+from .detect_artifacts import detect_artifact_periods, detect_artifact_periods_by_envelope, detect_saturation_periods
+
 # for snippets
 from .align_snippets import AlignSnippets
 from warnings import warn