-
Notifications
You must be signed in to change notification settings - Fork 13
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
378 update button for execution #396
Conversation
@geek-yang & @cpranav93
|
return html.Div(['']), utilities.blank_fig() | ||
if (not sel_methods): | ||
return html.Div(['']), utilities.blank_fig() | ||
# if ((ctx.triggered[0]["prop_id"] == "upload-model-img.filename") or |
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.
If this part is no longer needed, it can maybe be deleted?
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.
I think this part relates to the global cache in #418. So I do not want to delete it yet.
I have modified the layout to account fo @laurasootes's comments. I tried moving the button to the right but then I feel that the length of the window is too large - I would not recommend it. We may anyway redo the whole layout based on jesus' comments. Also, a round button is apparently complicated in dash! Who would have thought! I have now also added a stop explanation button which will stop the loading in the dashboard - however the actual computation in the background still continues to go on. I am in the process of trying to figure ths out but I think this warrants a new issue in itself. So if you agree and are happy with this PR, we can merge it and I can create a new issue. |
Implement an update button for the dashboard to run the given XAI methods with provided model and inputs.
This PR solves the rapid refreshing problem that occurs in PR #387 and it closes #378.