fix _replaced_order_element_ids to avoid None returns

Author: ernestoarbitrio

src/cr/cube/dimension.py

@@ -63,7 +63,7 @@ class Dimensions(tuple):
     """Collection containing every dimension defined in cube response."""
 
     def __repr__(self) -> str:
-        "str representation of this dimension sequence." ""
+        """str representation of this dimension sequence."""
         dim_lines = "\n".join(str(d) for d in self)
         return f"Dimensions:\n{dim_lines}"
 
@@ -875,7 +875,12 @@ def _replaced_order_element_ids(self, element_ids) -> List[Optional[str]]:
         The explicit order transform includes a list of ids that can be specified in
         many different ways, this translate them to the subvariable aliases.
         """
-        return [self.translate_element_id(_id) for _id in element_ids]
+        return list(
+            filter(
+                lambda v: v is not None,
+                [self.translate_element_id(_id) for _id in element_ids],
+            )
+        )
 
     @lazyproperty
     def _subvar_aliases(self) -> Tuple[str, ...]:

tests/integration/test_cubepart.py

@@ -22,6 +22,31 @@
 class Test_Slice:
     """Integration-test suite for _Slice object."""
 
+    def test_it_provides_values_for_cat_x_mr_x_mr_with_odd_order(self):
+        transf = {
+            "columns_dimension": {
+                "elements": {
+                    "2": {"hide": True},
+                    "3": {"hide": True},
+                    "4": {"hide": True},
+                    "foo": {"hide": True},
+                    "bar": {"hide": True},
+                },
+                "order": {
+                    "element_ids": [1, 2, 3, 4, 6, "foo", "bar"],
+                    "type": "explicit",
+                },
+            },
+        }
+        parts = Cube(CR.CAT_X_MR_X_MR, transforms=transf, population=9001).partitions
+        assert len(parts) == 2
+        assert parts[0].column_proportions == pytest.approx(
+            np.array([[0.60553814], [0.10292581], [0.10031348]])
+        )
+        assert parts[1].column_proportions == pytest.approx(
+            np.array([[0.08141321], [0.60522273], [0.5847414]])
+        )
+
     def test_it_provides_values_for_cat_x_cat(self):
         slice_ = Cube(CR.CAT_X_CAT, population=9001).partitions[0]