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 support for MiRS NOAA-21 ATMS products #665

Closed
kathys opened this issue Dec 6, 2023 · 1 comment · Fixed by pytroll/satpy#2677
Closed

Add support for MiRS NOAA-21 ATMS products #665

kathys opened this issue Dec 6, 2023 · 1 comment · Fixed by pytroll/satpy#2677
Assignees

Comments

@kathys
Copy link
Collaborator

kathys commented Dec 6, 2023

We have a new version of the MiRS software that is about to be released as part of CSPP LEO. It adds support for NOAA-21 ATMS. The new software is v11r9. P2G v3.0 creates good SNPP and NOAA-20 MiRS images from this new output, but it fails when trying to create NOAA-21 images.

I know we do not have any limb corrections yet for the NOAA-21 ATMS individual bands, but P2G v3.0 also fails when trying to produce non-brightness temperature images too, such as rain_rate, sea_ice and tpw.

Sample data bumi:/data/users/kathys/test_data/mirs/2023_examples

Log files:
bumi:/data/users/kathys/test_data/mirs/noaa21/mirs_noaa21_band_geotiff_20231009_175328.log
bumi:/data/users/kathys/test_data/mirs/noaa21/mirs_noaa21_product_geotiff_20231009_175328.log

@kathys kathys assigned kathys and djhoese and unassigned kathys Dec 6, 2023
@kathys kathys added this to the Polar2Grid 3.1 milestone Dec 6, 2023
@djhoese
Copy link
Member

djhoese commented Dec 7, 2023

It should be a "simple" update of these two python dictionaries:

https://github.com/pytroll/satpy/blob/fbe017c55c207bfba73a7d4b10491230f8edcaf4/satpy/readers/mirs.py#L50-L80

Looks like I'll need to double check consistency though as it looks like I have entries to handle filenames having "jpss" in them as well as "n20" and "npp". I'm surprised it isn't "jpssN" for 1, 2, etc.

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

Successfully merging a pull request may close this issue.

2 participants