Skip to content

Conversation

@simone-silvestri
Copy link
Collaborator

@simone-silvestri simone-silvestri commented Apr 3, 2025

A PR that will introduce the pieces of an OMIP prototype. requires #437 and #453

@navidcy
Copy link
Member

navidcy commented Aug 21, 2025

@simone-silvestri, with the two omip-branch-2 branches for Oceananigans and ClimaSeaIce, still seems that the speedup is not apparent? See https://clima.github.io/ClimaOceanDocumentation/previews/PR456/literated/one_degree_simulation/

@simone-silvestri
Copy link
Collaborator Author

Weird, from my calculations, it should be about 10 times faster, roughly. Maybe it did not load the correct branches?

@navidcy
Copy link
Member

navidcy commented Aug 21, 2025

Do you think that the -O0 matters? (see

- "$TARTARUS_HOME/julia-$JULIA_VERSION/bin/julia --color=yes -O0 --project=docs/ docs/make.jl"
)

Comment on lines +9 to +22
using Pkg
Pkg.add(PackageSpec(
name = "Oceananigans",
url = "https://github.com/CliMA/Oceananigans.jl",
rev = "ss/omip-branch-2"
))

Pkg.add(PackageSpec(
name = "ClimaSeaIce",
url = "https://github.com/CliMA/ClimaSeaIce.jl",
rev = "ss/omip-branch-2"
))
Pkg.resolve()
@show Pkg.status()
Copy link
Member

Choose a reason for hiding this comment

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

we don't need these right?

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

Labels

build docs Add this label to built the docs in a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants