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

[Feature Request] Dual Support for Binary and netCDF4 Coefficient Files #54

Closed
5 of 6 tasks
BenjaminTJohnson opened this issue Jul 25, 2023 · 1 comment
Closed
5 of 6 tasks
Assignees
Labels
CRTM CRTM enhancement New feature or request

Comments

@BenjaminTJohnson
Copy link
Contributor

BenjaminTJohnson commented Jul 25, 2023

Summary: As part of our gradual transition away from binary formats, we plan to implement the capability to read both binary format files and netCDF4 coefficient files in CRTM. The preference will be given to netCDF4 files when both types are available. This feature will ensure smooth transition and compatibility until all legacy coefficients are converted to netCDF4.

Details:

  1. Dual Support: CRTM will be enhanced to read both binary and netCDF4 coefficient files. This will allow for greater flexibility and easier transition for users.
  2. Preference: When both file types are available, CRTM will prioritize netCDF4 files over binary files.
  3. Transition Phase: This mixed file type capability will be a transitional phase until all binary files are converted to netCDF4. We will provide a clear timeline and support for this conversion process.

Plan:

  • Design and develop the capability to read both binary and netCDF4 coefficient files (essentially done, but needs verification that we can fully transition)
  • Implement a mechanism to prioritize netCDF4 files over binary format when both are available (this should be on a per request basis, check for nc4 first, then down to binary).
  • Thorough testing to ensure compatibility and correct prioritization.
  • Update documentation to guide users about the new feature and the transitioning process.
  • Release this feature in CRTM v3.1.0 (see CRTM v3.1.0 release preparation #53)
  • Work with NESDIS/STAR to start the conversion process for legacy coefficients.

Expected Release Date: TBD

We encourage everyone's input on this process. Please feel free to comment below with any feedback, suggestions, or concerns you may have regarding this planned feature. Thanks!

@BenjaminTJohnson
Copy link
Contributor Author

Legacy coefficient conversion will have to move to AOP24 as a "background" task.
Once the conversion tools are completed (i.e., supports antenna correction), then we'll start converting. Also need a verification tool that verifies that all elements are correct in the conversion. This will be a new issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CRTM CRTM enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants