-
Notifications
You must be signed in to change notification settings - Fork 11
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
Failing test_calc_basic tests after updating xarray from 10.2 to 10.3 #268
Comments
Basically in the failing tests the loaded dataset's time bounds array is not indexed by the
This is probably related to the fact that the time array is length-1: all of the failures are for the Looking at the xarray 10.3 what's new, here's a potential culprit: pydata/xarray#2048 |
I have to run for now, but it's something in our preprocess func that's causing the problem:
|
Upon looking at things more closely, this is failing in Lines 177 to 178 in f240c72
I think times.ensure_time_as_dim was written before expand_dims existed in Xarray, so the logic there might be overdue for a clean up. Could that be where things are going wrong? Maybe focus your attention there in #269?
|
Originally noted here
All of the failures are the same error. Note that this one doesn't involve regional averaging, so it's unrelated to what I'm implementing in #266. Here's the traceback of one:
The text was updated successfully, but these errors were encountered: