-
Notifications
You must be signed in to change notification settings - Fork 34
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
Interpolation METHOD="spline"
fails for TPXO8
#238
Comments
METHOD="spline"
fails for TPXO8
"linear" also seems to fail with a different error:
|
That's not good! I'll work on this today. |
…238 ci: update versions to prevent deprecations
Thanks @robbibt for finding this bug and pointing it out! Should be fixed in 2.0.8. Let me know if you have any more trouble! |
Amazing, thanks so much @tsutterley for the quick fix (as always). I'll test it out and let you know. |
It works nicely now on 2.0.8 - awesome! 🚀 A related TPXO8 question: for some of our other models, we've found we can get a really big performance boost by clipping our tide model NetCDF files to Australia before passing them to These aren't something I can easily clip, so I was wondering if |
Excellent!
p.s. the above definition "file" uses the |
Neat! That's working nicely for me - I had to make two small changes:
Am now able to model tides for all |
Thanks @robbibt! I'll make those change to my version of the definition file. I'm still thinking about the best way for users to enter these arguments, while still allowing for ease of use. The definition files are a bit of a kludgy solution. Awesome! That's super exciting! 🚀 |
Issue
I can easily model tides for TPXO8 using "bilinear" interpolation:
However, if I try the same code with "spline" interpolation (the default), this fails with
ValueError: x dimension of z must have same number of elements as x
:Other models work fine with "spline" interpolation, e.g. FES2014:
Expected outcome
All supported tide models can be run with the default "spline" interpolation method without raising errors.
Details
I'm using
pyTMD=2.0.7
andscipy=1.10.1
. Full error:The text was updated successfully, but these errors were encountered: