Skip to content

Testing reduced Oceananigans script. DO NOT MERGE#2425

Merged
wsmoses merged 14 commits intomainfrom
jlk9/oceananigans-benchmark-reduced
Feb 13, 2026
Merged

Testing reduced Oceananigans script. DO NOT MERGE#2425
wsmoses merged 14 commits intomainfrom
jlk9/oceananigans-benchmark-reduced

Conversation

@jlk9
Copy link
Collaborator

@jlk9 jlk9 commented Feb 12, 2026

DO NOT MERGE. This is checking if reductions to the Oceananigans script in #2362 allow it to run with CI (particularly TPUs).

initial_salinity,
u_wind_stress,
v_wind_stress,
temp_flux
Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
temp_flux
temp_flux,

Duplicated(u_wind_stress, du_wind_stress),
Duplicated(v_wind_stress, dv_wind_stress),
Duplicated(temp_flux, dtemp_flux),
Duplicated(Δz, dΔz)
Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
Duplicated(Δz, dΔz)
Duplicated(Δz, dΔz),

@wsmoses wsmoses force-pushed the jlk9/oceananigans-benchmark-reduced branch from 9241ec4 to a8d5d8f Compare February 13, 2026 12:54
@wsmoses
Copy link
Member

wsmoses commented Feb 13, 2026

okay @jlk9 / @avik-pal I've fixed this. The tests still take forever so we should consider reducing in scope perhaps [with a flag?] also perhaps adding options for turning checkpointing/optimizations for AD/etc off. but nevertheless now is hopefully unstuck

@wsmoses wsmoses merged commit 025cf40 into main Feb 13, 2026
13 of 15 checks passed
@wsmoses wsmoses deleted the jlk9/oceananigans-benchmark-reduced branch February 13, 2026 15:07
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.

3 participants