-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve the ProcessMonitor
#636
Comments
I believe point 1. is already implemented via the |
Hi @danielhollas , Thanks very much for pointing out this. I didn't notice this because of the name of the variable |
To clarify, on_sealed callbacks are run exactly once when the process finishes (i.e. when the corresponding node in aiida DB is sealed, made readonly). Arguably the name exposes an implementation detail of AiiDA and could be named on_finished or something, but I think having a good documentation for this should suffice instead of making breaking change. |
Ah, I may have misunderstood your comment, I see what you mean now. |
We may add more categories:
|
The ProcessMonitor class monitors a process by executing callback functions at specified intervals as long as the process is alive.
We may extend it to cover other use cases, for example:
The text was updated successfully, but these errors were encountered: