Skip to content

add ice thickness test data#229

Merged
srherbener merged 4 commits intodevelopfrom
feature/ODB-read-iceThickness-iceFreeboard
Feb 6, 2026
Merged

add ice thickness test data#229
srherbener merged 4 commits intodevelopfrom
feature/ODB-read-iceThickness-iceFreeboard

Conversation

@twsearle
Copy link
Copy Markdown
Contributor

@twsearle twsearle commented Jan 5, 2026

Description

Add test data for reading ice thickness observations from ODB files.

Issue(s) addressed

following on from https://github.com/JCSDA-internal/ufo/issues/3770 as part of operational implementation of ice thickness at the Met Office.

Dependencies

build-group=https://github.com/JCSDA-internal/ioda/pull/1601

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

@twsearle twsearle self-assigned this Jan 5, 2026
@twsearle twsearle requested a review from DJDavies2 January 5, 2026 17:39
Copy link
Copy Markdown
Contributor

@DJDavies2 DJDavies2 left a comment

Choose a reason for hiding this comment

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

See comment in OPS ticket. In terms of getting it merged, many assigning someone like @srherbener as revewer?

@twsearle twsearle force-pushed the feature/ODB-read-iceThickness-iceFreeboard branch from 7c7cc16 to c7431e6 Compare January 6, 2026 14:37
@twsearle twsearle force-pushed the feature/ODB-read-iceThickness-iceFreeboard branch from c7431e6 to ec973f8 Compare January 7, 2026 13:06
@DJDavies2
Copy link
Copy Markdown
Contributor

An ioda test is currently failing with this:

test_ioda-convert_stationsnow_statid_odc

This is with ioda 1601.

@twsearle
Copy link
Copy Markdown
Contributor Author

twsearle commented Feb 5, 2026

An ioda test is currently failing with this:

test_ioda-convert_stationsnow_statid_odc

This is with ioda 1601.

should be fixed now! thanks

Copy link
Copy Markdown
Contributor

@mo-RowanMcL mo-RowanMcL 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, and thanks for fixing the reference data for stationsnow.

@srherbener srherbener merged commit c652e6a into develop Feb 6, 2026
@srherbener srherbener deleted the feature/ODB-read-iceThickness-iceFreeboard branch February 6, 2026 03:36
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.

4 participants