STeng618 / pe

0 stars 0 forks source link

Alt frame in UML for hydration #16

Open STeng618 opened 2 months ago

STeng618 commented 2 months ago

Used an alt frame but there is no "else"

Screenshot 2024-04-19 at 5.34.14 PM.png

nus-se-script commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Alt frame in UML for calorie

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.


Use of alt frame but there is no "else". I think you meant "opt"

Screenshot 2024-04-19 at 5.24.18 PM.png


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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Hi there, thank you for the bug report!

Our team recognizes that opt can be used here also. However, we feel that we are not violating the rules by using alt here as it is not clearly mentioned that when only one condition is present, alt cannot be used.

Semantically, when there is only one condition, alt and opt mean the same thing.

As such, we have chosen not in scope label for this issue as we believe that it is a valid concern but we should not be penalized for this.

image.png

Thank you!

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: ### As per the course guidelines, these are different diagrams and thus cannot be considered duplicate bugs

Screenshot 2024-04-23 at 4.54.54 PM.png


## :question: Issue response Team chose [`response.NotInScope`] - [x] I disagree **Reason for disagreement:** Refer to lecture video uploaded on 22/03/2024 at 28:00. The professor explained the difference between alt and opt frames alt frames are used for "alternative" paths (literally, more than 1). Using an alt frame instead of an opt frame does not comply with course materials.