Skip to content

Test difference between distributed and serial outputs for a prescribed atmosphere OceanSeaIceModel#727

Draft
taimoorsohail wants to merge 19 commits intomainfrom
ts/test-for-atmosphere-halo-filling
Draft

Test difference between distributed and serial outputs for a prescribed atmosphere OceanSeaIceModel#727
taimoorsohail wants to merge 19 commits intomainfrom
ts/test-for-atmosphere-halo-filling

Conversation

@taimoorsohail
Copy link
Collaborator

This PR adds a test for the prescribed-atmosphere-forced OSIM to ensure the serial and distributed runs have exactly the same output.

… atmospheric forced simulation to chack for discrepancies
@taimoorsohail taimoorsohail self-assigned this Jan 14, 2026
@taimoorsohail taimoorsohail added enhancement New feature or request tests Helpful for getting a good night's sleep labels Jan 14, 2026
@navidcy navidcy added the atmosphere-ocean coupling ⛅️ 🌊 When the relationship isn't as smooth as expected label Jan 14, 2026
@navidcy navidcy changed the title Utility to check difference between distributed and serial outputs for a prescribed atmosphere OceanSeaIceModel Test difference between distributed and serial outputs for a prescribed atmosphere OceanSeaIceModel Jan 14, 2026
@taimoorsohail
Copy link
Collaborator Author

The cluster keeps running out of memory - I have tried to reduce the problem size as much as possible but no luck. Any thoughts?

@navidcy
Copy link
Member

navidcy commented Jan 15, 2026

We need to add the @root statements? @simone-silvestri could you push those changes?

@taimoorsohail
Copy link
Collaborator Author

taimoorsohail commented Jan 15, 2026

Tests are failing with

ERROR: The following 1 direct dependency failed to precompile:
ReactantCUDAExt --code-coverage=none --color=yes --check-bounds=yes --warn-overwrite=yes --depwarn=yes --inline=yes --startup-file=no --track-allocation=none

Copy link
Collaborator

@simone-silvestri simone-silvestri left a comment

Choose a reason for hiding this comment

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

Good addition if it passes!

@navidcy
Copy link
Member

navidcy commented Jan 22, 2026

@simone-silvestri could you have a look here and resolve the conflicts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

atmosphere-ocean coupling ⛅️ 🌊 When the relationship isn't as smooth as expected enhancement New feature or request tests Helpful for getting a good night's sleep

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants