Skip to content
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.

Sign up for GitHub

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

Jump to bottom

cylc 6: start-up task back-compat problem #1008

Closed
hjoliver opened this issue Jul 5, 2014 · 1 comment
Closed

cylc 6: start-up task back-compat problem #1008

hjoliver opened this issue Jul 5, 2014 · 1 comment
Assignees
@benfitzpatrick
Milestone
cylc-6

Comments

@hjoliver
Copy link
Member

hjoliver commented Jul 5, 2014

If prep is an old-style start-up task,

[scheduling]
   [[dependencies]]
        [[[0,6,12,18]]]
graph ="prep => X"

This gets auto-upgraded to prep[^] => X for the first four cycle points, then just X for the general recurrence. Which is not the same as in cylc-5, where the first instance of X triggers off prep, but all subsequent instances do not (mad but true).

@benfitzpatrick - is it easy for you to just apply the auto graph upgrade to the first cycle point in a list, where start-up and cold-start tasks are involved? (I'm sure it is easy as stated, but are there consequences for other suites...). Otherwise "wontfix" is probably fine as I doubt this is of much consequence in reality.
@hjoliver hjoliver added this to the cylc-6 milestone Jul 5, 2014
@hjoliver hjoliver mentioned this issue Jul 5, 2014
Merged
@hjoliver hjoliver changed the title 119: cold-start back-compat problem 119: start-up task back-compat problem Jul 5, 2014
benfitzpatrick added a commit to benfitzpatrick/cylc that referenced this issue Jul 21, 2014
@benfitzpatrick
cylc#1008: more minimal start-up dependencies
afda806
@benfitzpatrick benfitzpatrick mentioned this issue Jul 21, 2014
Merged
benfitzpatrick added a commit to benfitzpatrick/cylc that referenced this issue Jul 24, 2014
@benfitzpatrick
cylc#1008: add graphing tests
339accb
benfitzpatrick added a commit to benfitzpatrick/cylc that referenced this issue Jul 28, 2014
@benfitzpatrick
cylc#1008: add graph tests for cyclers suites
d141b17
benfitzpatrick added a commit to benfitzpatrick/cylc that referenced this issue Jul 28, 2014
@benfitzpatrick
cylc#1008: adjust tests for new start-up/async behaviour
8ef7ddf
benfitzpatrick added a commit to benfitzpatrick/cylc that referenced this issue Jul 28, 2014
@benfitzpatrick
cylc#1008: remove debug statements
dd85c27
benfitzpatrick added a commit to benfitzpatrick/cylc that referenced this issue Jul 28, 2014
@benfitzpatrick
cylc#1008: remove another debug statement
5118a9c
@hjoliver
Copy link
Member Author

The run time behaviour here is now fixed, but graphing is still broken, see: #1026 (comment)

hjoliver added a commit that referenced this issue Jul 30, 2014
@hjoliver
Merge pull request #1026 from benfitzpatrick/1008.start-up-back-comp-…
8c908cd 
…dep-tweak

#1008: more minimal start-up dependencies
@matthewrmshin matthewrmshin changed the title 119: start-up task back-compat problem cylc 6: start-up task back-compat problem Aug 15, 2014
benfitzpatrick added a commit to benfitzpatrick/cylc that referenced this issue Aug 18, 2014
@benfitzpatrick
cylc#1008: fix start-up graphing in simplest case
a93eb57
@benfitzpatrick benfitzpatrick mentioned this issue Aug 18, 2014
Merged
hjoliver added a commit that referenced this issue Aug 18, 2014
@hjoliver
Merge pull request #1094 from benfitzpatrick/1008.fix-start-up-graph
36f872f 
#1008: fix start-up graphing in simplest case
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@benfitzpatrick benfitzpatrick

Labels
None yet
Projects
None yet
Milestone
cylc-6
Development

No branches or pull requests
2 participants
@benfitzpatrick @hjoliver