-
Notifications
You must be signed in to change notification settings - Fork 0
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
Show dynamic job status in workflow page #90
Comments
A new endpoint was introduced in fractal-server 1.3.5 (and refined later, e.g. in 1.3.12): {
"status": {
1: "submitted",
2: "failed",
3: "done"
}
} where 1, 2 and 3 are WorkflowTask IDs. Here is a first proposal of how we can expose this endpoint for testing (*):
(*) The longer-term perspective is to integrate this information directly in the Workflow page, but here I'm only proposing a first way to play with this new feature and verify that its response matches with expectations. Note that the longer-term perspective above also requires establishing a link between a Workflow and a Dataset. |
Great, this sounds like a good way to start testing the status feature! |
I've sketched some scenarios for monitoring and how this can be displayed on the workflow page. The idea is that each task in the workflow task list shows the processing status as real-time as possible (how real-time is the database for the output ds history? Does it get updated during the run or only at the end of it?) Full sketches here: https://www.figma.com/file/n5CzmEcOadzSlMkUkA9Lut/Fractal-Web-Workflow-Monitoring?type=design&node-id=0-1&mode=design&t=zH5flcZpWwkDhMHy-0 It mostly comes down to already specifying the dataset(s) on the workflow page, instead of only during submission. That way, the history can then be shown on the workflow page. One of the conclusions: This will get much simpler if we move to a "single dataset approach", e.g. there is 1 dataset that the workflow works on, not input & output dataset. See also some analysis here on how the input datasets in only used very lightly: fractal-analytics-platform/fractal-server#785 What is still missing from those sketches: |
This is currently in-place. Let's open specific issues if needed. |
Some complexities:
The text was updated successfully, but these errors were encountered: