Skip to content

Conversation

@weiwilliam
Copy link
Contributor

@weiwilliam weiwilliam commented Feb 19, 2025

Description

This PR adds geoval and observation files for the new ufo tests ufo_test_tier1_test_ufo_opr_crtm_vis_albedo_aerosol and ufo_test_tier1_test_ufo_linopr_crtm_vis_albedo_aerosol.

this is associated with UFO PR: https://github.com/JCSDA-internal/ufo/pull/3614

Issue(s) addressed

Resolves #<issue_number>

Dependencies

List the other PRs that this PR is dependent on:

  • [ ]

Impact

N/A

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 self-assigned this Feb 19, 2025
@weiwilliam weiwilliam added the COMPO Atmospheric COMPOsition label Mar 4, 2025
Copy link
Collaborator

@fcvdb fcvdb left a comment

Choose a reason for hiding this comment

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

File testinput_tier_1/viirs_j1_albedo-small.geovals.2021082315.nc4 is 2.&Mb. Could you reduce its size to less than 1Mb?

@weiwilliam
Copy link
Contributor Author

File testinput_tier_1/viirs_j1_albedo-small.geovals.2021082315.nc4 is 2.&Mb. Could you reduce its size to less than 1Mb?

@fcvdb No problem, it is 10 times smaller.

Copy link
Collaborator

@fcvdb fcvdb left a comment

Choose a reason for hiding this comment

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

Thanks Shih-Wei!

@fcvdb fcvdb added the coordinate merge Ready for merge but needs to be coordinated with other repos label Mar 25, 2025
Copy link
Collaborator

@fcvdb fcvdb left a comment

Choose a reason for hiding this comment

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

Thanks Shih-Wei!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants