Skip to content

Add integration test for simple example #18

Open
ma595 wants to merge 3 commits intomainfrom
integration
Open

Add integration test for simple example #18
ma595 wants to merge 3 commits intomainfrom
integration

Conversation

@ma595
Copy link
Copy Markdown
Member

@ma595 ma595 commented Mar 4, 2026

Adds a simple integration test that produces a restart.nc state from various forecast .npy outputs.

Fixes #9

TODOs:

  • Fix the add_bottom_velocity function (separate PR then rebase)
  • Correct the data products to point to zenodo (not personal scratch)

@ma595 ma595 self-assigned this Mar 4, 2026
Automates the download of data for tests and the simplest example. It is
utilised in the CI and can be called manually by a user.

NOTE: the implementation will be updated when space is freed up in the
project directory on spirit, enabling https access on thredds.
ma595 added 2 commits March 6, 2026 16:27
This compares the generated NEW_DINO_00576000_restart.nc to the function
output of propagate_pred using numpy allclose with 1e-5 tolerance.

- add documentation in tests/README.md
- remove duplicate test in ci-restart.yml that always fails
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.

Integration test for simple restart execution

1 participant