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

Option to represent date-times in long ISO format #962

Open
MetRonnie opened this issue Mar 7, 2022 · 5 comments
Open

Option to represent date-times in long ISO format #962

MetRonnie opened this issue Mar 7, 2022 · 5 comments
Milestone

Comments

@MetRonnie
Copy link
Member

Describe exactly what you would like to see in an upcoming release

We should consider displaying date-times in the long ISO format in the UI, even if the cycle point format of a workflow is the short format, as it will make cycle points easier to read. I.e. 2000-01-01T00:00Z instead of 20000101T0000Z.

Pull requests welcome!

@MetRonnie MetRonnie added the question Flag this as a question for the next Cylc project meeting. label Mar 7, 2022
@MetRonnie MetRonnie added this to the 1.x milestone Mar 7, 2022
@hjoliver
Copy link
Member

hjoliver commented Mar 7, 2022

Agreed, that would be nice. Although it might be a little dangerous, if it leads users to use the wrong datetime format on the command line? (maybe we can automatically convert to the workflow format... I think we do that already for cylc suite-state xtriggers).

@MetRonnie
Copy link
Member Author

I don't think it matters what users use on the command line? According to the docs

The time zone you specify here will be used only for writing/dumping cycle points

@hjoliver
Copy link
Member

hjoliver commented Mar 8, 2022

I don't think it matters what users use on the command line? According to the docs

The time zone you specify here will be used only for writing/dumping cycle points

I mean the whole datetime format, not just the timezone.

E.g. for a monthly-cycling workflow with this:

cycle point format = %Y-%m

The CLI has to use the same format as the scheduler because no conversion is done. If wf//2025-01/foo is running:

$ cylc kill wf//2025-01/foo  # OK
$ cylc kill wf//202501/foo  # ERROR (task not found)

@MetRonnie
Copy link
Member Author

Argh, clearly I needed more coffee yesterday ☕

@MetRonnie MetRonnie changed the title Represent date-times in long ISO format Option to represent date-times in long ISO format May 4, 2022
@MetRonnie
Copy link
Member Author

Since cylc/cylc-flow#4881 the CLI no longer has to be given the same format as the workflow cycle point format

@MetRonnie MetRonnie added accessibility and removed question Flag this as a question for the next Cylc project meeting. labels Jun 8, 2022
@oliver-sanders oliver-sanders modified the milestones: 1.x, Pending Jun 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants