Flake8/PEP8

clearml/binding/click.py

@@ -1,10 +1,10 @@
 try:
-    from click.core import Command, Option, Argument, OptionParser, Group, Context
-    from click.types import BoolParamType
+    from click.core import Command, Option, Argument, Group, Context  # noqa
+    from click.types import BoolParamType  # noqa
 except ImportError:
     Command = None
 
-from .frameworks import _patched_call
+from .frameworks import _patched_call  # noqa
 from ..config import running_remotely, get_remote_task_id
 from ..utilities.dicts import cast_str_to_bool
 

clearml/task.py

@@ -432,7 +432,7 @@ class Task(_Task):
                 cls.__main_task.__register_at_exit(cls.__main_task._at_exit)
                 # TODO: Check if the signal handler method is safe enough, for the time being, do not unhook
                 # cls.__main_task.__register_at_exit(None, only_remove_signal_and_exception_hooks=True)
-                
+
                 # start all reporting threads
                 BackgroundMonitor.start_all(task=cls.__main_task)
 
@@ -2897,10 +2897,10 @@ class Task(_Task):
             return
         # shutdown will clear the main, so we have to store it before.
         # is_main = self.is_main_task()
-        # fix debugger signal in the middle
+        # fix debugger signal in the middle, catch everything
         try:
             self.__shutdown()
-        except:
+        except:  # noqa
             pass
         # In rare cases we might need to forcefully shutdown the process, currently we should avoid it.
         # if is_main: