Remove pre-existing units from analysis files (#1453)

* pop off existing units from analysis files

* update changelog

Author: samuelbray32

CHANGELOG.md

@@ -57,6 +57,7 @@ DecodingParameters().alter()
     - Add custom/dynamic `AnalysisNwbfile` creation #1435
     - Allow nullable `DataAcquisitionDevice` foreign keys #1455
     - Improve error transparency on duplicate `Electrode` ids #1454
+    - Remove pre-existing `Units` from created analysis nwb files #1453
     - Allow multiple VideoFile entries during ingestion #1462
 - Decoding
     - Ensure results directory is created if it doesn't exist #1362

src/spyglass/utils/mixins/analysis.py

@@ -209,6 +209,10 @@ def create(
                     if isinstance(nwb_object, pynwb.core.LabelledDict):
                         for module in list(nwb_object.keys()):
                             nwb_object.pop(module)
+                    # Remove units to avoid conflict with spyglass-managed spikesorting
+                    if field == "units":
+                        nwbf._remove_child(nwb_object)
+                        nwbf.fields[field] = None
 
             # pop off optogenetic_epochs if it exists
             if (