yangdx commited on
Commit
6ad6d1c
·
1 Parent(s): c11e4ce

Deprecate and remove logging parameters in LightRAG.

Browse files

- Set log_level and log_file_path to None by default
- Issue warnings if deprecated parameters are used
- Maintain backward compatibility with warnings

Files changed (1) hide show
  1. lightrag/lightrag.py +9 -7
lightrag/lightrag.py CHANGED
@@ -88,8 +88,8 @@ class LightRAG:
88
 
89
  # Logging (Deprecated, use setup_logger in utils.py instead)
90
  # ---
91
- log_level: int = field(default=logger.level)
92
- log_file_path: str = field(default=os.path.join(os.getcwd(), "lightrag.log"))
93
 
94
  # Entity extraction
95
  # ---
@@ -268,21 +268,23 @@ class LightRAG:
268
  )
269
 
270
  # Handle deprecated parameters
271
- kwargs = self.__dict__
272
- if "log_level" in kwargs:
273
  warnings.warn(
274
  "WARNING: log_level parameter is deprecated, use setup_logger in utils.py instead",
275
  UserWarning,
276
  stacklevel=2,
277
  )
278
- # Remove the attribute to prevent its use
279
- delattr(self, "log_level")
280
- if "log_file_path" in kwargs:
281
  warnings.warn(
282
  "WARNING: log_file_path parameter is deprecated, use setup_logger in utils.py instead",
283
  UserWarning,
284
  stacklevel=2,
285
  )
 
 
 
 
 
286
  delattr(self, "log_file_path")
287
 
288
  initialize_share_data()
 
88
 
89
  # Logging (Deprecated, use setup_logger in utils.py instead)
90
  # ---
91
+ log_level: int | None = field(default=None)
92
+ log_file_path: str | None = field(default=None)
93
 
94
  # Entity extraction
95
  # ---
 
268
  )
269
 
270
  # Handle deprecated parameters
271
+ if self.log_level is not None:
 
272
  warnings.warn(
273
  "WARNING: log_level parameter is deprecated, use setup_logger in utils.py instead",
274
  UserWarning,
275
  stacklevel=2,
276
  )
277
+ if self.log_file_path is not None:
 
 
278
  warnings.warn(
279
  "WARNING: log_file_path parameter is deprecated, use setup_logger in utils.py instead",
280
  UserWarning,
281
  stacklevel=2,
282
  )
283
+
284
+ # Remove these attributes to prevent their use
285
+ if hasattr(self, "log_level"):
286
+ delattr(self, "log_level")
287
+ if hasattr(self, "log_file_path"):
288
  delattr(self, "log_file_path")
289
 
290
  initialize_share_data()