Open nus-pe-bot opened 2 years ago
Firstly, thank you for the report. Our team is rejecting it for the following reasons:
This is indeed a factor that our team has considered. The most obvious way to check that both words exist, separately, is to check for full word matches for both words (such that answering 'anteater' does not match 'ant'). However, we do not want such a strict keyword-matching/checking as users' answers may have typos or plural forms ('ant' vs 'ants') or answer in a different tense (e.g. 'add' vs 'adds' vs 'added'). Taking this into consideration, we have designed the feature as it currently behaves, checking for substrings. If 'ant' is not an intended answer, the keywords are listed in the screenshot above and the user can edit these keywords.
We also believe that this behaviour does not prevent the application from being reasonably useful in normal usage.
Therefore, given the above considerations and the scope of our project, we believe that the current implementation of the feature is the most ideal and the behaviour cannot be considered a flaw.
--
Description of the bug: For the short answer question, if one were to have two keywords, with one being a subset of the other, then there is no way of ensuring that both are in the answer separately.
I think this would be a small feature flaw as while may not be common, this would prevent users from making questions with such answers.
Steps to reproduce:
saq qn/Example question? ans/k/ant k/anteater i/2
quiz
ans/anteater
The answer will be marked correct as per the UG description since both keywords are within the answer, even though 'ant' may not be an intended answer from the user.![Screenshot 2021-11-12 at 4.46.14 PM.png](https://raw.githubusercontent.com/nus-cs2103-AY2122S1/pe/master/files/35426714-1688-4964-af86-b3819efaa70e.png)
[original: nus-cs2103-AY2122S1/pe-interim#1333] [original labels: severity.Low type.FeatureFlaw]