feat: now all logging instances are based on logging, removed logging dependency

poetry.lock

@@ -4074,4 +4074,4 @@ files = [
 [metadata]
 lock-version = "2.0"
 python-versions = "~3.10"
-content-hash = "54a69094c5b62a8e766dbcba310e0d11aca34a546bfd2ef293441b31a3b781db"
+content-hash = "2e4a00a9ef0e444d2f23419f8b5cf8ddb9fbadeba0974a8f28e835bed0997b4f"

pyproject.toml

@@ -14,7 +14,6 @@ version = "1.4.0"
 bson = "^0.5.10"
 contextily = "^1.6.0"
 geopandas = "^1.0.1"
-loguru = "^0.7.2"
 numpy = "1.25.2"
 opencv-python-headless = "^4.8.1.78"
 pillow = "^10.4.0"
@@ -31,7 +30,6 @@ optional = true
 
 [tool.poetry.group.fastapi.dependencies]
 fastapi = "^0.111.0"
-loguru = "^0.7.2"
 pydantic = "^2.8.2"
 uvicorn = "^0.30.1" 
 

requirements.txt

@@ -3,7 +3,6 @@ contextily==1.6.0
 fastapi==0.111.0
 geopandas==1.0.1
 lisa-on-cuda==1.3.3
-loguru==0.7.2
 numpy==1.25.2
 opencv-python-headless==4.8.1.78
 pillow==10.4.0

samgis_lisa_on_zero/__init__.py

@@ -4,24 +4,16 @@ import os
 # not used here but contextily_tile is imported in samgis_lisa_on_zero.io_package.tms2geotiff
 from contextily import tile as contextily_tile
 from pathlib import Path
+
+from lisa_on_cuda.utils import session_logger
+
 from samgis_lisa_on_zero.utilities.constants import SERVICE_NAME
 
+
 ROOT = Path(globals().get("__file__", "./_")).absolute().parent.parent
 PROJECT_ROOT_FOLDER = Path(os.getenv("PROJECT_ROOT_FOLDER", ROOT))
 WORKDIR = Path(os.getenv("WORKDIR", ROOT))
 MODEL_FOLDER_PROJECT_ROOT_FOLDER = Path(PROJECT_ROOT_FOLDER / "machine_learning_models")
 MODEL_FOLDER = Path(os.getenv("MODEL_FOLDER", MODEL_FOLDER_PROJECT_ROOT_FOLDER))
-
-IS_AWS_LAMBDA = bool(os.getenv("IS_AWS_LAMBDA", ""))
-
-if IS_AWS_LAMBDA:
-    try:
-        from aws_lambda_powertools import Logger
-
-        app_logger = Logger(service=SERVICE_NAME)
-    except ModuleNotFoundError:
-        print("this should be AWS LAMBDA environment but we miss the required aws lambda powertools package")
-else:
-    from samgis_core.utilities.fastapi_logger import setup_logging
-
-    app_logger = setup_logging(debug=True)
+loglevel = os.getenv('LOGLEVEL', 'INFO').upper()
+session_logger.change_logging(loglevel)

samgis_lisa_on_zero/io_package/coordinates_pixel_conversion.py

@@ -1,12 +1,16 @@
 """functions useful to convert to/from latitude-longitude coordinates to pixel image coordinates"""
+import logging
+
 from lisa_on_cuda.utils import session_logger
 from samgis_core.utilities.type_hints import TupleFloat, TupleFloatAny
-from samgis_lisa_on_zero import app_logger
 from samgis_lisa_on_zero.utilities.constants import TILE_SIZE, EARTH_EQUATORIAL_RADIUS
 from samgis_lisa_on_zero.utilities.type_hints import ImagePixelCoordinates
 from samgis_lisa_on_zero.utilities.type_hints import LatLngDict
 
 
+app_logger = logging.getLogger(__name__)
+
+
 @session_logger.set_uuid_logging
 def _get_latlng2pixel_projection(latlng: LatLngDict) -> ImagePixelCoordinates:
     from math import log, pi, sin

samgis_lisa_on_zero/io_package/geo_helpers.py

@@ -1,10 +1,14 @@
 """handle geo-referenced raster images"""
+import logging
+
 from affine import Affine
 from numpy import ndarray as np_ndarray
 
 from lisa_on_cuda.utils import session_logger
 from samgis_core.utilities.type_hints import ListFloat, TupleFloat, DictStrInt
-from samgis_lisa_on_zero import app_logger
+
+
+app_logger = logging.getLogger(__name__)
 
 
 @session_logger.set_uuid_logging

samgis_lisa_on_zero/io_package/raster_helpers.py

@@ -1,14 +1,18 @@
 """helpers for computer vision duties"""
+import logging
+
 import numpy as np
 from numpy import ndarray, bitwise_not
 from rasterio import open as rasterio_open
 
 from lisa_on_cuda.utils import session_logger
-from samgis_lisa_on_zero import app_logger
 from samgis_lisa_on_zero.utilities.constants import OUTPUT_CRS_STRING
 from samgis_lisa_on_zero.utilities.type_hints import XYZTerrainProvidersNames
 
 
+app_logger = logging.getLogger(__name__)
+
+
 def get_nextzen_terrain_rgb_formula(red: ndarray, green: ndarray, blue: ndarray) -> ndarray:
     """
     Compute a 32-bits 2d digital elevation model from a nextzen 'terrarium' (terrain-rgb) raster.

samgis_lisa_on_zero/io_package/tms2geotiff.py

@@ -1,3 +1,4 @@
+import logging
 import os
 
 from numpy import ndarray
@@ -5,13 +6,13 @@ from samgis_core.utilities.type_hints import TupleFloat
 from xyzservices import TileProvider
 
 from lisa_on_cuda.utils import session_logger
-from samgis_lisa_on_zero import app_logger
 from samgis_lisa_on_zero.utilities.constants import (OUTPUT_CRS_STRING, DRIVER_RASTERIO_GTIFF, N_MAX_RETRIES,
                                                      N_CONNECTION, N_WAIT,
                                                      ZOOM_AUTO, BOOL_USE_CACHE)
 from samgis_lisa_on_zero.utilities.type_hints import tuple_ndarray_transform
 
 
+app_logger = logging.getLogger(__name__)
 bool_use_cache = int(os.getenv("BOOL_USE_CACHE", BOOL_USE_CACHE))
 n_connection = int(os.getenv("N_CONNECTION", N_CONNECTION))
 n_max_retries = int(os.getenv("N_MAX_RETRIES", N_MAX_RETRIES))

samgis_lisa_on_zero/io_package/wrappers_helpers.py

@@ -1,17 +1,20 @@
 """lambda helper functions"""
+import logging
 from typing import Dict
 
 from xyzservices import providers, TileProvider
 
 from lisa_on_cuda.utils import session_logger
 from lisa_on_cuda.utils.app_helpers import get_cleaned_input
-from samgis_lisa_on_zero import app_logger
 from samgis_lisa_on_zero.io_package.coordinates_pixel_conversion import get_latlng_to_pixel_coordinates
 from samgis_lisa_on_zero.utilities.constants import COMPLETE_URL_TILES_MAPBOX, COMPLETE_URL_TILES_NEXTZEN
 from samgis_lisa_on_zero.utilities.type_hints import (
     ApiRequestBody, XYZTerrainProvidersNames, XYZDefaultProvidersNames, StringPromptApiRequestBody)
 
 
+app_logger = logging.getLogger(__name__)
+
+
 @session_logger.set_uuid_logging
 def get_parsed_bbox_points_with_string_prompt(request_input: StringPromptApiRequestBody) -> Dict:
     """

samgis_lisa_on_zero/prediction_api/lisa.py

@@ -1,3 +1,4 @@
+import logging
 from datetime import datetime
 
 from lisa_on_cuda.utils import session_logger
@@ -11,11 +12,11 @@ from samgis_lisa_on_zero.utilities.constants import DEFAULT_URL_TILES, LISA_INFE
 
 
 msg_write_tmp_on_disk = "found option to write images and geojson output..."
+app_logger = logging.getLogger(__name__)
 
 
 @session_logger.set_uuid_logging
 def load_model_and_inference_fn(inference_function_name_key: str):
-    from samgis_lisa_on_zero import app_logger
     from lisa_on_cuda.utils import app_helpers
     from samgis_lisa_on_zero.prediction_api.global_models import models_dict
 
@@ -61,7 +62,6 @@ def lisa_predict(
         Affine transform
     """
     from os import getenv
-    from samgis_lisa_on_zero import app_logger
     from samgis_lisa_on_zero.prediction_api.global_models import models_dict
 
     if source_name is None:

samgis_lisa_on_zero/prediction_api/predictors.py

@@ -1,10 +1,12 @@
 """functions using machine learning instance model(s)"""
+import logging
+
 from lisa_on_cuda.utils import session_logger
 from samgis_core.prediction_api import sam_onnx2, sam_onnx_inference
 from samgis_core.utilities.constants import MODEL_ENCODER_NAME, MODEL_DECODER_NAME, DEFAULT_INPUT_SHAPE
 from samgis_core.utilities.type_hints import LlistFloat, DictStrInt, ListDict
 
-from samgis_lisa_on_zero import app_logger, MODEL_FOLDER
+from samgis_lisa_on_zero import MODEL_FOLDER
 from samgis_lisa_on_zero.io_package.geo_helpers import get_vectorized_raster_as_geojson
 from samgis_lisa_on_zero.io_package.raster_helpers import get_raster_terrain_rgb_like, get_rgb_prediction_image
 from samgis_lisa_on_zero.io_package.tms2geotiff import download_extent
@@ -12,6 +14,8 @@ from samgis_lisa_on_zero.io_package.wrappers_helpers import check_source_type_is
 from samgis_lisa_on_zero.prediction_api.global_models import models_dict, embedding_dict
 from samgis_lisa_on_zero.utilities.constants import DEFAULT_URL_TILES, SLOPE_CELLSIZE
 
+app_logger = logging.getLogger(__name__)
+
 
 @session_logger.set_uuid_logging
 def samexporter_predict(