You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We have now overloaded showResult in the Training view with functionality unrelated to showing results. IMO we should abstract View.showResult to something like View.LongRunningTaskOnComplete which then calls showResultDialog and mainModel.training_complete.
However...
Our refactor plans include refactoring the entire LongRunningTask mechanism. So this info could be incorporated in our replacement, or it could end up being a mute point.
The text was updated successfully, but these errors were encountered:
In the current design:
We have now overloaded
showResult
in the Training view with functionality unrelated to showing results. IMO we should abstractView.showResult
to something likeView.LongRunningTaskOnComplete
which then callsshowResultDialog
andmainModel.training_complete
.However...
Our refactor plans include refactoring the entire LongRunningTask mechanism. So this info could be incorporated in our replacement, or it could end up being a mute point.
The text was updated successfully, but these errors were encountered: