search

Wincenttjoi / CS2103T-PracticalExam

0 stars 0 forks source link

Branching out from activity diagram is made complicated #12

Open Wincenttjoi opened 4 years ago

Wincenttjoi commented 4 years ago

image.png

image.png

What command? Like the other activity diagram, its stated review command. There is an inconsistency

Furthermore, I think the diagram should branch out into 4 different scenarios from the first diamond, instead of multiple diamonds and have triple if else statements.

nus-se-bot commented 4 years ago

Team's Response

Hi, thanks for your report! We believe the branching out is not an issue as we followed the format of activity diagrams in the textbook.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Take the example of highest to lowest success rate, it does not have to depend and check if there is sorting based on least to most reviewed first. My message I am trying to bring across is that it should all branch out from one point instead of multiple if elses, since they are not sequentially determined and dependent from each other.

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: The team does not provide their reason for VeryLow?