From a0402f54808ad8b4cbbae6b21206a63aca07096a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Agust=C3=ADn=20Castro?= Date: Tue, 6 Feb 2024 01:08:40 -0300 Subject: [PATCH] Add Ignore button --- norfair/common_reference_ui.py | 126 +++++++++++++++++++++++++++------ 1 file changed, 106 insertions(+), 20 deletions(-) diff --git a/norfair/common_reference_ui.py b/norfair/common_reference_ui.py index 06c1cb47..c2571494 100644 --- a/norfair/common_reference_ui.py +++ b/norfair/common_reference_ui.py @@ -40,6 +40,9 @@ handling_mark_functions = None handle_mark_annotation = None +global button_says_ignore +global button_ignore + def set_reference( reference: str, @@ -57,6 +60,8 @@ def set_reference( Creates a UI to annotate points that match in reference and footage, and estimate the transformation. To add a point, just click a pair of points (one from the footage window, and another from the reference window) and select "Add" To remove a point, just select the corresponding point at the bottom left corner, and select "Remove". + You can also ignore point, by clicking them and selecting "Ignore". The transformation will not used ingored points. + To 'uningnore' points that have been previously ignored, just click them and select "Unignore". If either footage or reference are videos, you can jump to future frames to pick points that match. For example, to jump 215 frames in the footage, just write an integer number of frames to jump next to 'Frames to skip (footage)', and select "Skip frames". @@ -99,6 +104,8 @@ def set_reference( global transformation global button_finish + global button_says_ignore + global button_ignore global reference_point_canvas global footage_point_canvas @@ -154,26 +161,24 @@ def set_reference( def estimate_transformation(points): global button_finish - if len(points) >= 4: - curr_pts = np.array( - [point["reference"] for point in points.values()] - ) # use current points as reference points - prev_pts = np.array( - [point["footage"] for point in points.values()] - ) # use previous points as footage points (to deduce reference -> footage) - - try: + curr_pts = np.array( + [point["reference"] for point in points.values() if not point["ignore"]] + ) # use current points as reference points + prev_pts = np.array( + [point["footage"] for point in points.values() if not point["ignore"]] + ) # use previous points as footage points (to deduce reference -> footage) + + button_finish.configure(fg="black", highlightbackground="green") + try: + transformation = transformation_getter(curr_pts, prev_pts)[1] + except: + transformation = None - button_finish.configure(fg="black", highlightbackground="green") - return transformation_getter(curr_pts, prev_pts)[1] - except np.linalg.LinAlgError: - button_finish.configure( - fg="grey", highlightbackground="SystemButtonFace" - ) - return None + if transformation is not None: + button_finish.configure(fg="black", highlightbackground="green") else: button_finish.configure(fg="grey", highlightbackground="SystemButtonFace") - return None + return transformation def test_transformation( change_of_coordinates, @@ -263,9 +268,23 @@ def handle_annotation(event): global reference_canvas_size global footage_original_size global footage_canvas_size + global button_says_ignore + global button_ignore + global points points[key]["marked"] = not points[key]["marked"] + marked_points = [ + point["ignore"] for point in points.values() if point["marked"] + ] + + if (len(marked_points) > 0) and (all(marked_points)): + button_says_ignore = False + button_ignore.configure(text="Unignore") + else: + button_says_ignore = True + button_ignore.configure(text="Ignore") + if points[key]["marked"]: points[key]["button"].configure(fg="black", highlightbackground="red") @@ -289,7 +308,7 @@ def handle_annotation(event): try: reference_point_in_rel_coords = skipper["reference"][ "motion_transformation" - ].abs_to_rel(np.array([points[key]["footage"]]))[0] + ].abs_to_rel(np.array([points[key]["reference"]]))[0] reference_point_in_rel_coords = np.multiply( reference_point_in_rel_coords, np.array( @@ -303,13 +322,27 @@ def handle_annotation(event): reference_point_in_rel_coords = points[key]["reference_canvas"] pass + if points[key]["ignore"]: + color = "gray" + else: + color = "red" + draw_point_in_canvas( - canvas_footage, footage_point_in_rel_coords, color="red" + canvas_footage, footage_point_in_rel_coords, color=color ) draw_point_in_canvas( - canvas_reference, reference_point_in_rel_coords, color="red" + canvas_reference, reference_point_in_rel_coords, color=color ) else: + if points[key]["ignore"]: + points[key]["button"].configure( + fg="gray", highlightbackground="gray" + ) + else: + points[key]["button"].configure( + fg="black", highlightbackground="SystemButtonFace" + ) + points[key]["button"].configure( fg="black", highlightbackground="SystemButtonFace" ) @@ -729,6 +762,7 @@ def handle_add_annotation(): "footage_canvas": footage_point_canvas, "button": None, "marked": False, + "ignore": False, } points[points_sampled] = new_point @@ -824,6 +858,58 @@ def handle_test_mode(event): frame_options_annotate_or_test.pack(side=tk.TOP) + ###### MAKE SUBBLOCK TO IGNORE POINTS + + button_says_ignore = True + frame_options_ignore = tk.Frame(master=frame_options) + text_ignore = tk.Label( + master=frame_options_ignore, + text="Ignore points", + foreground="white", + background="#5f9ea0", + width=20, + height=1, + ) + button_ignore = tk.Button( + master=frame_options_ignore, + text="Ignore", + width=16, + height=1, + bg="blue", + fg="black", + command=lambda: handle_ignore_point(), + ) + + def handle_ignore_point(): + global points + global transformation + global button_says_ignore + + if button_says_ignore: + fg = "gray" + highlightbackground = "gray" + else: + fg = "black" + highlightbackground = "SystemButtonFace" + + for key, couple in points.items(): + if couple["marked"]: + points[key]["ignore"] = button_says_ignore + points[key]["button"].configure( + fg=fg, highlightbackground=highlightbackground + ) + points[key]["marked"] = False + remove_drawings_in_canvas(canvas_footage) + remove_drawings_in_canvas(canvas_reference) + button_says_ignore = True + button_ignore.configure(text="Ignore") + transformation = estimate_transformation(points) + + text_ignore.pack(side=tk.LEFT) + button_ignore.pack(side=tk.LEFT) + + frame_options_ignore.pack(side=tk.TOP) + ###### MAKE SUBBLOCK TO REMOVE POINTS frame_options_remove = tk.Frame(master=frame_options)