Skip to content

Add test files for ExtCoeffProfCRTM operator - #574

Open
weiwilliam wants to merge 8 commits into
developfrom
feature/extcoeffprof
Open

Add test files for ExtCoeffProfCRTM operator#574
weiwilliam wants to merge 8 commits into
developfrom
feature/extcoeffprof

Conversation

@weiwilliam

Copy link
Copy Markdown
Contributor

Description

This PR adds the test files (IODA file with hofx group and geovals file) from CALIOP.

Issue(s) addressed

Resolves ufo#4163

Dependencies

List the other PRs that this PR is dependent on:

build-group=JCSDA-internal/ufo#4163

Impact

Expected impact on downstream repositories:

Checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have run the unit tests before creating the PR

@weiwilliam weiwilliam added needs review Asking others to review - often used for pull requests COMPO Atmospheric COMPOsition labels Jun 11, 2026
@BenjaminRuston

Copy link
Copy Markdown
Collaborator

@weiwilliam thanks,,, I reduced the files by doing compression and shuffling. No changes to precision or removal of fields was done I executed a command like so:

nccopy -d 9 -s -u caliop_l2_extcoeff.geovals.20190722_0217.nc4 caliop_l2_extcoeff.geovals.20190722_0217.shuffle.nc4

there was a more modest 15% reduction in the GeoVaLs file but greater than 80% reduction in the observation!

@BenjaminRuston
BenjaminRuston requested a review from chengdang July 20, 2026 18:53

@BenjaminRuston BenjaminRuston left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @weiwilliam for contributing the new unit test and files

@BenjaminRuston

Copy link
Copy Markdown
Collaborator

@rajichidamb , @chengdang or @jeromebarre could any of you have a look at this

I've compressed (and shuffled) the file to reduce their size and was able to do a pretty substantial reduction, but to merge the associated UFO PR4163 we'll need this too

@BenjaminRuston BenjaminRuston added coordinate merge Ready for merge but needs to be coordinated with other repos ready for merge PR is reviewed and is ready for merge and removed needs review Asking others to review - often used for pull requests labels Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

COMPO Atmospheric COMPOsition coordinate merge Ready for merge but needs to be coordinated with other repos ready for merge PR is reviewed and is ready for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants