search

chishanw / pe

0 stars 0 forks source link

Model diagram too complicated/looks messy #12

Open chishanw opened 4 years ago

chishanw commented 4 years ago

As per the title, this diagram couold have been broken up into 2 separate diagrams, with the ModelManager and below components extracted into another diagram.

image.png

nus-pe-bot commented 4 years ago

Team's Response

Thank you for your report.

This diagram is already one out of two Model diagrams, i.e. the details of the Deck component and those under it are already abstracted out to the diagram after this one. What we have now is what we believe to be a reasonable diagram without removing too many important interacting details of our application or having too many abstracted-out diagrams. Moving the ModelManager component to the next diagram may then lead to the cluttering of the next diagram.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]