Skip to content

Commit ff53849

Browse files
committed
Miniexpr now correctly detects that it does not support nested mixed-types
1 parent 4f5a969 commit ff53849

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

src/blosc2/lazyexpr.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1293,12 +1293,6 @@ def fast_eval( # noqa: C901
12931293
use_miniexpr = False
12941294

12951295
if use_miniexpr:
1296-
op_dtypes = {op.dtype for op in operands.values() if isinstance(op, blosc2.NDArray)}
1297-
# This is for avoiding type casting issues in miniexpr, like in:
1298-
# tests/ndarray/test_lazyexpr_fields.py::test_where_fusion6[dtype_fixture0-shape_fixture0-chunks_blocks_fixture0]
1299-
# TODO: remove this restriction when miniexpr supports type casting better
1300-
if len(op_dtypes) > 1:
1301-
use_miniexpr = False
13021296
# Avoid padding issues except for 1D arrays (contiguous along the only axis).
13031297
if len(shape) != 1 and builtins.any(s % c != 0 for s, c in zip(shape, chunks, strict=True)):
13041298
use_miniexpr = False

0 commit comments

Comments
 (0)