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
@ alejoe91, can you explain why this call to self._save_run_info() line got deleted? I think it's important to have it saved before the run starts, otherwise if the run crashes there is no run_info present to be loaded.
It doesn't accomplish the intended use case, which was to allow the user (or the code) to check the run info dict for run_completed as being true or false in case the analyzer code crashes. As it is, one could check for the existence of the run info json file as a proxy for whether the run finished, I suppose, but that's a bit counterintuitive.
Originally posted by @jonahpearl in ff07ac6
The text was updated successfully, but these errors were encountered: