Skip to content

Pre batch processing task state (Option 1)#6860

Closed
dwsutherland wants to merge 1 commit intocylc:masterfrom
dwsutherland:ds-pre-batch-proc-option1
Closed

Pre batch processing task state (Option 1)#6860
dwsutherland wants to merge 1 commit intocylc:masterfrom
dwsutherland:ds-pre-batch-proc-option1

Conversation

@dwsutherland
Copy link
Copy Markdown
Member

@dwsutherland dwsutherland commented Jul 18, 2025

closes #6567, closes #6861, closes #6859

Like #6859 but uses pre-existing task state.

Check List

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.
  • Contains logically grouped changes (else tidy your branch by rebase).
  • Does not contain off-topic changes (use other PRs for other changes).
  • Applied any dependency changes to both setup.cfg (and conda-environment.yml if present).
  • Tests are included (or explain why tests are not needed).
  • Changelog entry included if this is a change that can affect users
  • Cylc-Doc pull request opened if required at cylc/cylc-doc/pull/XXXX.
  • If this is a bug fix, PR should be raised against the relevant ?.?.x branch.

Copy link
Copy Markdown
Member

@oliver-sanders oliver-sanders left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This state may well be incorrect leading to confusion.

@oliver-sanders
Copy link
Copy Markdown
Member

We shouldn't be patching the data in this way, but moreover, I've managed to reproduce the bug (instructions) on this branch.

I'm not quite sure how this bug is working, but I think it's different to what we've assumed.

I'll close this to help tidy up open PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

data store: blank task state

2 participants