Repository navigation
Update TestReference for ObsErrorFactorConventional PR - #600
Conversation
…r fix Regenerated the reference error factors used by ufo_function_obserrorfactorconv with the fix in ufo bugfix/convLayers (ObsErrorFactorConventional now uses the model layer containing the observation, as GSI errormod). Two values change, both at the lowest VAD levels and both decrease: 982 hPa 1.08110 -> 1.07982, 968 hPa 1.07835 -> 1.07834. windNorthward and all other variables are unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
Copilot review overview
🔵 Needs a closer look
The binary NetCDF changes cannot be inspected from the LFS pointer, and the referenced upstream PR was inaccessible.
Review effort: Balanced
Findings: None
What changed in this PR
Updates a VAD wind observation reference dataset following upstream error-factor changes.
Changes:
- Replaces the Git LFS object for the NetCDF test fixture.
- Preserves the file size at 111,912 bytes.
| File | Description |
|---|---|
testinput_tier_1/converr_vadwind_obs_2020120112_s.nc4 |
Updates the reference dataset’s LFS pointer. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
think we need to add the build group here for the test to succeed, will do that still getting |
I'm afraid I don't understand this comment. Are you saying that it's failing after you added a build-group? This PR and the have build-group, but maybe I did it wrong? |
BenjaminRuston
left a comment
There was a problem hiding this comment.
thanks for the update @ClaraDraper-NOAA
Update reference file for one test, after changed in:
build-group=https://github.com/JCSDA-internal/ufo/pull/4391
Two of 36 values change in one test changed. both failures are at low VAD levels, and both get smaller, which is exactly as as expected.
Checklist