yyyaohhh / pe

0 stars 0 forks source link

wrong use of capitalisation in assign command #10

Open yyyaohhh opened 10 months ago

yyyaohhh commented 10 months ago

Since except for NAME, all parameters are case sensitive, assigning department here used cardiology but at the appendix used Cardiology, where c is in caps. So which standard should we follow? with caps or without caps?

image.png

image.png

nus-pe-script commented 9 months ago

Team's Response

Thanks for raising this issue!

However, we believe that we have stated clearly under the 'Features' section of our User Guide that all parameters except for name are case-insensitive (please see the screenshot below).

Screenshot 2023-11-20 at 7.26.51 PM.png

Therefore, both cardiology and Cardiology are accepted. We feel like it is unnecessary to provide all possible cases for the capitalization of department inputs in the table, which is why we only provided one example of acceptable input(s) and believe that it would not cause any ambiguity. As for the example, it was not capitalized precisely because we wanted to showcase case-insensitivity of the department input.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]