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

Merge the clear-sky ABI gsrcsr python version bufr2ioda python file a… #235

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

xyzemc
Copy link
Contributor

@xyzemc xyzemc commented Nov 25, 2024

This PR is addressing issure #221 to convert the clear sky GOES ABI radiance bufr data to ioda.
Two files have been added to RDASApp/rrfs_test/IODA/python.

…nd jason file.

These two files are based on that from GDASApp.
@SamuelDegelia-NOAA
Copy link
Contributor

@xyzemc Thanks for adding the new converter. Would you like to stage the ABI IODA files in RDAS_DATA?

@xyzemc
Copy link
Contributor Author

xyzemc commented Nov 26, 2024

@xyzemc Thanks for adding the new converter. Would you like to stage the ABI IODA files in RDAS_DATA?

Yes. Thanks for reminding me.
Please copy the following data from the directory: /scratch2/NCEPDEV/fv3-cam/Xiaoyan.Zhang/noscrub/JEDI/RDASApp_1107/expr/mpas_2024052700/data/obs

abi_g16.satbias.nc4      abi_g16.tlapse.txt   abi_g18.satbias_cov.nc4  ioda_abi_g16.nc@
abi_g16.satbias_cov.nc4  abi_g18.satbias.nc4  abi_g18.tlapse.txt       ioda_abi_g18.nc@

@SamuelDegelia-NOAA
Copy link
Contributor

@xyzemc Thanks for adding the new converter. Would you like to stage the ABI IODA files in RDAS_DATA?

Yes. Thanks for reminding me. Please copy the following data from the directory: /scratch2/NCEPDEV/fv3-cam/Xiaoyan.Zhang/noscrub/JEDI/RDASApp_1107/expr/mpas_2024052700/data/obs

abi_g16.satbias.nc4      abi_g16.tlapse.txt   abi_g18.satbias_cov.nc4  ioda_abi_g16.nc@
abi_g16.satbias_cov.nc4  abi_g18.satbias.nc4  abi_g18.tlapse.txt       ioda_abi_g18.nc@

Will do! Do you plan to include channel 17?

@xyzemc
Copy link
Contributor Author

xyzemc commented Nov 26, 2024

@xyzemc Thanks for adding the new converter. Would you like to stage the ABI IODA files in RDAS_DATA?

Yes. Thanks for reminding me. Please copy the following data from the directory: /scratch2/NCEPDEV/fv3-cam/Xiaoyan.Zhang/noscrub/JEDI/RDASApp_1107/expr/mpas_2024052700/data/obs

abi_g16.satbias.nc4      abi_g16.tlapse.txt   abi_g18.satbias_cov.nc4  ioda_abi_g16.nc@
abi_g16.satbias_cov.nc4  abi_g18.satbias.nc4  abi_g18.tlapse.txt       ioda_abi_g18.nc@

Will do! Do you plan to include channel 17?

Do you mean GOES_17?

@SamuelDegelia-NOAA
Copy link
Contributor

Oops, I see those are satellites now. I first read them and thought the files were split by channel. You can ignore this!

@SamuelDegelia-NOAA
Copy link
Contributor

The data have been staged on all supported machines.

Just out of my curiosity, are the ABI data provided with a parallax correction applied? Or is that something that we can apply through JEDI filters?

@xyzemc
Copy link
Contributor Author

xyzemc commented Nov 26, 2024

The data have been staged on all supported machines.

Just out of my curiosity, are the ABI data provided with a parallax correction applied? Or is that something that we can apply through JEDI filters?

Please remind me what the parallax correction is. The abi data we used in operational is the 16x16 averaged products (CSR), instead of the original pixel data.

@SamuelDegelia-NOAA
Copy link
Contributor

The parallax correction adjusts for location errors due to tall clouds. See here. Since this PR is for clear-sky ABI it should not be relevant, but once we get to all-sky assimilation then we might need to think about it if the data are not already corrected.

@xyzemc
Copy link
Contributor Author

xyzemc commented Nov 27, 2024

The parallax correction adjusts for location errors due to tall clouds. See here. Since this PR is for clear-sky ABI it should not be relevant, but once we get to all-sky assimilation then we might need to think about it if the data are not already corrected.

That's a good point. Let's think about it when we implement all-sky in the near future.

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 this pull request may close these issues.

2 participants