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

using warning #1636

Closed
liasiegelman opened this issue May 4, 2021 · 15 comments
Closed

using warning #1636

liasiegelman opened this issue May 4, 2021 · 15 comments

Comments

@liasiegelman
Copy link

liasiegelman commented May 4, 2021

Hi, do you know how to fix the following ? Thanks
Screen Shot 2021-05-04 at 2 59 28 PM

@ali-ramadhan
Copy link
Member

Oh this is kinda weird. I don't think Oceananigans uses StaticArrays anymore.

What version of Oceananigans.jl and Julia are you using? You should be able to get this info from executing versioninfo() and using Pkg; Pkg.status() at the REPL.

@liasiegelman
Copy link
Author

Screen Shot 2021-05-04 at 3 26 02 PM

@ali-ramadhan
Copy link
Member

Ah Julia installed Oceananigans v0.5.1 (an extremely ancient version), probably due to conflicts between dependency versions. The latest release is Oceananigans v0.57.1.

To grab the latest version you can try to explicitly update by typing ] add Oceananigans@0.57.1 in the REPL.

There may be conflicts with some other packages like CUDA v3.1.0. Oceananigans is still depending on CUDA v2 so you might have to ] rm CUDA to be able to ] add Oceananigans@0.57.1.

This might also explain the weird error you reported in #1635.

PS: You might encounter issues when using Oceananigans.jl with Julia 1.6 right now. It still requires Julia 1.5 for now.

@liasiegelman
Copy link
Author

ah thanks but the all point is that I would like to test the GPU ...

@liasiegelman
Copy link
Author

I ll try with CUDA there and see what I get.

@liasiegelman
Copy link
Author

it works but gives me this warning
Screen Shot 2021-05-04 at 4 10 14 PM

@ali-ramadhan
Copy link
Member

ali-ramadhan commented May 4, 2021

When you ] add Oceananigans it will also add/install a compatible version of CUDA.jl so you can still use the GPU.

You only need to ] add CUDA if you want to use CUDA.jl directly.

@ali-ramadhan
Copy link
Member

Ah I wouldn't worry about the MPI warning for now.

@liasiegelman
Copy link
Author

ok cool thanks a lot !
So when trying running https://clima.github.io/OceananigansDocumentation/stable/generated/ocean_wind_mixing_and_convection/
I get this warning, seems ok no?
Screen Shot 2021-05-04 at 4 14 02 PM

@ali-ramadhan
Copy link
Member

That looks good! It's a mostly harmless warning about halo regions that the original grid is now different from model.grid.

Hopefully if you run the entire example you'll end up with a movie that looks like the one from the example.

@liasiegelman
Copy link
Author

Yay, thanks a lot Ali !

@francispoulin
Copy link
Collaborator

@ali-ramadhan and I created that warning message. Maybe it's too agressive and we should rethink it as people clearly become concerned by it? Will have to think about that a bit more but glad it's working!

@glwagner
Copy link
Member

glwagner commented May 5, 2021

I think the warning is a good idea. Probably a solution to alleviating user stress is to make halo=(3, 3, 3) the default, so that the warning is rarely invoked. #1245

@navidcy
Copy link
Collaborator

navidcy commented May 31, 2021

Is this resolved or should it stay open?

@francispoulin
Copy link
Collaborator

I'm okay with the warning message that we have and closing this.

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

No branches or pull requests

5 participants