Skip to content

Add new testfiles for SetSurfaceType AAPP tests#505

Merged
mikecooke77 merged 9 commits intodevelopfrom
feature/setsurface_aappsg
Nov 11, 2025
Merged

Add new testfiles for SetSurfaceType AAPP tests#505
mikecooke77 merged 9 commits intodevelopfrom
feature/setsurface_aappsg

Conversation

@brettcandy
Copy link
Contributor

@brettcandy brettcandy commented Sep 12, 2025

Description

This PR adds a new obs file and associated geovals to simulate a couple of swaths of MWS data. It is used for the new ufo ctest ufo_test_tier1_test_ufo_function_setsurfacetype_aappsg

Issue(s) addressed

Required for ufo PR below

Dependencies

List the other PRs that this PR is dependent on:

Impact

None expected

Checklist

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

@brettcandy brettcandy self-assigned this Sep 12, 2025
@brettcandy brettcandy added the OBS OBS processing, UFO label Sep 12, 2025
@BenjaminRuston BenjaminRuston added the needs review Asking others to review - often used for pull requests label Nov 10, 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 Brett!

@rajichidamb rajichidamb 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 Nov 10, 2025
@mikecooke77 mikecooke77 merged commit b3f57a3 into develop Nov 11, 2025
3 checks passed
@mikecooke77 mikecooke77 deleted the feature/setsurface_aappsg branch November 11, 2025 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

coordinate merge Ready for merge but needs to be coordinated with other repos OBS OBS processing, UFO 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