ClearML/clearml
1
0
Fork 0
mirror of https://github.com/clearml/clearml synced 2025-06-26 18:16:07 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix: ensure _logger is not None in Session

Browse Source 
This ensures that the module does not crash in case the logger was called when verbosity was not defined. The old behaviour caused it to crash since _logger was None, but was still called anyways.
This commit is contained in:
AbdulHamid Merii 2025-05-09 12:52:53 +01:00
parent 4d6b54d51a
commit 030f306088
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
clearml/backend_api/session/session.py
View File

@ -154,10 +154,14 @@ class Session(TokenManager):
self.__class__._sessions_weakrefs.append(weakref.ref(self))
self._verbose = verbose if verbose is not None else ENV_VERBOSE.get()
self._logger = logger
if self._verbose and not self._logger:
level = resolve_logging_level(ENV_VERBOSE.get(converter=str))
self._logger = get_logger(level=level, stream=sys.stderr if level is logging.DEBUG else None)
level = (
resolve_logging_level(ENV_VERBOSE.get(converter=str))
if self._verbose
else logging.WARNING
)
self._logger = logger or get_logger(
level=level, stream=sys.stderr if level is logging.DEBUG else None
)
self.__worker = worker or self.get_worker_host_name()
self.client = ", ".join("{}-{}".format(*x) for x in self._client)