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

Add time period to output file #4

Open
gold2718 opened this issue Mar 7, 2024 · 2 comments · May be fixed by #12
Open

Add time period to output file #4

gold2718 opened this issue Mar 7, 2024 · 2 comments · May be fixed by #12
Assignees
Labels
bug Something isn't working

Comments

@gold2718
Copy link
Collaborator

gold2718 commented Mar 7, 2024

Currently, monthly and yearly files indicate the time period in their filenames.
However, when using the --merge-all option, the resulting date range is not included.
This information should be added to the compressed output files.
Noted by @lisesg

@gold2718 gold2718 added the bug Something isn't working label Mar 7, 2024
@gold2718 gold2718 self-assigned this Apr 16, 2024
@gold2718
Copy link
Collaborator Author

@lisesg ,
What is a good format for the date range?

<case>.<component>.<href>.yyyy-mm-dd_YYYY-MM-DD.nc   ?
<case>.<component>.<href>.yyyy-mm-dd_to_YYYY-MM-DD.nc   ?

Something else? Here:

  • yyyy-mm-dd is the earliest date on an input file
  • YYYY-MM-DD is the latest date on an input file
  • <case> is the usual case name prefix (<compset>_<res>_<casename>)
  • <component> is the component name (e.g., cam, blom)
  • <href> is the history file number (e.g., h1, hbgcd)

@lisesg
Copy link

lisesg commented Apr 23, 2024

I suggest to follow the same conventions we use for the substring in the daily CMIP6/CMOR-ized filenames that indicate the start and end dates, so something like:

<case>.<component>.<href>.yyyymmdd-YYYYMMDD.nc

And for experiments run using the multi-instance component:
<case>.<component>.<href>_<member>.yyyymmdd-YYYYMMDD.nc

(I added an underscore instead of a dot between <href> and <member> because this is in line with the default naming conventions of the raw model (time-slice) output. )

@gold2718 gold2718 linked a pull request May 3, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Todo
Development

Successfully merging a pull request may close this issue.

2 participants