pqienso / pe

0 stars 0 forks source link

Sequence diagrams: return arrows on the same level #14

Open pqienso opened 5 months ago

pqienso commented 5 months ago

Screenshot 2024-04-19 172751.png Seen on page 10 of DG. Not sure if the arrow stating "caloriesBurnt.value" and the arrow stating "increment counter for caloriesCount" should be on the same level, as one should return sequentially after the other.

Same can be said for the 2 opt and loop blocks above this opt block in the same diagram.

nus-pe-script commented 5 months ago

Team's Response

I understand, however this was done for diagram simplicity as stated by the DG requirement to avoid over-complicated diagrams.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: I do not see how the arrow being on the same level will make the diagram simpler. In fact, it makes the diagram less clear about how it is run.


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