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

Expand NetCDF test data (with groups/various levels) #14

Open
srizvi-NASA opened this issue Apr 2, 2024 · 2 comments · May be fixed by #20
Open

Expand NetCDF test data (with groups/various levels) #14

srizvi-NASA opened this issue Apr 2, 2024 · 2 comments · May be fixed by #20
Labels
enhancement New feature or request

Comments

@srizvi-NASA
Copy link

The current test data for converting netcdf to cog seems limited. To ensure thorough testing, I would like to request adding more netcdf files with the following variations:

  1. Files with and without groups.
  2. Files with different data levels (e.g., TEMPO granules from L2 or L3).

Including a wider variety of data will help identify and address potential issues during the conversion process.

@frankinspace frankinspace added the enhancement New feature or request label Apr 3, 2024
@frankinspace
Copy link
Member

Only concern here is the size of the test data files; we can't use git-lfs (technically we can but the free space we are given is shared across the entire podaac/ org and not very large, and there's too much red tape involved trying to purchase more LFS storage)

@frankinspace
Copy link
Member

Another thought I've had is to setup a pytest fixture that could download granule CMR based on concept-id prior to test and then remove it after test

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
Development

Successfully merging a pull request may close this issue.

2 participants