search

hshiah / pe

0 stars 0 forks source link

Duplication detection for multiple blankspaces #6

Open hshiah opened 9 months ago

hshiah commented 9 months ago

image.png

There is no duplication detection for multiple spaces.

nus-pe-script commented 9 months ago

Team's Response

Duplicate issue as issue #501

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Program allows duplicate questions with spaces in between

Description: The user is able to key in dupliocate questions by varying the spaces between words in the add question platform.

Steps to reproduce:

  1. Add questions:
short What is  0.56    n fraction?/1\slash2/fraction/Hard
short What is  0.56       n fraction?/1\slash2/fraction/Hard
short What is  0.56             n fraction?/1\slash2/fraction/Hard

Expected: Program should reject the second and third command.

Actual: Program accepts the second and third commands.

Screenshots:

image.png

[original: nus-cs2113-AY2324S1/pe-interim#421] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

  • For the current version of the app, it may not be feasible to use advanced string matching to detect duplicates.
  • Detecting duplicates is only meant to be featurefor detecting 100% matches of duplicate questions at this stage.
  • Currently the algorithm detects 100% string matches for both question and module to conclude a match.
  • Determining the threshold of string correspondence to conclude a match would most likely not be included in the current version.

    Items for the Tester to Verify

    :question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]