Minor fixes: recompute delete (#1413)

* Check matching env based on sorted dict, not fetch

* nwb_dict -> nwb_deps

* Remove spyglass from warning editable install warning

* See details

- `UserEnvironment` inherits mixin.
- `v1.SpikeSortingRecording` passes hashing precision as a dictionary.
- `SpyglassMixin.delete_orphans` convenience method added.
- `NwbfileHasher` now ...
  - accepts an int as a default precision
  - checks that datasets are numeric before rounding
- `RecordingRecompute`s now have `recheck` methods to double-check keys

* Add tests

* Update changelog

* Fix failing tests

* Update changelog

* v0 recompute delete fixes

* revert conflict edit. update changelog

Author: CBroz1

CHANGELOG.md

@@ -30,6 +30,7 @@ import all foreign key references.
 ### Infrastructure
 
 - Auto-load within-Spyglass tables for graph operations #1368
+- Allow rechecking of recomputes #1380, #1413
 
 ### Pipelines
 

src/spyglass/spikesorting/v0/spikesorting_recompute.py

@@ -513,7 +513,7 @@ def delete_files(
     ) -> None:
         """If successfully recomputed, delete files for a given restriction."""
         query = self & "matched=1" & restriction
-        file_names = query.fetch("analysis_file_name")
+        file_names = query.fetch("nwb_file_name")
         prefix = "DRY RUN: " if dry_run else ""
         msg = f"{prefix}Delete {len(file_names)} files?\n\t" + "\n\t".join(
             file_names
@@ -528,8 +528,9 @@ def delete_files(
 
         for key in query.proj():
             old, new = self._get_paths(key)
-            new.unlink(missing_ok=True)
-            old.unlink(missing_ok=True)
+            logger.info(f"Deleting old: {old}, new: {new}")
+            shutil_rmtree(old, ignore_errors=True)
+            shutil_rmtree(new, ignore_errors=True)
 
     def delete(self, *args, **kwargs) -> None:
         """Delete recompute attempts when deleting rows."""