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.

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

special notation for initial and final cycle points #1862

Conversation

oliver-sanders
Copy link
Member

Closes #1038

Sections can now be specified like:

  • [[[ R1/^+PT1H ]]]
  • [[[ R1/$-PT1H ]]]

@benfitzpatrick Please Review
@hjoliver Please Review

@oliver-sanders oliver-sanders added this to the next release milestone May 25, 2016
@hjoliver
Copy link
Member

This is a nice, simple solution. We'll need to document it in the User Guide though. I'll leave this under "next-release" for now, in case this is all done in time, otherwise will push it back to "soon" so we can get the recent rash of bug fixes out quickly...

graph = "end_garply"

# Repeat once three days before the final cycle point
Copy link
Member

@hjoliver hjoliver May 26, 2016

Choose a reason for hiding this comment

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

This is consistent with our existing documentation, but I wonder if we should change "Repeat once" to "Run once" - the former is kind of nonsensical (it suggests doing something twice)??

Copy link
Member

Choose a reason for hiding this comment

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

(can be another PR)

Copy link
Contributor

Choose a reason for hiding this comment

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

👍

Copy link
Member Author

@oliver-sanders oliver-sanders May 27, 2016

Choose a reason for hiding this comment

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

I agree run makes more sense than repeat, will do in a future pull request.

@benfitzpatrick
Copy link
Contributor

This is a nice, simple solution

I agree!

@hjoliver hjoliver modified the milestones: soon, next release May 29, 2016
@hjoliver
Copy link
Member

(pushed back to soon; we can bring forward again if done in time).

@hjoliver hjoliver modified the milestones: next release, soon Jun 1, 2016
[scheduling]
[[dependencies]]
# Repeat once 12 hours after the initial cycle point
# Equivalent to [[[ R1/T12 ]]]
Copy link
Contributor

Choose a reason for hiding this comment

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

Equivalent to [[[ R1/+PT12H ]]]

@benfitzpatrick
Copy link
Contributor

One comment to fix, otherwise looks good.

@oliver-sanders oliver-sanders force-pushed the 1038.special-notation-for-initial-and-final-cycle-points branch from a30ac8c to 0899c38 Compare June 8, 2016 10:59
@oliver-sanders
Copy link
Member Author

@benfitzpatrick Comment addressed.

@benfitzpatrick
Copy link
Contributor

0899c38 looks good.

@benfitzpatrick benfitzpatrick merged commit 5716964 into cylc:master Jun 15, 2016
@oliver-sanders oliver-sanders deleted the 1038.special-notation-for-initial-and-final-cycle-points branch May 23, 2017 08:54
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.

3 participants