Revert "Import pose (#1225)" (#1245)

This reverts commit 783e7a3.

Author: CBroz1

CHANGELOG.md

@@ -17,17 +17,16 @@
     - Set `probe_id` as `probe_description` when inserting from nwb file #1220
     - Default `AnalysisNwbfile.create` permissions are now 777 #1226
 - Position
-    - Allow population of missing `PositionIntervalMap` entries during population of `DLCPoseEstimation` #1208
     - Allow population of missing `PositionIntervalMap` entries during population
         of `DLCPoseEstimation` #1208
-    - Enable import of existing pose data to `ImportedPose` in position pipeline #1225
 - Spikesorting
     - Fix compatibility bug between v1 pipeline and `SortedSpikesGroup` unit
         filtering #1238
 
 - Behavior
     - Implement pipeline for keypoint-moseq extraction of behavior syllables #1056
 
+
 ## [0.5.4] (December 20, 2024)
 
 ### Infrastructure

docs/src/ForDevelopers/UsingNWB.md

@@ -256,14 +256,6 @@ hdmf.common.table.DynamicTable </b>
 | PositionSource.SpatialSeries |           id           |   int(nwbf.processing.behavior.position.\[index\]) (the enumerated index number) |               |                       |
 | RawPosition.PosObject        | raw_position_object_id |                            nwbf.processing.behavior.position.\[index\].object_id |               |                       |
 
-<b> NWBfile Location: nwbf.processing.behavior.PoseEstimation </br> Object type:
-(ndx_pose.PoseEstimation) </b>
-
-| Spyglass Table               |          Key           |                                                                 NWBfile Location | Config option |                 Notes |
-| :--------------------------- | :--------------------: | -------------------------------------------------------------------------------: | ------------: | --------------------: |
-| ImportedPose | interval_list_name | pose_{PoseEstimation.name}_valid_times |
-| ImportedPose.BodyPart | pose | nwbf.processing.behavior.PoseEstimation.pose_estimation_series.name |
-
 <b> NWBfile Location: nwbf.processing.video_files.video </br> Object type:
 pynwb.image.ImageSeries </b>
 

pyproject.toml

@@ -15,7 +15,6 @@ authors = [
     { name = "Ryan Ly", email = "[email protected]" },
     { name = "Daniel Gramling", email = "[email protected]" },
     { name = "Chris Brozdowski", email = "[email protected]" },
-    { name = "Samuel Bray", email = "[email protected]" },
 ]
 classifiers = [
     "Programming Language :: Python :: 3",
@@ -46,7 +45,6 @@ dependencies = [
     "ipympl",
     "matplotlib",
     "ndx_franklab_novela>=0.1.0",
-    "ndx-pose",
     "non_local_detector",
     "numpy",
     "opencv-python",

src/spyglass/common/populate_all_common.py

@@ -83,8 +83,6 @@ def single_transaction_make(
             if table.__name__ == "PositionSource":
                 # PositionSource only uses nwb_file_name - full calls redundant
                 key_source = dj.U("nwb_file_name") & key_source
-            if table.__name__ == "ImportedPose":
-                key_source = Nwbfile()
 
             for pop_key in (key_source & file_restr).fetch("KEY"):
                 try:
@@ -118,7 +116,6 @@ def populate_all_common(
     List
         A list of keys for InsertError entries if any errors occurred.
     """
-    from spyglass.position.v1.imported_pose import ImportedPose
     from spyglass.spikesorting.imported import ImportedSpikeSorting
 
     declare_all_merge_tables()
@@ -146,7 +143,6 @@ def populate_all_common(
             PositionSource,  # Depends on Session
             VideoFile,  # Depends on TaskEpoch
             StateScriptFile,  # Depends on TaskEpoch
-            ImportedPose,  # Depends on Session
         ],
         [
             RawPosition,  # Depends on PositionSource

src/spyglass/position/position_merge.py

@@ -3,7 +3,6 @@
 from pandas import DataFrame
 
 from spyglass.common.common_position import IntervalPositionInfo as CommonPos
-from spyglass.position.v1.imported_pose import ImportedPose
 from spyglass.position.v1.position_dlc_pose_estimation import DLCPoseEstimation
 from spyglass.position.v1.position_dlc_selection import DLCPosV1
 from spyglass.position.v1.position_trodes_position import TrodesPosV1
@@ -15,7 +14,6 @@
     "IntervalPositionInfo": CommonPos,
     "DLCPosV1": DLCPosV1,
     "TrodesPosV1": TrodesPosV1,
-    "ImportedPose": ImportedPose,
     "DLCPoseEstimation": DLCPoseEstimation,
 }
 
@@ -67,17 +65,6 @@ class CommonPos(SpyglassMixin, dj.Part):
         -> CommonPos
         """
 
-    class ImportedPose(SpyglassMixin, dj.Part):
-        """
-        Table to pass-through upstream Pose information from NWB file
-        """
-
-        definition = """
-        -> PositionOutput
-        ---
-        -> ImportedPose
-        """
-
     def fetch1_dataframe(self) -> DataFrame:
         """Fetch a single dataframe from the merged table."""
         # proj replaces operator restriction to enable