-
Notifications
You must be signed in to change notification settings - Fork 2
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
Can we remove the space from InterpolationsRegridder
?
#113
Comments
Yes, I think that should be possible. The only place where it is not easy is where we determine the extrapolation boundary conditions automatically (at the constructor): ClimaUtilities.jl/ext/InterpolationsRegridderExt.jl Lines 32 to 73 in 5171f8c
This would change a little bit the spirit of the regridder (the initial idea was to have a regridder attached to a specific variable and calling it multitple times would be cheap), but I don't see big problems with this. |
I'm seeing huge stack traces in https://buildkite.com/clima/climaland-long-runs/builds/1330#01925384-a92c-4f92-885e-69fe03731fc1. I think it's because there are a lot (~23) of
InterpolationsRegridder
objects (and therefore spaces) in a DataLayouts kernel, and this results in a massive method signatureThe text was updated successfully, but these errors were encountered: