Skip to content

Conversation

@rajichidamb
Copy link
Contributor

@rajichidamb rajichidamb commented Oct 1, 2025

Description

Surface pressure operator WRFDA scheme code was looking for virtualTemperature ObsValue in observation, but according to https://jointcenterforsatellitedataassimilation-jedi-docs.readthedocs-hosted.com/en/latest/inside/jedi-components/ufo/obsops.html#id65 observed airTemperature and specificHumidity can be used to calculate in the absence. The code is updated and to perform unit test new observation files without variables are created.

jedi-ci-test-select=gcc

Issue(s) addressed

Resolves https://github.com/JCSDA-internal/ufo/issues/3820

Dependencies

https://github.com/JCSDA-internal/ufo/pull/3830

Impact

Test the bug fix

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

…st without OBS virtualTemp and specificHumidity variable
@rajichidamb rajichidamb self-assigned this Oct 1, 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 Raji!

@rajichidamb rajichidamb added the coordinate merge Ready for merge but needs to be coordinated with other repos label Oct 13, 2025
@BenjaminRuston BenjaminRuston added the OBS OBS processing, UFO label Nov 11, 2025
Copy link
Collaborator

@BenjaminRuston BenjaminRuston left a comment

Choose a reason for hiding this comment

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

thanks @rajichidamb this is ready to move forward

@BenjaminRuston BenjaminRuston self-requested a review November 15, 2025 01:04
Copy link
Collaborator

@BenjaminRuston BenjaminRuston left a comment

Choose a reason for hiding this comment

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

thank you @rajichidamb file sizes are reasonable and tested with SpackStack v1.8 locally with gnu

@huishao-r huishao-r merged commit cc26e6f into develop Nov 24, 2025
@huishao-r huishao-r deleted the feature/sfcpressure_entanglement branch November 24, 2025 16:19
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants