search

yeoshuheng / pe

0 stars 0 forks source link

Weight input. #10

Open yeoshuheng opened 2 months ago

yeoshuheng commented 2 months ago

I expect the current weight to be changed to somethign very small with the following command:

edit 1 w/0.000000000000000000000000000000000000000000000000000000000000000000000000000000001

However, this does not edit the weight, but instead treats it as 0, causing a removal of the weight instead. Whilst this is a fringe scenario, I'm thinking that it is due to the value being rounded as 0, causing the app to delete the weight instead. It might be good to consider the pre-rounded value of the weight to decide how to treat the weight input in your future enhancements. :)

nus-pe-script commented 2 months ago

Team's Response

image.png

This particular behaviour has already been specified in the UG and it is working as specified.

image.png

Additionally, we think this falls under extreme user behaviour.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: "particular behaviour has already been specified in the UG and it is working as specified."

I disagree with this, the specified UG behavior does not consider rounding in effect, but rather just what 0 does. However, I understand that this is indeed extreme behaviour, but given that it can be fixed easily, I think it should still be considered by the team, and I urge for a NIS instead.