Skip to content

Add geovals for the SeaIceFreeboard operator test - #601

Open
guillaumevernieres wants to merge 1 commit into
developfrom
feature/seaicefreeboard
Open

guillaumevernieres wants to merge 1 commit into
developfrom
feature/seaicefreeboard

Conversation

@guillaumevernieres

@guillaumevernieres guillaumevernieres commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Description

Adds testinput_tier_1/cryosat2-2018-04-15_seaicefreeboard_geovals.nc, the geovals for the new SeaIceFreeboard operator tests in ufo.

The file has the 8065 locations of cryosat2-2018-04-15_geovals.nc, but with single-category (per unit ice area) sea_ice_thickness, sea_ice_snow_thickness and sea_ice_area_fraction. The existing file has 5 categories, all with the same values everywhere. The new fields are synthetic and vary smoothly with latitude and longitude:

  • ice thickness 1.6–3.8 m
  • snow thickness 0.16–0.42 m
  • ice fraction 0.53–0.96

A few locations have missing ice or snow thickness, to test the operator's missing-value handling.

Issue(s) addressed

Works toward jcsda-internal/ufo/issues/4388

Dependencies

None. The companion ufo PR (jcsda-internal/ufo/pull/<#>) depends on this one.

Impact

None. This is a new file and no existing data is modified.

Need for: https://github.com/JCSDA-internal/ufo/pull/4395

Checklist

  • I have performed a self-review of my own code
  • I have run the unit tests before creating the PR

@BenjaminRuston BenjaminRuston added the coordinated merge Needs to be coordinated with other repos label Oct 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

coordinated merge Needs to be coordinated with other repos

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants