search

CXIA17 / pe

0 stars 0 forks source link

Error in sequence diagram #12

Open CXIA17 opened 2 months ago

CXIA17 commented 2 months ago

Should not have blocks that are broken down in the middle.

Screenshot 2024-04-19 at 5.41.07 PM.png

nus-pe-bot commented 2 months ago

Team's Response

From the textbook:

image.png

The two separate activation bars are for two separate events when the UI's methods are invoked.

The first bar signifies that the readUserCommand() method is called for the UI to read the user command.

The second bar signifies that the talk() method is called for the UI to print the execution output.

This diagram is therefore valid.

As this appears to be a misunderstanding of the diagram, we will reduce the severity of this bug from Medium to VeryLow.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

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