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

The gross check is missing for the q observations from the rawinsonde #431

Draft
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

gmao-wgu
Copy link
Contributor

Description

There is no gross check for the q observations from the rawinsonde

@gmao-wgu gmao-wgu added the bug Something isn't working label Sep 19, 2024
Copy link
Contributor

@rtodling rtodling left a comment

Choose a reason for hiding this comment

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

I would expect this to come with an update to the benchmark numbers in ufo_tests.yaml ... I'd consider suspicious if the sondes numbers does change with this ... !?

@gmao-wgu
Copy link
Contributor Author

It's odd that we were able to get matching numbers between GSI and JEDI without the gross check included. @asewnath what is your take on this?

@gmao-wgu
Copy link
Contributor Author

I am still looking into it.

@Dooruk
Copy link
Collaborator

Dooruk commented Sep 25, 2024

I ran a Tier1 test with this branch and ufo is indeed failing:
https://github.com/GEOS-ESM/swell/actions/runs/10947010213

Looks like it's a minor diff though, @gmao-wgu if you want to change the value to 116684:
FAILED passed == passedBenchmark evaluated as [116684] == [116687]

sondes:
filter_test:
passedBenchmark: 116687

@asewnath
Copy link
Contributor

I ran a Tier1 test with this branch and ufo is indeed failing: https://github.com/GEOS-ESM/swell/actions/runs/10947010213

Looks like it's a minor diff though, @gmao-wgu if you want to change the value to 116684: FAILED passed == passedBenchmark evaluated as [116684] == [116687]

sondes:
filter_test:
passedBenchmark: 116687

Thanks for checking this @Dooruk!

@gmao-wgu
Copy link
Contributor Author

gmao-wgu commented Oct 1, 2024

This is still incomplete. I have not figured out how to set the lower and upper bounds for the q observation errors.

@Dooruk Dooruk marked this pull request as draft October 1, 2024 15:56
@Dooruk
Copy link
Collaborator

Dooruk commented Oct 1, 2024

This is still incomplete. I have not figured out how to set the lower and upper bounds for the q observation errors.

Sounds good. I thought the UFO value was the final change required.

Copy link
Contributor

@rtodling rtodling left a comment

Choose a reason for hiding this comment

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

I'm fine w/ this.

@Dooruk
Copy link
Collaborator

Dooruk commented Oct 24, 2024

@gmao-wgu is this ready to go in? needs to be merged with develop

@gmao-wgu
Copy link
Contributor Author

@Dooruk It is still a work in progress because the upper and lower bounds of the observation error are missing in the gross check.

@Dooruk
Copy link
Collaborator

Dooruk commented Oct 24, 2024

@Dooruk It is still a work in progress because the upper and lower bounds of the observation error are missing in the gross check.

Sounds good, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants