Add notify api running calls as soon as the flow runs start
czi flow run uses async/await. Wait for prefect future results before sending it to async calls
Remove subflow for small dm conversion flow
Revert Completion/Failed to original success/error api calls on notify-api-completion
Fix BRT flow: Add dependencies between task runs (replaces wait_for upstream tasks)... Increase cluster size for BRT workflow to be able to handle large number of inputs.
cleanup workdir moved to hooks from tasks.
Add error for task as messages in callback response using task failure hooks (uses file i/o)
Shortcomings
Using file i/o to store errors (this is used to collect error messages to collate in callback response; also, here to store which temporary file locations need to be cleared by on_completion/on_failure hooks) --- Two functions of interest in utils.py are: get_flow_run_tmpdirs and store_exception_hook (for two respective issues).
This PR doesn't introduce any:
[x] Binary files
[x] Temporary files, auto-generated files
[x] Secret keys
[x] Local debugging print statements
[x] Unwanted comments (e.g: # Gets user from environment for code os.environ['user'] )
Changes
notify api running
calls as soon as the flow runs startCompletion/Failed
to originalsuccess/error
api calls on notify-api-completioncleanup workdir
moved to hooks from tasks.Shortcomings
utils.py
are:get_flow_run_tmpdirs
andstore_exception_hook
(for two respective issues).This PR doesn't introduce any:
print
statementsos.environ['user']
)This PR contains valid: