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

Merge date placeholder type forecast initiation time and forecast lead time. #860

Closed
jfrost-mo opened this issue Oct 25, 2024 · 0 comments · Fixed by #1053
Closed

Merge date placeholder type forecast initiation time and forecast lead time. #860

jfrost-mo opened this issue Oct 25, 2024 · 0 comments · Fixed by #1053
Labels
enhancement New feature or request

Comments

@jfrost-mo
Copy link
Member

What problem does your feature request solve?

Some data contains both the forecast initiation time and the forecast lead time in the file name. To allow selecting this kind of data we should be able to have the placeholders in the same date placeholder type.

Describe the solution you'd like

Merge the lead time date type into the forecast initiation date type. There is no conflict, as the lead time uses a different syntax for its placeholders. The documentation will also need updating.

Describe alternatives you've considered
@jfrost-mo jfrost-mo added the enhancement New feature or request label Oct 25, 2024
jfrost-mo added a commit that referenced this issue Jan 17, 2025
@jfrost-mo
Merge lead time into initiation time cycling
5626327 
The lead time functionality is a subset, so we only need the one mode.
The entry for lead time in rose edit will be removed in a follow-up change.

Fixes #860
@jfrost-mo jfrost-mo mentioned this issue Jan 17, 2025
Merged
7 tasks
jfrost-mo added a commit that referenced this issue Jan 20, 2025
@jfrost-mo
Merge lead time into initiation time cycling
8b8b768 
The lead time functionality is a subset, so we only need the one mode.
The entry for lead time in rose edit will be removed in a follow-up change.

Fixes #860
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Merge lead time into initiation time cycling MetOffice/CSET
1 participant
@jfrost-mo