search

Tanishq4331 / pe

0 stars 0 forks source link

Sequence diagram bug #16

Open Tanishq4331 opened 2 years ago

Tanishq4331 commented 2 years ago

image.png

Image seems to be cut off.

nus-pe-bot commented 2 years ago

Team's Response

The image is not cut off, there are only no return arrows, which are optional for sequence diagrams.

No return arrows do not affect the purpose of the diagram.

We changed it to VeryLow because it is only a cosmetic issue.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: If omitting the return arrows is a deliberate choice, this should be explicitly mentioned in the DG, to ensure that it does not hinder understanding. As it stands, the reader might be led to believe that the diagram is incomplete, especially since the diagram contains activation bars but no return arrows.

:question: Issue severity

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

Reason for disagreement: Since there is no explanation about the omission, the diagram might mislead the reader. Thus, I believe that the issue is of severity.Low.