deleted specified time printing and made other minor changes

	- deleted uncessary comments
	- slight annotation refactor
	- set integration test back to not saving image

Author: vichua2006

modules/detect_target/detect_target_brightspot.py

@@ -65,6 +65,7 @@ def __init__(
         min_area_pixels: Minimum area in pixels.
         max_area_pixels: Maximum area in pixels.
         min_brightness_threshold: Minimum brightness threshold for bright spots.
+        min_average_brightness_threshold: Minimum absolute average brightness of detected blobs.
         """
         self.brightspot_percentile_threshold = brightspot_percentile_threshold
         self.filter_by_color = filter_by_color
@@ -150,12 +151,9 @@ def run(
             grey_image, combined_threshold, 255, cv2.THRESH_BINARY
         )
         if threshold_used == 0:
-            self.__local_logger.error(f"{time.time()}: Failed to percentile threshold image.")
+            self.__local_logger.error("Failed to percentile threshold image.")
             return False, None
 
-        # cv2.imshow("Thresholded", bw_image)  # type: ignore
-        # cv2.waitKey(0)  # type: ignore
-
         # Set up SimpleBlobDetector
         params = cv2.SimpleBlobDetector_Params()
         params.filterByColor = self.__config.filter_by_color
@@ -178,9 +176,7 @@ def run(
 
         # A lack of detections is not an error, but should still not be forwarded
         if len(keypoints) == 0:
-            self.__local_logger.info(
-                f"{time.time()}: No brightspots detected (before blob average filter)."
-            )
+            self.__local_logger.info("No brightspots detected (before blob average filter).")
             return False, None
 
         # Compute the average brightness of each blob
@@ -221,16 +217,9 @@ def run(
 
         # A lack of detections is not an error, but should still not be forwarded
         if len(filtered_keypoints) == 0:
-            self.__local_logger.info(
-                f"{time.time()}: No brightspots detected (after blob average filter)."
-            )
+            self.__local_logger.info("No brightspots detected (after blob average filter).")
             return False, None
 
-        # Annotate the image (green circle) with detected keypoints
-        image_annotated = cv2.drawKeypoints(
-            image, filtered_keypoints, None, (0, 255, 0), cv2.DRAW_MATCHES_FLAGS_DRAW_RICH_KEYPOINTS
-        )
-
         # Process bright spot detection
         result, detections = detections_and_time.DetectionsAndTime.create(data.timestamp)
         if not result:
@@ -269,6 +258,15 @@ def run(
         if self.__filename_prefix != "":
             filename = self.__filename_prefix + str(self.__counter)
 
+            # Annotate the image (green circle) with detected keypoints
+            image_annotated = cv2.drawKeypoints(
+                image,
+                filtered_keypoints,
+                None,
+                (0, 255, 0),
+                cv2.DRAW_MATCHES_FLAGS_DRAW_RICH_KEYPOINTS,
+            )
+
             # Annotated image
             cv2.imwrite(filename + ".png", image_annotated)  # type: ignore
 

tests/integration/test_detect_target_worker.py

@@ -59,7 +59,7 @@
 )
 
 SHOW_ANNOTATIONS = False
-SAVE_NAME = "test"  # No need to save images
+SAVE_NAME = ""  # No need to save images
 
 WORK_COUNT = 3
 DELAY_FOR_SIMULATED_ULTRALYTICS_WORKER = 1  # seconds