Merge pull request #4196 from chrishalcrow/fix-export-to-phy-unitrefine

Change `export_to_phy` args in auto-curation

Author: alejoe91

src/spikeinterface/curation/model_based_curation.py

@@ -188,14 +188,9 @@ def _check_params_for_classification(self, enforce_metric_params=False, model_in
                     else:
                         warnings.warn(warning_message)
 
-    def _export_to_phy(self, classified_units):
+    def _export_to_phy(self, classified_df):
         """Export the classified units to Phy as cluster_prediction.tsv file"""
 
-        import pandas as pd
-
-        # Create a new DataFrame with unit_id, prediction, and probability columns from dict {unit_id: (prediction, probability)}
-        classified_df = pd.DataFrame.from_dict(classified_units, orient="index", columns=["prediction", "probability"])
-
         # Export to Phy format
         try:
             sorting_path = self.sorting_analyzer.sorting.get_annotation("phy_folder")

src/spikeinterface/curation/tests/test_model_based_curation.py

@@ -99,7 +99,9 @@ def test_model_based_classification_get_metrics_for_classification(
 def test_model_based_classification_export_to_phy(sorting_analyzer_for_curation, model):
     # Test the _export_to_phy() method of ModelBasedClassification
     model_based_classification = ModelBasedClassification(sorting_analyzer_for_curation, model[0])
-    classified_units = {0: (1, 0.5), 1: (0, 0.5), 2: (1, 0.5), 3: (0, 0.5), 4: (1, 0.5)}
+    import pandas as pd
+
+    classified_units = pd.DataFrame.from_dict({0: (1, 0.5), 1: (0, 0.5), 2: (1, 0.5), 3: (0, 0.5), 4: (1, 0.5)})
     # Function should fail here
     with pytest.raises(ValueError):
         model_based_classification._export_to_phy(classified_units)