-
Notifications
You must be signed in to change notification settings - Fork 13
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
problem with one_degree_near_global_simulation.jl #87
Comments
What code are you running? @simone-silvestri is this resolved by #80 ? |
That error refers to the fact that the previous bathymetry data stored in the On the other hand, that data is not necessary anymore. In #80 you can build a bathymetry with bottom_height = regrid_bathymetry(grid, connected_regions_allowed = 0) I suggest to also add the keyword argument |
yeah and whatever code that is being ran, should be deleted or fixed |
This is a little cryptic. Can you make it so that the wrong choice throws an error? I don't understand what needs to be done. |
This is the same issue I ran into a while back when running GlobalOceanBioME with one_degree_near_global_simulation. I have the missing bathymetry files on my local machine if you want them temporarily @francispoulin, but likely makes more sense to update to #80 as suggested rather than run the old code. |
For the moment a warning in the bathymetry right suffice I'll add it to #88
I think it would be a good moment to update |
Yes you can try that! We are working on validating CATKE for realistic global stuff. The latest Oceananigans also has a two-equation closure that can be tested. One missing piece I think is a near global configuration with restoring? |
I opened a draft proposal for the restoring following the discussion in issue #81. |
Thanks everyone for the replies! @vtamsitt , that is very kind of you to offer but I don't need it right away. If the update includes bathymetry and winds, that would definitely solve my problem. I was doing this today to prepare for a talk on Monday. I was fortunate to have a plot of topography from a while back. Sadly, I never plotted the winds at any time. I am very excited to run this new code as I have been learning about OceanBioME recently and it'll be great to see it used in a global setting! |
I guess we can return to this after #81 gets resolved. I'll keep it open in the mean time, unless peopel prefer that I close it. |
I pointed this out on Oceananigans.jl, but when I try running this script I get an error when I try and access the data. Any idea how to resolve this error?
The text was updated successfully, but these errors were encountered: