Fixes operands could not be broadcast error when the subscripts contain ellipsis and the operands are shapes.

[nova77]

Description

When the subscript contain ellipsis and operands are shape, we get the following error:

ValueError: operands could not be broadcast together with shapes (..)

This is due to the fact that the parser converts the operands into numpy arrays which have strict rules on comparison.

This PR addresses #235.

Todos

Notable points that this PR has either accomplished or will accomplish.

• enforce operand shapes to be list of lists
• adds a unittest to verify the fixed behavior

Status

• Ready to go

[dgasmith]

This is a subtle bug related to a blocking issue in #204, where possibly_convert_to_numpy's behavior is incorrect and we're not quite normalizing on internal representations the way we should. I'll add your test to #204 and see if I fix this in a more robust way for the next release.

[dgasmith]

Yes, I can confirm #204 fixes it: 602e61b

[nova77]

Wonderful. I'm happy to have it fixed either way.

[dgasmith]

Closed by #204.