search

nus-cs2113-AY2223S2 / pe-dev-response

0 stars 0 forks source link

`Matrix` Simple matrix operations with transpose on the first matrix do not work #1456

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Description

When the first matrix in the equation is transposed, no operations work as expected.

This limitation is not documented in the UG.

None of the operators appear to be parsed correctly for the first matrix and it will always throw an exception and say that the operator in expression is unknown.

As a user, I would expect that transposing the first matrix would work.

Screenshots & Reproduction

Element-wise operation failing

image.png

Matrix addition failing

image.png

[original: nus-cs2113-AY2223S2/pe-interim#1141] [original labels: type.FunctionalityBug severity.High]

0nandon commented 1 year ago

Team's Response

Issue accepted. Thanks for your bug finding. But I set this issue as very low since the equation you suggested to us should not be executed due to shape mismatch.

So, this issue is more about incorrect exception message which does not hinder user's usage.

Duplicate status (if any):

--