From 7afc30012833fdb60228b44281fdf8fb290491a7 Mon Sep 17 00:00:00 2001 From: Daniel Hollas Date: Fri, 8 Sep 2023 11:03:56 +0100 Subject: [PATCH] Fix Warning coming from ProcessFollowerWidget (#512) We were passing a wrong thing to the ProcessMonitor constructor, but because it was just ignored as an extra keyword argument, it did not manifest as an actual bug. The functionality itself worked because we were passing the process uuid by linking the traitlets via `dlink()`. * Longer default timeout in ProcessFollowerWidget --------- Co-authored-by: Jusong Yu --- aiidalab_widgets_base/process.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/aiidalab_widgets_base/process.py b/aiidalab_widgets_base/process.py index 4501b7884..ffb6c9efa 100644 --- a/aiidalab_widgets_base/process.py +++ b/aiidalab_widgets_base/process.py @@ -261,7 +261,7 @@ def __init__( self, process=None, followers=None, - update_interval=0.1, + update_interval=1.0, path_to_root="../", **kwargs, ): @@ -302,7 +302,6 @@ def follow(self, detach=False): if self._monitor is None: self._monitor = ProcessMonitor( - process=self.process, callbacks=[self.update], on_sealed=self._run_after_completed, timeout=self.update_interval,