Merge pull request #5122 from voxel51/bug-clear-fields

Fix target view for clear sample/frame field operators
voxel51 · Nov 15, 2024 · f9770af · f9770af
2 parents 02ae185 + 6ff3d05
commit f9770af
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/fiftyone/operators/builtin.py b/fiftyone/operators/builtin.py
@@ -622,8 +622,11 @@ def resolve_input(self, ctx):
 
     def execute(self, ctx):
         field_name = ctx.params["field_name"]
+        target = ctx.params.get("target", None)
+
+        target_view = _get_target_view(ctx, target)
 
-        ctx.dataset.clear_sample_field(field_name)
+        target_view.clear_sample_field(field_name)
         ctx.trigger("reload_dataset")
 
 
@@ -713,8 +716,11 @@ def resolve_input(self, ctx):
 
     def execute(self, ctx):
         field_name = ctx.params["field_name"]
+        target = ctx.params.get("target", None)
+
+        target_view = _get_target_view(ctx, target)
 
-        ctx.dataset.clear_frame_field(field_name)
+        target_view.clear_frame_field(field_name)
         ctx.trigger("reload_dataset")