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

Create new reader for CSPP VIIRS Vegetation Indices #617

Closed
kathys opened this issue Jun 13, 2023 · 6 comments · Fixed by #626
Closed

Create new reader for CSPP VIIRS Vegetation Indices #617

kathys opened this issue Jun 13, 2023 · 6 comments · Fixed by #626
Assignees
Labels

Comments

@kathys
Copy link
Collaborator

kathys commented Jun 13, 2023

Request the addition of a reader fore the CSPP VIIRS Vegetation Index products which are parameters that are part of the CSPP Land Surface Reflectance NetCDF product files. There are two different arrays we would like to support:

NDVI (Normalized Difference Vegetation Index)
EVI (Ehanced Vegetation Index)

Again, Nick did create readers for these two arrays as a plug-in for Polar2Grid Version 2.3. But it never really worked correctly.

There is a standard color table that is used for the Vegetation Index products, and I would like it to be applied by default. Nick and I worked hard on the reader because there are a bunch of Quality Flag bits that need to be used to screen out things like clouds, aerosols, snow/ice, etc. It is important for you to start with what Nick was working to save you from having to go through all of this again. It includes the application of the color table too.

Test product files for the Vegetation Index reader can be found here:

bumi:/data/users/kathys/test_data/viirs_edr/j01/lsr
bumi:/data/users/kathys/test_data/viirs_edr/npp/lsr

@kathys kathys added enhancement component:reader reading/loading data labels Jun 13, 2023
@kathys kathys added this to the Polar2Grid 3.1 milestone Jun 13, 2023
@djhoese
Copy link
Member

djhoese commented Jun 14, 2023

Where is Nick's existing code?

@kathys
Copy link
Collaborator Author

kathys commented Jun 14, 2023

I found the most recent tarball that Nick provided to me in May 2022:

bumi:/data/users/kathys/test_data/viirs_edr/polar2grid_v_2_3-viirs-lsr-patch-v0.6.tar.gz

@kathys
Copy link
Collaborator Author

kathys commented Jun 14, 2023

@djhoese
Copy link
Member

djhoese commented Jun 14, 2023

As discussed in the meeting today, I'll make two readers in Satpy. One for the surface reflectances (see #616) that only has the surface refl bands and not the optional NDVI-based products (even though they might be in the files). The other reader will be for the vegetation indices and will have two file types, one for veg index files and one for surface reflectance files with priority going to veg index files. That way if someone passed both types of files to the veg index reader, only the veg index files would be used.

@djhoese
Copy link
Member

djhoese commented Jul 19, 2023

An update on decisions:

There will be one reader called viirs_edr that will look for the surface reflectance files. These files will contain vegetation indexes iff they came from CSPP software. The NOAA processing only produces a gridded vegetation index product separately from the surface reflectance file. The initial version of this reader (as far as P2G funding goes) will not support these NOAA VI files. This means the reader will need to dynamically check if the VI variables are in the surface reflectance for them to be considered "available". That means one reader (viirs_edr), one file type for these variables, and 2 possibly unavailable VI products (NDVI, EVI).

@djhoese
Copy link
Member

djhoese commented Jul 22, 2023

For my future reference, here is a copy of the standard NDVI table that should be used:

https://gitlab.ssec.wisc.edu/cspp/cspp-viirs-lsr-polar2grid-extension/-/blob/master/src/colormaps/viirs_l2_lsr_ndvi.cmap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants