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 unlimited to time dimension in gaussian_grid netcdf output #1521

Closed
benjamin-cash opened this issue Dec 6, 2022 · 2 comments · Fixed by NOAA-EMC/fv3atm#612 or #1535
Closed

Add unlimited to time dimension in gaussian_grid netcdf output #1521

benjamin-cash opened this issue Dec 6, 2022 · 2 comments · Fixed by NOAA-EMC/fv3atm#612 or #1535
Labels
enhancement New feature or request

Comments

@benjamin-cash
Copy link

Description

gaussian_grid netcdf output files include "time" as a dimension but it is not defined as an unlimited dimension. @DeniseWorthen reports that this is not true when tiled netcdf output is used. It appears that this option was disabled for parallel netcdf output due to performance issues on Hera. Lack of a record dimension causes a number of follow on issues, particularly with the use of netcdf operators.

Solution

Allow for users to enable unlimited definition for netcdf output.

Alternatives

If applicable, add a description of any alternative solutions or features you've considered.

Related to

Directly reference any issues or PRs in this or other repositories that this is related to, and describe how they are related. Example:

  • required to support noaa-emc/nems/issues/<issue_number>
  • needed by noaa-emc/fv3atm/pull/<pr_number>
@benjamin-cash benjamin-cash added the enhancement New feature or request label Dec 6, 2022
@DusanJovic-NOAA
Copy link
Collaborator

Do 'netcdf operators' require that each netcdf file has a record dimension?

@DeniseWorthen
Copy link
Collaborator

To concatenate a set of netcdf files into a timeseries using NCO, the time dimension needs to be a "record" dimension ie, be "unlimited". If the time dimension is not unlimited, you need to first pre-process the netcdf file using ncks --mk_rec_dim.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
3 participants