-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Stephan's sprintbattical #12
Conversation
Now O(n) instead of O(n^2), even though we do have to iterate through every element of an array in Python (gasp!).
Refactored the `broadcast_variables` function and fixed a bug in `variable.transpose` (see the new test case).
Added better test that uses string labels instead of just integers.
I think these names are much more straightforward. The only annoying aspect is that "array" is the name of a built-in module, which conflicts with naming the module "Array" is in "array".
We should probably remove Array.aggregate to reduce confusion, but for now I'll keep aggregate for error checks.
Looks like a great change! I'm seeing some failing tests: ~/dev/scidata (DataView)$ python setup.py test ERROR: test_aggregate (test.test_array.TestArray)Traceback (most recent call last): ERROR: test_groupby (test.test_array.TestArray)Traceback (most recent call last): ERROR: test_getitem (test.test_dataset.DataTest)Traceback (most recent call last): ERROR: test_labeled_by (test.test_dataset.DataTest)Traceback (most recent call last): ERROR: test_getitem (test.test_dataset.NetCDF4DataTest)Traceback (most recent call last): ERROR: test_labeled_by (test.test_dataset.NetCDF4DataTest)Traceback (most recent call last): ERROR: test_getitem (test.test_dataset.ScipyDataTest)Traceback (most recent call last): ERROR: test_labeled_by (test.test_dataset.ScipyDataTest)Traceback (most recent call last): ERROR: test (test.test_utils.TestNum2DatetimeIndex)Traceback (most recent call last): FAIL: test_dump_and_open_dataset (test.test_dataset.ScipyDataTest)Traceback (most recent call last): FAIL: test_aggregate (test.test_dataset_array.TestDatasetArray)Traceback (most recent call last): FAIL: test_groupby (test.test_dataset_array.TestDatasetArray)Traceback (most recent call last): FAIL: test_intersection (test.test_dataset_array.TestDatasetArray)Traceback (most recent call last): FAIL: test_labeled_by (test.test_dataset_array.TestDatasetArray)Traceback (most recent call last): FAIL: test_loc (test.test_dataset_array.TestDatasetArray)Traceback (most recent call last): Ran 100 tests in 1.248s FAILED (failures=6, errors=9, skipped=6) |
Thanks for @ebrevdo for taking a look! I'm pretty sure the issue here is the numpy version. I am running numpy 1.8 on my machine. It looks like we need at least numpy 1.7 for timedelta math [1], and it appears that [1] http://pandas.pydata.org/pandas-docs/stable/timeseries.html#numpy-1-7-compatibility |
We need this for our current use of np.array_equal.
Thanks for looking at that. I'll do a more thorough evaluation over the On Fri, Feb 14, 2014 at 5:49 PM, Stephan Hoyer notifications@github.comwrote:
|
Implements most of GitHub issue #13.
@@ -0,0 +1,572 @@ | |||
import functools |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As we just discussed lets rename this to xarray.py.
This is all great. I've been experimenting with this branch and the majority of it is running fine. Given that this project is still under heavy development and rather than bloating this pull request, lets go ahead and merge it into master and iterate on top of it. |
FYI -- I just pushed a commit renaming "Array" to "XArray" to master. There On Thu, Feb 20, 2014 at 4:36 PM, akleeman notifications@github.com wrote:
|
fix: relax instrumented store checks for v3
No description provided.