search

nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Model component diagram too complicated #1282

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

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.

The following diagram is too complicate and the texts are too small. Can be broken down into multiple smaller diagrams image.png

[original: nus-cs2103-AY1920S2/pe-interim#1239]

Dban1 commented 4 years ago

Team's Response

This is very debatable. Our group feels that the complexity of this model is acceptable, and that the document can be zoomed in if need be. Moreover, the diagram clearly shows all the packages involved in the application. We feel that splitting it up further into multiple smaller diagrams of the packages will hinder the reader's understanding of the overall picture, since he would have to refer to multiple different diagrams.

Our team has decided to reject this and downgrade it since we do not see it as an issue. We shall let the teaching team have the final say on this matter.

Duplicate status (if any):

--