Closed danielhollas closed 1 year ago
Patch coverage: 100.00
% and no project coverage change.
Comparison is base (
ed0fd4e
) 79.32% compared to head (56be575
) 79.32%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
ProcessMonitor has a background thread that polls the process status. The default interval is now 0.1 seconds which is quite aggressive, and may potentially lead to memory leak as discussed on https://github.com/aiidalab/issues/issue/13
Here I propose to increase the default to 1.0s, since the typical AiiDA workflow will anyway take quite a bit longer. If that seems to large, perhaps 0.5s might be a good compromise? In any case, widgets can override this by passing the timeout parameter to the constructor.