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

VIIRS L1B DNB_SENZ file_key #2790

Closed
wjsharpe opened this issue Apr 23, 2024 · 0 comments · Fixed by #2791
Closed

VIIRS L1B DNB_SENZ file_key #2790

wjsharpe opened this issue Apr 23, 2024 · 0 comments · Fixed by #2791

Comments

@wjsharpe
Copy link
Contributor

Describe the bug
The DNB_SENZ dataset in the viirs_l1b reader uses 'geolocation_data/solar_zenith' as the file_key when it should be using 'geolocation_data/sensor_zenith'

Environment Info:

  • Readers and writers dependencies (when relevant): viirs_l1b

Additional context
DNB_SENZ has name as 'dnb_satellite_zenith_angle' which leads me to believe the file_key should be sensor zenith not solar zenith. Additionally DNB_SZA is already pointing at 'geolocation_data/solar_zenith' for the same file type so as it is currently setup DNB_SENZ and DNB_SZA are duplicates.

@wjsharpe wjsharpe changed the title Fixed VIIRS L1B DNB_SENZ file_key VIIRS L1B DNB_SENZ file_key Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant