-
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
ProcessMonitorWidget: Print traceback for callback exceptions #427
Conversation
@yakutovicha here's a code snippet to test this in Jupyter Notebook from aiida.orm import load_node
from aiidalab_widgets_base import ProcessMonitor
def raise_error():
raise ValueError("We have a problem")
monitor = ProcessMonitor(
timeout=0.5,
callbacks=[
raise_error,
],
)
process_pk = 1217
monitor.value = load_node(process_pk).uuid |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @danielhollas. I just have a minor request.
Co-authored-by: Aliaksandr Yakutovich <yakutovicha@gmail.com>
@yakutovicha thanks, I've accepted your suggestion. Once this is merged, could you please release a new beta version? I want to use this for better in my app for easier debugging. |
sure, will do that right away |
In order to use mbarkhau/bumpver#207 for backport compatibility support for branch `support-23.04.x`.
Closes #386 (see the issue for details why this change is needed)
Before:
After:
NOTES: