search

nus-cs2113-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Sequence Diagram for MiniMax algorithm #696

Open nus-pe-bot opened 10 months ago

nus-pe-bot commented 10 months 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 sequence diagram for the minimax algorithm does not properly follow the UML notation from class. For example activation bars should end after returning and self invocations need their own activation bars

Screenshot 2023-11-17 at 17.38.54.pngScreenshot 2023-11-17 at 17.38.54

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

onx001 commented 10 months ago

Team's Response

The minimax sequence diagram is an artefact from the PE-D where no bugs were reported. There exists several documentation bugs which can cause an annoyance and hinder the flow or cause confusion, hence a medium rating seems reasonable.

Duplicate status (if any):

--