Deprecate and remove logging parameters in LightRAG.
- Set log_level and log_file_path to None by default - Issue warnings if deprecated parameters are used - Maintain backward compatibility with warnings
This commit is contained in:
@@ -88,8 +88,8 @@ class LightRAG:
|
|||||||
|
|
||||||
# Logging (Deprecated, use setup_logger in utils.py instead)
|
# Logging (Deprecated, use setup_logger in utils.py instead)
|
||||||
# ---
|
# ---
|
||||||
log_level: int = field(default=logger.level)
|
log_level: int | None = field(default=None)
|
||||||
log_file_path: str = field(default=os.path.join(os.getcwd(), "lightrag.log"))
|
log_file_path: str | None = field(default=None)
|
||||||
|
|
||||||
# Entity extraction
|
# Entity extraction
|
||||||
# ---
|
# ---
|
||||||
@@ -268,21 +268,23 @@ class LightRAG:
|
|||||||
)
|
)
|
||||||
|
|
||||||
# Handle deprecated parameters
|
# Handle deprecated parameters
|
||||||
kwargs = self.__dict__
|
if self.log_level is not None:
|
||||||
if "log_level" in kwargs:
|
|
||||||
warnings.warn(
|
warnings.warn(
|
||||||
"WARNING: log_level parameter is deprecated, use setup_logger in utils.py instead",
|
"WARNING: log_level parameter is deprecated, use setup_logger in utils.py instead",
|
||||||
UserWarning,
|
UserWarning,
|
||||||
stacklevel=2,
|
stacklevel=2,
|
||||||
)
|
)
|
||||||
# Remove the attribute to prevent its use
|
if self.log_file_path is not None:
|
||||||
delattr(self, "log_level")
|
|
||||||
if "log_file_path" in kwargs:
|
|
||||||
warnings.warn(
|
warnings.warn(
|
||||||
"WARNING: log_file_path parameter is deprecated, use setup_logger in utils.py instead",
|
"WARNING: log_file_path parameter is deprecated, use setup_logger in utils.py instead",
|
||||||
UserWarning,
|
UserWarning,
|
||||||
stacklevel=2,
|
stacklevel=2,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
# Remove these attributes to prevent their use
|
||||||
|
if hasattr(self, "log_level"):
|
||||||
|
delattr(self, "log_level")
|
||||||
|
if hasattr(self, "log_file_path"):
|
||||||
delattr(self, "log_file_path")
|
delattr(self, "log_file_path")
|
||||||
|
|
||||||
initialize_share_data()
|
initialize_share_data()
|
||||||
|
Reference in New Issue
Block a user