YoengKokLeong / pe

0 stars 0 forks source link

DG sequence diagram #13

Open YoengKokLeong opened 2 months ago

YoengKokLeong commented 2 months ago

Screenshot 2024-04-19 at 5.42.58 PM.png

Omitting the optional element of activation bars may result in ambiguity

nus-se-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

No Activation Bar in the UG Seq Diagrams

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.


截屏2024-04-19 下午5.32.02.png

ALL sequence diagrams do not have activation bar... Attached is one example screen shot. Activation bar shows the time needed for an actor to process the task and during which the object remains active.

Possible solution: when generate diagram please add command "active XXX"


[original: nus-cs2113-AY2324S2/pe-interim#2336] [original labels: severity.Medium type.DocumentationBug]

Their Response to the 'Original' Bug

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

Textbook has clearly stated that activation bar can be considered an optional feature (refer to image below). While we agree that we may lose some details as a result of omitting this detail, we believe that this is not sufficient to constitute a bug since the intended message of the diagram is still conveyed to those reading the DG.

image.png

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.Rejected`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]