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

119: time zone of initial and final cycle point #1014

Closed
hjoliver opened this issue Jul 15, 2014 · 2 comments
Closed

119: time zone of initial and final cycle point #1014

hjoliver opened this issue Jul 15, 2014 · 2 comments
Assignees
@benfitzpatrick
Labels
bug? Not sure if this is a bug or not
Milestone
cylc-6

Comments

@hjoliver
Copy link
Member

@benfitzpatrick - if I run a suite with my laptop on NZST rather than UTC, and the suite is not in UTC mode, the first and subsequent cycle points get "+12" appended, but $CYLC_SUITE_(INITIAL|FINAL)_CYCLE_POINTdon't - so comparisons with those variables in task scripting will always fail.
@hjoliver hjoliver added this to the cylc-6 milestone Jul 15, 2014
@hjoliver hjoliver added the bug? label Jul 15, 2014
@hjoliver
Copy link
Member Author

It looks like the variable just gets set to the literal string in suite.rc file, rather than set to the standard representation.

@hjoliver
Copy link
Member Author

(literal string comparison may be the wrong thing to do now, but I think the timezone is still an issue)

benfitzpatrick added a commit to benfitzpatrick/cylc that referenced this issue Jul 16, 2014
@benfitzpatrick
cylc#1014: correctly format cycle point env vars
69a5214
@benfitzpatrick benfitzpatrick mentioned this issue Jul 16, 2014
Merged
benfitzpatrick added a commit to benfitzpatrick/cylc that referenced this issue Jul 17, 2014
@benfitzpatrick
cylc#1014: remove redundant rm
c518b7d
hjoliver added a commit that referenced this issue Jul 20, 2014
@hjoliver
Merge pull request #1020 from benfitzpatrick/1014.initial-final-cycle…
3005096 
…-point-env-tz

#1014: correctly format cycle point env vars
@hjoliver hjoliver mentioned this issue Jul 20, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@benfitzpatrick benfitzpatrick

Labels
bug? Not sure if this is a bug or not
Projects
None yet
Milestone
cylc-6
Development

No branches or pull requests
2 participants
@benfitzpatrick @hjoliver