Merge pull request #1291 from kremnik/singleton

deepface/DeepFace.py

@@ -16,7 +16,7 @@
 
 # package dependencies
 from deepface.commons import package_utils, folder_utils
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 from deepface.modules import (
     modeling,
     representation,
@@ -29,7 +29,7 @@
 )
 from deepface import __version__
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # -----------------------------------
 # configurations for dependencies

deepface/api/src/app.py

@@ -5,9 +5,9 @@
 # project dependencies
 from deepface import DeepFace
 from deepface.api.src.modules.core.routes import blueprint
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 
 def create_app():

deepface/api/src/modules/core/routes.py

@@ -1,9 +1,9 @@
 from flask import Blueprint, request
 from deepface import DeepFace
 from deepface.api.src.modules.core import service
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 blueprint = Blueprint("routes", __name__)
 

deepface/basemodels/ArcFace.py

@@ -3,9 +3,9 @@
 from deepface.commons import package_utils, folder_utils
 from deepface.models.FacialRecognition import FacialRecognition
 
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # pylint: disable=unsubscriptable-object
 

deepface/basemodels/DeepID.py

@@ -2,9 +2,9 @@
 import gdown
 from deepface.commons import package_utils, folder_utils
 from deepface.models.FacialRecognition import FacialRecognition
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 tf_version = package_utils.get_tf_major_version()
 

deepface/basemodels/Dlib.py

@@ -5,9 +5,9 @@
 import numpy as np
 from deepface.commons import folder_utils
 from deepface.models.FacialRecognition import FacialRecognition
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # pylint: disable=too-few-public-methods
 

deepface/basemodels/Facenet.py

@@ -2,9 +2,9 @@
 import gdown
 from deepface.commons import package_utils, folder_utils
 from deepface.models.FacialRecognition import FacialRecognition
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # --------------------------------
 # dependency configuration

deepface/basemodels/FbDeepFace.py

@@ -3,9 +3,9 @@
 import gdown
 from deepface.commons import package_utils, folder_utils
 from deepface.models.FacialRecognition import FacialRecognition
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # --------------------------------
 # dependency configuration

deepface/basemodels/GhostFaceNet.py

@@ -8,9 +8,9 @@
 # project dependencies
 from deepface.commons import package_utils, folder_utils
 from deepface.models.FacialRecognition import FacialRecognition
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 tf_major = package_utils.get_tf_major_version()
 if tf_major == 1:

deepface/basemodels/OpenFace.py

@@ -3,9 +3,9 @@
 import tensorflow as tf
 from deepface.commons import package_utils, folder_utils
 from deepface.models.FacialRecognition import FacialRecognition
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 tf_version = package_utils.get_tf_major_version()
 if tf_version == 1:

deepface/basemodels/SFace.py

@@ -10,9 +10,9 @@
 # project dependencies
 from deepface.commons import folder_utils
 from deepface.models.FacialRecognition import FacialRecognition
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # pylint: disable=line-too-long, too-few-public-methods
 

deepface/basemodels/VGGFace.py

@@ -5,9 +5,9 @@
 from deepface.commons import package_utils, folder_utils
 from deepface.modules import verification
 from deepface.models.FacialRecognition import FacialRecognition
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # ---------------------------------------
 

deepface/commons/file_utils.py

@@ -5,9 +5,9 @@
 import gdown
 
 # project dependencies
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 
 def download_external_file(file_name: str, exact_file_path: str, url: str) -> None:

deepface/commons/folder_utils.py

@@ -1,8 +1,8 @@
 import os
 from pathlib import Path
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 
 def initialize_folder() -> None:

deepface/commons/logger.py

@@ -4,18 +4,34 @@
 
 # pylint: disable=broad-except
 class Logger:
-    def __init__(self, module=None):
-        self.module = module
-        log_level = os.environ.get("DEEPFACE_LOG_LEVEL", str(logging.INFO))
-        try:
-            self.log_level = int(log_level)
-        except Exception as err:
-            self.dump_log(
-                f"Exception while parsing $DEEPFACE_LOG_LEVEL."
-                f"Expected int but it is {log_level} ({str(err)})."
-                "Setting app log level to info."
-            )
-            self.log_level = logging.INFO
+    """
+    A Logger class for logging messages with a specific log level.
+
+    The class follows the singleton design pattern, ensuring that only one
+    instance of the Logger is created. The parameters of the first instance 
+    are preserved across all instances.
+    """
+
+    __instance = None
+
+    def __new__(cls):
+        if cls.__instance is None:
+            cls.__instance = super(Logger, cls).__new__(cls)
+        return cls.__instance
+
+    def __init__(self):
+        if not hasattr(self, "_singleton_initialized"):
+            self._singleton_initialized = True # to prevent multiple initializations
+            log_level = os.environ.get("DEEPFACE_LOG_LEVEL", str(logging.INFO))
+            try:
+                self.log_level = int(log_level)
+            except Exception as err:
+                self.dump_log(
+                    f"Exception while parsing $DEEPFACE_LOG_LEVEL."
+                    f"Expected int but it is {log_level} ({str(err)})."
+                    "Setting app log level to info."
+                )
+                self.log_level = logging.INFO
 
     def info(self, message):
         if self.log_level <= logging.INFO:
@@ -39,16 +55,3 @@ def critical(self, message):
 
     def dump_log(self, message):
         print(f"{str(datetime.now())[2:-7]} - {message}")
-
-
-def get_singletonish_logger():
-    # singleton design pattern
-    global model_obj
-
-    if not "model_obj" in globals():
-        model_obj = {}
-
-    if "logger" not in model_obj.keys():
-        model_obj["logger"] = Logger(module="Singleton")
-
-    return model_obj["logger"]

deepface/commons/package_utils.py

@@ -2,9 +2,9 @@
 import tensorflow as tf
 
 # package dependencies
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 
 def get_tf_major_version() -> int:

deepface/detectors/CenterFace.py

@@ -10,9 +10,9 @@
 # project dependencies
 from deepface.commons import folder_utils
 from deepface.models.Detector import Detector, FacialAreaRegion
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # pylint: disable=c-extension-no-member
 

deepface/detectors/DetectorWrapper.py

@@ -15,9 +15,9 @@
     YuNet,
     CenterFace,
 )
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 
 def build_model(detector_backend: str) -> Any:

deepface/detectors/Dlib.py

@@ -5,9 +5,9 @@
 import numpy as np
 from deepface.commons import folder_utils
 from deepface.models.Detector import Detector, FacialAreaRegion
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 
 class DlibClient(Detector):

deepface/detectors/Ssd.py

@@ -7,9 +7,9 @@
 from deepface.detectors import OpenCv
 from deepface.commons import folder_utils
 from deepface.models.Detector import Detector, FacialAreaRegion
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # pylint: disable=line-too-long, c-extension-no-member
 

deepface/detectors/Yolo.py

@@ -4,9 +4,9 @@
 import gdown
 from deepface.models.Detector import Detector, FacialAreaRegion
 from deepface.commons import folder_utils
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # Model's weights paths
 PATH = "/.deepface/weights/yolov8n-face.pt"

deepface/detectors/YuNet.py

@@ -10,9 +10,9 @@
 # project dependencies
 from deepface.commons import folder_utils
 from deepface.models.Detector import Detector, FacialAreaRegion
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 
 class YuNetClient(Detector):

deepface/extendedmodels/Age.py

@@ -4,9 +4,9 @@
 from deepface.basemodels import VGGFace
 from deepface.commons import package_utils, folder_utils
 from deepface.models.Demography import Demography
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # ----------------------------------------
 # dependency configurations

deepface/extendedmodels/Emotion.py

@@ -9,9 +9,9 @@
 # project dependencies
 from deepface.commons import package_utils, folder_utils
 from deepface.models.Demography import Demography
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # -------------------------------------------
 # pylint: disable=line-too-long

deepface/extendedmodels/Gender.py

@@ -9,9 +9,9 @@
 from deepface.basemodels import VGGFace
 from deepface.commons import package_utils, folder_utils
 from deepface.models.Demography import Demography
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # -------------------------------------
 # pylint: disable=line-too-long

deepface/extendedmodels/Race.py

@@ -9,9 +9,9 @@
 from deepface.basemodels import VGGFace
 from deepface.commons import package_utils, folder_utils
 from deepface.models.Demography import Demography
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # --------------------------
 # pylint: disable=line-too-long

deepface/modules/detection.py

@@ -11,9 +11,9 @@
 from deepface.models.Detector import DetectedFace, FacialAreaRegion
 from deepface.detectors import DetectorWrapper
 from deepface.commons import image_utils
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 # pylint: disable=no-else-raise
 

deepface/modules/recognition.py

@@ -12,9 +12,9 @@
 # project dependencies
 from deepface.commons import image_utils
 from deepface.modules import representation, detection, verification
-from deepface.commons import logger as log
+from deepface.commons.logger import Logger
 
-logger = log.get_singletonish_logger()
+logger = Logger()
 
 
 def find(