-
Notifications
You must be signed in to change notification settings - Fork 15
Rework OBC Mask #214
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
base: main
Are you sure you want to change the base?
Rework OBC Mask #214
Conversation
Thanks @manishvenu! |
thanks @helenmacdonald! |
…om6 into rework_bc_mask
Looks like it's not working for tides for me? But to add it for |
Details:
I've been taking a look at the OBC mask for our BGC stuff, and think I've come to an improvement/happy solution-> But it definetely needs to be tested on other people's computers!
Right now, our two big OBC generating functions,
regrid_velocity_tracers
®rid_tides
, pass their datasets through amask_dataset
function inregridding.py
. If bathymetry is provided (and it is not by default), the dataset is masked, if bathy isn't provided, all NaNs are filled with zeros to avoid the failures we were getting in CROCODILE-CESM#25.However, the problem(s) are:
The fix is that:
Changes:
mask_dataset
andget_boundary_mask
inregridding.py
according to the description above.