search

weiseng18 / pe

0 stars 0 forks source link

find command chained usage #9

Open weiseng18 opened 3 days ago

weiseng18 commented 3 days ago

image.png

I think it is worth explaining clearly how find tut/physics tut/math n/Alex Goh might work. It is stated that for each prefix given, it does an AND. Then for the n/ prefix, it is stated that students matching at least one of the keywords will be shown.

As a CS student, I can understand this quickly to mean that the above command will find students enrolled in math and physics tutorials, and their name contains either Alex or Goh. But for the target audience it might be a bit confusing. Since the example command is already given there, it might be worth it to also explain exactly what it does, because it isn't exactly a trivial command.

nus-pe-script commented 9 hours ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

We have indicated that using chaining will be "more specific" in the tip instead of "more general" to indicate the narrowing of specificity instead of widening. This shows the AND logic of the command instead of the OR logic in more layman terms.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]