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

Convert *.ncl files to *nc files in the test directory t #253

Open
mlee03 opened this issue Oct 20, 2023 · 2 comments
Open

Convert *.ncl files to *nc files in the test directory t #253

mlee03 opened this issue Oct 20, 2023 · 2 comments

Comments

@mlee03
Copy link
Contributor

mlee03 commented Oct 20, 2023

Is your feature request related to a problem? Please describe.
There are a lot of ncl files in the test directory. These files are converted tonetcdf(*.nc) files with ncgen for testing.
Because netcdf files occupy less space, it would be preferable to store the test input files as netcdf files instead of converting the ncl files to netcdf files.

Describe the solution you'd like
Replace all ncl files to netcdf files.

Describe alternatives you've considered
Leave the ncl files as they are currently.

Additional context

@ngs333
Copy link
Contributor

ngs333 commented Oct 24, 2023

@mlee03
@ceblanton
Is there a need for the size reduction, and how much of a benefit do you get. Note that the large files should already be here:
/home/fms/fre-nctools/test-datasets

@ceblanton
Copy link
Contributor

The advantage of the CDL files is that they are text and therefore more amenable to git tracking. But I don't have a strong opinion.

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