Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(0.95.6) Pin GPUArrays version + fix GPU testing pipeline #4037

Merged
merged 4 commits into from
Jan 11, 2025
Merged

Conversation

simone-silvestri
Copy link
Collaborator

@simone-silvestri simone-silvestri commented Jan 10, 2025

a lot of Oceananigans functionalities are broken at the moment because of a change in GPUArrays (see issue #4036)
This PR pins to a previous GPUArrays version to keep the repository (and the CI) functioning. Once the issue has been resolved we can remove this constraint.

@simone-silvestri simone-silvestri changed the title [0.95.6] Pin GPUArrays version (0.95.6) Pin GPUArrays version Jan 10, 2025
@navidcy navidcy self-requested a review January 10, 2025 19:23
@navidcy navidcy added GPU 👾 Where Oceananigans gets its powers from package 📦 Quite meta labels Jan 10, 2025
@glwagner
Copy link
Member

sverdrup is down right now so the GPU tests aren't running

@navidcy
Copy link
Collaborator

navidcy commented Jan 11, 2025

@navidcy navidcy added the testing 🧪 Tests get priority in case of emergency evacuation label Jan 11, 2025
@navidcy navidcy changed the title (0.95.6) Pin GPUArrays version (0.95.6) Pin GPUArrays version + fix GPU testing pipeline Jan 11, 2025
@navidcy navidcy merged commit e02564d into main Jan 11, 2025
46 checks passed
@navidcy navidcy deleted the ss/pin-cuda branch January 11, 2025 05:41
@navidcy
Copy link
Collaborator

navidcy commented Jan 11, 2025

Does this closes #4036?

navidcy added a commit to CliMA/ClimaOcean.jl that referenced this pull request Jan 12, 2025
trying out to see if scalar operations came with v0.95.7 and changes introduced by CliMA/Oceananigans.jl#4037
@simone-silvestri
Copy link
Collaborator Author

Does this closes #4036?

Not really, because we want to be able to use the new GPUArrays version. This is just patching the problem by pinning the GPUArrays version. I would keep the issue open until we can update CUDA and GPUArrays

navidcy added a commit to CliMA/ClimaOcean.jl that referenced this pull request Jan 20, 2025
)

* don't add / twice

* add missing deps + compats

* temporarily only leave the ECCO2 example on

* GLMakie -> CairoMakie; BuoyancyModels -> BuoyancyFormulations

* update

* code alignment

* add docs/Manifest.toml

* Update Project.toml

trying out to see if scalar operations came with v0.95.7 and changes introduced by CliMA/Oceananigans.jl#4037

* Update Project.toml

* homogenize compats
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GPU 👾 Where Oceananigans gets its powers from package 📦 Quite meta testing 🧪 Tests get priority in case of emergency evacuation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants