cylc / cylc-uiserver

A Jupyter Server extension that serves the cylc-ui web application for monitoring and controlling Cylc workflows.
https://cylc.org
GNU General Public License v3.0
15 stars 18 forks source link

Last task in workflow doesn't update to succeeded in graph GUI #630

Open jfrost-mo opened 1 week ago

jfrost-mo commented 1 week ago

Description

After a workflow reaches its final cycle point, and the last task completes, the reported state in the graph view of the GUI still claims the task to be running, when it is not.

Reproducible Example

Run a workflow, opening it in cylc gui on the graph view. Once the workflow reaches its final cycle point, and the last task completes, observe the reported state of that task on the graph. It will be left in a running state, despite actually having succeeded.

Expected Behaviour

The task in the GUI should be marked as succeeded.

Fraetor commented 1 week ago

An example. Note the workflow is stopped after having completed, but the task still claims to be running.

20240924_122816

MetRonnie commented 1 week ago

Duplicate of cylc/cylc-ui#1756

oliver-sanders commented 1 week ago

I think that's a different issue.

@jfrost-mo, if you switch back to the tree view, the task should still be visible there too, what state is it in?

MetRonnie commented 1 week ago

I can reproduce with a simple workflow.

if you switch back to the tree view, the task should still be visible there too, what state is it in?

The workflow completed so the tree view is empty after shutdown

oliver-sanders commented 1 week ago

Good point, would need an incomplete task to test the graph view against the tree view.

MetRonnie commented 1 week ago

Ok so not a duplicate because cylc/cylc-ui#1756 is just about clearing the graph view but the bug here is we are missing a delta update for the final state of the task before it is pruned

jfrost-mo commented 1 week ago

Clearing the graph would resolve this issue too, though I do personally find it quite disconcerting that the workflow finished state is just an empty window. Maybe some kind of graphic to indicate "Congratulations, your workflow finished." I see there is Issue https://github.com/cylc/cylc-ui/issues/332 for this.

oliver-sanders commented 1 week ago

That's the plan, we just haven't managed to get around to it!