Merge pull request #348 from funkelab/bulk-delete

Use UserDeleteNodes for efficiency

Author: cmalinmayor

pyproject.toml

@@ -31,7 +31,7 @@ classifiers = [
 
 dependencies =[
     "napari>=0.6.2,<1",
-    "funtracks==1.8.0-a3",
+    "funtracks>=1.8.0,<3",
     "appdirs>=1,<2",
     "numpy>=2,<3",
     "magicgui>=0.10.1",

src/motile_tracker/data_views/views/layers/track_points.py

@@ -8,7 +8,7 @@
 import numpy as np
 from funtracks.data_model import Tracks
 from funtracks.exceptions import InvalidActionError
-from funtracks.user_actions import UserAddNode, UserDeleteNode, UserUpdateNodeAttrs
+from funtracks.user_actions import UserAddNode, UserDeleteNodes, UserUpdateNodeAttrs
 from napari.layers.points._points_mouse_bindings import select
 from napari.utils.notifications import show_info
 from psygnal import Signal
@@ -259,8 +259,10 @@ def _update_data(self, event: Event):
                 self._refresh()
 
         elif event.action == "removed":
-            for node in self.tracks_viewer.selected_nodes.as_list:
-                UserDeleteNode(self.tracks_viewer.tracks, node=node)
+            UserDeleteNodes(
+                self.tracks_viewer.tracks,
+                nodes=self.tracks_viewer.selected_nodes.as_list,
+            )
 
         elif event.action == "changed":
             # we only want to allow this update if there is no seg layer

src/motile_tracker/data_views/views_coordinator/tracks_viewer.py

@@ -8,7 +8,7 @@
 from funtracks.user_actions import (
     UserAddEdge,
     UserDeleteEdge,
-    UserDeleteNode,
+    UserDeleteNodes,
     UserSwapPredecessors,
 )
 from psygnal import Signal
@@ -289,8 +289,7 @@ def delete_node(self, event=None):
 
         if self.tracks is None:
             return
-        for node in self.selected_nodes.as_list:
-            UserDeleteNode(self.tracks, node=node)
+        UserDeleteNodes(self.tracks, nodes=self.selected_nodes.as_list)
 
     def delete_edge(self, event=None):
         """Calls the UserAction to delete an edge between the two currently