Skip to content

BUG: pytables.BinOp.prune passing str to BinOp constructor #46077

Open
@jbrockmendel

Description

@jbrockmendel

computation.ops.BinOp docstring says that lhs and rhs should both be Term or Op objects. Enforcing this breaks a bunch of tests bc strings are passed to the constructors in pytables.BinOp.prune.

If we could actually rely on having lhs and rhs both be Term | Op, then we could rule out the ndarray case in BinOp.conform.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions