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

Phase 2 validation: sonde (adpupa 120/220) #245

Merged
merged 3 commits into from
Dec 13, 2024

Conversation

delippi
Copy link
Collaborator

@delippi delippi commented Dec 4, 2024

Description

Finished phase 2 (FV3-JEDI vs GSI) testing of sonde (adpupa 120/220). Most of each yaml was inherited from the mesonet yamls. The main difference was using the ObsErrorFactorConventional which also required the use of pre-sorting via obsgrouping in the obs space. All tests were shown to run successfully using the online domain check. Phase 2 doesn't guarantee each yaml will fully work with MPAS-JEDI (due to certain features not being ready in MPAS-JEDI and not due to issues with the yamls themselves).

I also want to note that in GSI I used ext_sonde=.false. and time_window_max=0.99. We should not be using the ext_sonde in GSI which has since been fixed in the latest ctest GSI case. I'm not sure why time_window_max=1.0 still gave more obs than expected. The correct setting should be time_window_max=1.5, but some more work is needed to extend the assimilation window for JEDI. This difference in settings should not affect results.

Issue(s) addressed

Resolves/Results are documented in Issue

Checklist

  • I have performed a self-review of my own code.

@guoqing-noaa
Copy link
Collaborator

@delippi Regarding due to certain features not being ready in MPAS-JEDI, could you list what features are missed? Do we have issues created for those? Thanks!

@delippi
Copy link
Collaborator Author

delippi commented Dec 4, 2024

@delippi Regarding due to certain features not being ready in MPAS-JEDI, could you list what features are missed? Do we have issues created for those? Thanks!

It has been a little while since I last tried running MPAS-JEDI and those could have already been fixed. The one that comes to mind was issues with specificHumidity when it tries to create the GeoVaLs/saturation_specific_humidity (I think it is called something else now). I haven't opened an issue with this yet since I haven't investigated too much and it could have been user error. My temporary solution was to just comment out those parts of the yaml. I have a feeling there was something else too, but I don't remember. Now might actually be a good time for me to start digging into some phase 3 tests and opening such issues.

@JingCheng-NOAA
Copy link

@delippi Nice job on updating the airTemperature, wind, and specificHumidity YAMLs. Will there be a plan to work on phase 2 of stationPressure for sondes data?

@delippi
Copy link
Collaborator Author

delippi commented Dec 4, 2024

@delippi Nice job on updating the airTemperature, wind, and specificHumidity YAMLs. Will there be a plan to work on phase 2 of stationPressure for sondes data?

Yes, I plan to. I'm not sure if that one needs would require using the python converter to create the IODA data or not. I need to look into it more. Have you tried sonde stationPressure yet?

@JingCheng-NOAA
Copy link

@delippi Nice job on updating the airTemperature, wind, and specificHumidity YAMLs. Will there be a plan to work on phase 2 of stationPressure for sondes data?

Yes, I plan to. I'm not sure if that one needs would require using the python converter to create the IODA data or not. I need to look into it more. Have you tried sonde stationPressure yet?

not with phase 2 yet.

@delippi
Copy link
Collaborator Author

delippi commented Dec 5, 2024

@delippi Regarding due to certain features not being ready in MPAS-JEDI, could you list what features are missed? Do we have issues created for those? Thanks!

It has been a little while since I last tried running MPAS-JEDI and those could have already been fixed. The one that comes to mind was issues with specificHumidity when it tries to create the GeoVaLs/saturation_specific_humidity (I think it is called something else now). I haven't opened an issue with this yet since I haven't investigated too much and it could have been user error. My temporary solution was to just comment out those parts of the yaml. I have a feeling there was something else too, but I don't remember. Now might actually be a good time for me to start digging into some phase 3 tests and opening such issues.

@guoqing-noaa, I tried all of these yamls in our MPAS-JEDI case. They all worked except specificHumidity. Here was the error message:

  0:  --> vars_nlevels: saturation_water_vapor_mixing_ratio_wrt_moist_air not available in MPAS domain or self % templated_fields

Is this because VADER is not in MPAS-JEDI? At least it doesn't seem to be. https://github.com/JCSDA-internal/mpas-jedi/issues/947
My temporary solution is to just comment the ObsErrorFactorPressureCheck.

@delippi
Copy link
Collaborator Author

delippi commented Dec 6, 2024

Are there any additional comments to be made on this PR? I've already got Phase 3 updates close to being ready for PR so I want to get this in soon, if possible.

Copy link
Contributor

@hu5970 hu5970 left a comment

Choose a reason for hiding this comment

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

looks good to me.

@hu5970 hu5970 merged commit 2968d5b into NOAA-EMC:develop Dec 13, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants