Fixed ValueGroup.__init__

gdementen · gdementen · commit ea91bb17e5cd · 2015-08-26T12:18:45.000+02:00
We cannot anymore check whether a key is valid via axis.translate because
that function is not valid in the case of sparse arrays (we cannot
translate each axis individually)

TODO: this should be replaced by something like "axis.is_valid(key)"
for simple keys this is just a matter of "key in axis"
diff --git a/larray/core.py b/larray/core.py
@@ -643,11 +643,19 @@ def __init__(self, key, name=None, axis=None):
         # impossible to know whether a name was explicitly given or computed
         self.name = name
 
-        if axis is not None:
+        # if axis is not None:
             # check the key is valid
             # TODO: for performance reasons, we should cache the result.
             # This will need to be invalidated correctly
-            axis.translate(key)
+
+            # we cannot do it via axis.translate anymore because that
+            # function is not valid in the case of sparse arrays (we
+            # cannot translate each axis individually)
+
+            # TODO: this should be replaced by something like
+            # axis.is_valid(key)
+            # for simple keys this is just a matter of "key in axis"
+            # axis.translate(key)
         self.axis = axis
 
     def __hash__(self):
