[refactor] improve exception log handling

events/colico_prompt1.json

@@ -0,0 +1,5 @@
+[
+    {"type":  "point", "data": [870, 500], "label": 1},
+    {"type":  "point", "data": [820, 600], "label": 0},
+    {"type": "rectangle", "data": [757, 470, 876, 616]}
+]

src/app.py

@@ -43,13 +43,13 @@ def lambda_handler(event: Dict, context: LambdaContext) -> str:
             app_logger.debug(f"output body_response:{body_response}.")
             response = get_response(HTTPStatus.OK.value, start_time, context.aws_request_id, body_response)
         except Exception as ex2:
-            app_logger.error(f"exception2:{ex2}.")
+            app_logger.exception(f"exception2:{ex2}.", exc_info=True)
             response = get_response(HTTPStatus.INTERNAL_SERVER_ERROR.value, start_time, context.aws_request_id, {})
     except ValidationError as va1:
-        app_logger.error(f"ValidationError:{va1}.")
+        app_logger.exception(f"ValidationError:{va1}.", exc_info=True)
         response = get_response(HTTPStatus.UNPROCESSABLE_ENTITY.value, start_time, context.aws_request_id, {})
     except Exception as ex1:
-        app_logger.error(f"exception1:{ex1}.")
+        app_logger.exception(f"exception1:{ex1}.", exc_info=True)
         response = get_response(HTTPStatus.BAD_REQUEST.value, start_time, context.aws_request_id, {})
 
     app_logger.debug(f"response_dumped:{response}...")

src/io/coordinates_pixel_conversion.py

@@ -23,7 +23,8 @@ def _get_latlng2pixel_projection(latlng: LatLngDict) -> ImagePixelCoordinates:
 
         return {"x": x, "y": y}
     except Exception as e_get_latlng2pixel_projection:
-        app_logger.error(f'e_get_latlng2pixel_projection:{e_get_latlng2pixel_projection}.')
+        app_logger.error(f'args type:{type(latlng)}, {latlng}.')
+        app_logger.exception(f'e_get_latlng2pixel_projection:{e_get_latlng2pixel_projection}.', exc_info=True)
         raise e_get_latlng2pixel_projection
 
 
@@ -40,7 +41,10 @@ def _get_point_latlng_to_pixel_coordinates(latlng: LatLngDict, zoom: int | float
             y=floor(world_coordinate["y"] * scale)
         )
     except Exception as e_format_latlng_to_pixel_coordinates:
-        app_logger.error(f'format_latlng_to_pixel_coordinates:{e_format_latlng_to_pixel_coordinates}.')
+        app_logger.error(f'latlng type:{type(latlng)}, {latlng}.')
+        app_logger.error(f'zoom type:{type(zoom)}, {zoom}.')
+        app_logger.exception(f'e_format_latlng_to_pixel_coordinates:{e_format_latlng_to_pixel_coordinates}.',
+                             exc_info=True)
         raise e_format_latlng_to_pixel_coordinates
 
 

src/io/geo_helpers.py

@@ -26,7 +26,9 @@ def load_affine_transformation_from_matrix(matrix_source_coefficients: list_floa
         center = tuple.__new__(Affine, [a, b, c, d, e, f, 0.0, 0.0, 1.0])
         return center * Affine.translation(-0.5, -0.5)
     except Exception as e:
-        app_logger.error(f"exception:{e}, check https://github.com/rasterio/affine project for updates")
+        app_logger.exception(f"exception:{e}, check updates on https://github.com/rasterio/affine",
+                             extra=e,
+                             stack_info=True, exc_info=True)
         raise e
 
 
@@ -79,5 +81,11 @@ def get_vectorized_raster_as_geojson(mask: np_ndarray, transform: tuple_float) -
             "n_shapes_geojson": len(shapes_list)
         }
     except Exception as e_shape_band:
-        app_logger.error(f"e_shape_band:{e_shape_band}.")
+        try:
+            app_logger.error(f"mask type:{type(mask)}.")
+            app_logger.error(f"transform type:{type(transform)}, {transform}.")
+            app_logger.error(f"mask shape:{mask.shape}, dtype:{mask.dtype}.")
+        except Exception as e_shape_dtype:
+            app_logger.exception(f"mask shape or dtype not found:{e_shape_dtype}.", exc_info=True)
+        app_logger.exception(f"e_shape_band:{e_shape_band}.", exc_info=True)
         raise e_shape_band

src/io/tms2geotiff.py

@@ -125,7 +125,11 @@ def crop_raster(w: float, s: float, e: float, n: float, raster: ndarray, raster_
         app_logger.info(f"cropped image::{cropped_image_ndarray.shape}.")
         return cropped_image_ndarray, cropped_transform
     except Exception as e_crop_raster:
-        app_logger.exception(f"arguments raster: {type(raster)}, {raster}.")
+        try:
+            app_logger.error(f"raster type:{type(raster)}.")
+            app_logger.error(f"raster shape:{raster.shape}, dtype:{raster.dtype}.")
+        except Exception as e_shape_dtype:
+            app_logger.exception(f"raster shape or dtype not found:{e_shape_dtype}.", exc_info=True)
         app_logger.exception(f"e_crop_raster:{e_crop_raster}.", exc_info=True)
         raise e_crop_raster
 
@@ -160,9 +164,9 @@ def get_transform_raster(raster: ndarray, raster_bbox: tuple_float) -> tuple_nda
         transform = from_origin(x[0] - res_x / 2, y[-1] + res_y / 2, res_x, res_y)
         return np_rgb, transform
     except Exception as e_get_transform_raster:
-        app_logger.exception(f"arguments raster: {type(raster)}, {raster}.")
-        app_logger.exception(f"arguments raster_bbox: {type(raster_bbox)}, {raster_bbox}.")
-        app_logger.exception(f"e_get_transform_raster:{e_get_transform_raster}.")
+        app_logger.error(f"arguments raster: {type(raster)}, {raster}.")
+        app_logger.error(f"arguments raster_bbox: {type(raster_bbox)}, {raster_bbox}.")
+        app_logger.exception(f"e_get_transform_raster:{e_get_transform_raster}.", exc_info=True)
         raise e_get_transform_raster
 
 
@@ -172,6 +176,6 @@ def reshape_as_image(arr):
 
         return swapaxes(swapaxes(arr, 0, 2), 0, 1)
     except Exception as e_reshape_as_image:
-        app_logger.exception(f"arguments: {type(arr)}, {arr}.")
+        app_logger.error(f"arguments: {type(arr)}, {arr}.")
         app_logger.exception(f"e_reshape_as_image:{e_reshape_as_image}.", exc_info=True)
         raise e_reshape_as_image

src/prediction_api/predictors.py

@@ -46,9 +46,8 @@ def samexporter_predict(
     app_logger.debug(f"using a {model_name} instance model...")
     models_instance = models_dict[model_name]["instance"]
 
-    app_logger.info(f'tile_source: {url_tile}!')
     pt0, pt1 = bbox
-    app_logger.info(f"downloading geo-referenced raster with bbox {bbox}, zoom {zoom}.")
+    app_logger.info(f"tile_source: {url_tile}: downloading geo-referenced raster with bbox {bbox}, zoom {zoom}.")
     img, transform = download_extent(w=pt1[1], s=pt1[0], e=pt0[1], n=pt0[0], zoom=zoom, source=url_tile)
     app_logger.info(
         f"img type {type(img)} with shape/size:{img.size}, transform type: {type(transform)}, transform:{transform}.")