search

nathanshew / pe

0 stars 0 forks source link

Error with interactions with findmedcon #12

Open nathanshew opened 3 days ago

nathanshew commented 3 days ago

image.png

when I use findmedcon Stage 4 cancer, all patients with different stages of cancer show up, rather than the ones with only stage 4 cancer. This limits the ability to store different types of illness.

nus-pe-bot 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

Thank you for your feedback! As mentioned in the User Guide, the current implementation of the findMedCon command will only find patients with conditions containing the given keyword. Patients with medical conditions matching at least one keyword will be returned (i.e. OR search).

image.png

As such, based on your example, patients with conditions containing the keywords Stage , 4 and cancer will be found and this is intended based on how we designed this command.

While our implementation of the findMedCon command may result in some patients being displayed in the Patient List with conditions the doctor might not be looking for, we feel that this causes only minor inconvenience for our users as they will still be able to pinpoint and identify the patients with the specific conditions they are looking for, since those patients will also be displayed in the Patient List. Therefore, we have decided to lower the severity of this issue to Low.

However, we do acknowledge that it would be better to modify findMedCon to allow users to find patients with the given conditions instead of the given keywords as it allows doctors to easily locate the specific patients they might be looking for. This can be improved in future iterations of our application.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your reason]

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