Spaces:
Sleeping
Sleeping
Commit
·
1709520
1
Parent(s):
6c983cf
Refactor logging setup in app.py and blob_utils.py to use module-level loggers for better context and remove redundant file handler configuration in logging_config.py.
Browse files- app.py +1 -1
- blob_utils.py +1 -1
- logging_config.py +2 -15
app.py
CHANGED
|
@@ -25,7 +25,7 @@ import install_private_repos # noqa: F401
|
|
| 25 |
from seavision import load_model
|
| 26 |
|
| 27 |
# Get loggers
|
| 28 |
-
logger = get_logger()
|
| 29 |
|
| 30 |
|
| 31 |
TITLE = """
|
|
|
|
| 25 |
from seavision import load_model
|
| 26 |
|
| 27 |
# Get loggers
|
| 28 |
+
logger = get_logger(__name__)
|
| 29 |
|
| 30 |
|
| 31 |
TITLE = """
|
blob_utils.py
CHANGED
|
@@ -9,7 +9,7 @@ from typing import Dict, Any
|
|
| 9 |
from logging_config import get_logger
|
| 10 |
|
| 11 |
# Module logger
|
| 12 |
-
logger = get_logger(
|
| 13 |
|
| 14 |
|
| 15 |
class BlobConverter:
|
|
|
|
| 9 |
from logging_config import get_logger
|
| 10 |
|
| 11 |
# Module logger
|
| 12 |
+
logger = get_logger(__name__)
|
| 13 |
|
| 14 |
|
| 15 |
class BlobConverter:
|
logging_config.py
CHANGED
|
@@ -5,7 +5,6 @@ Provides centralized logging setup with proper formatting and levels.
|
|
| 5 |
|
| 6 |
import logging
|
| 7 |
import sys
|
| 8 |
-
from pathlib import Path
|
| 9 |
|
| 10 |
|
| 11 |
class LoggerManager:
|
|
@@ -27,7 +26,7 @@ class LoggerManager:
|
|
| 27 |
def _setup_default_logging(self):
|
| 28 |
"""Setup default logging configuration."""
|
| 29 |
# Create main logger
|
| 30 |
-
self.logger = logging.getLogger(
|
| 31 |
self.logger.setLevel(logging.INFO)
|
| 32 |
|
| 33 |
# Prevent duplicate handlers
|
|
@@ -44,22 +43,10 @@ class LoggerManager:
|
|
| 44 |
console_handler.setFormatter(console_formatter)
|
| 45 |
self.logger.addHandler(console_handler)
|
| 46 |
|
| 47 |
-
# File handler (logs/app.log)
|
| 48 |
-
log_file = Path("logs/app.log")
|
| 49 |
-
log_file.parent.mkdir(parents=True, exist_ok=True)
|
| 50 |
-
file_handler = logging.FileHandler(log_file)
|
| 51 |
-
file_handler.setLevel(logging.DEBUG)
|
| 52 |
-
file_formatter = logging.Formatter(
|
| 53 |
-
fmt="%(asctime)s - %(name)s - %(levelname)s - %(funcName)s:%(lineno)d - %(message)s",
|
| 54 |
-
datefmt="%Y-%m-%d %H:%M:%S",
|
| 55 |
-
)
|
| 56 |
-
file_handler.setFormatter(file_formatter)
|
| 57 |
-
self.logger.addHandler(file_handler)
|
| 58 |
-
|
| 59 |
def get_logger(self, name: str = None) -> logging.Logger:
|
| 60 |
"""Get a logger instance."""
|
| 61 |
if name:
|
| 62 |
-
return logging.getLogger(
|
| 63 |
return self.logger
|
| 64 |
|
| 65 |
|
|
|
|
| 5 |
|
| 6 |
import logging
|
| 7 |
import sys
|
|
|
|
| 8 |
|
| 9 |
|
| 10 |
class LoggerManager:
|
|
|
|
| 26 |
def _setup_default_logging(self):
|
| 27 |
"""Setup default logging configuration."""
|
| 28 |
# Create main logger
|
| 29 |
+
self.logger = logging.getLogger()
|
| 30 |
self.logger.setLevel(logging.INFO)
|
| 31 |
|
| 32 |
# Prevent duplicate handlers
|
|
|
|
| 43 |
console_handler.setFormatter(console_formatter)
|
| 44 |
self.logger.addHandler(console_handler)
|
| 45 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 46 |
def get_logger(self, name: str = None) -> logging.Logger:
|
| 47 |
"""Get a logger instance."""
|
| 48 |
if name:
|
| 49 |
+
return logging.getLogger(name)
|
| 50 |
return self.logger
|
| 51 |
|
| 52 |
|