search

nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Redundant Logic Architecture Diagram #2443

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.

image.png

image.png

The exact same information is provided in the first architecture diagram, information in the diagram in 2.3 is repeated. Consider abstracting the first architecture diagram and have those information only avaliable here.

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

kevinputera commented 4 years ago

Team's Response

The said logic architecture diagram is provided to zoom in into just the architecture of the Logic component. We believe that this will enhance the comprehensibility (as the user won't need to jump up and down the document, can just focus on the current abstraction level/component) of our developer guide.

Duplicate status (if any):

--