fixed LArray[:]

gdementen · gdementen · commit 63bcb95cbbf2 · 2015-01-28T12:12:50.000+01:00
diff --git a/larray/core.py b/larray/core.py
@@ -1054,7 +1054,8 @@ def broadcast_with(self, target):
     # deprecated since Python 2.0 but we need to define it to catch "simple"
     # slices (with integer bounds !) because ndarray is a "builtin" type
     def __getslice__(self, i, j):
-        return self[slice(i, j)]
+        # sadly LArray[:] translates to LArray.__getslice__(0, sys.maxsize)
+        return self[slice(i, j) if i != 0 or j != sys.maxsize else slice(None)]
 
     def __str__(self):
         if not self.ndim:
diff --git a/larray/tests/test_la.py b/larray/tests/test_la.py
@@ -593,6 +593,9 @@ def test_getitem(self):
         self._assert_equal_raw(la['1,5,9', lipro159],
                                raw[[1, 5, 9]][..., [0, 4, 8]])
 
+        # single None slice
+        self._assert_equal_raw(la[:], raw)
+
         # key with duplicate axes
         # la[[1, 5, 9], age['1,5,9']]
         self.assertRaises(ValueError, la.__getitem__, ([1, 5], age['1,5']))
