search

ChoonSiang / pe

0 stars 0 forks source link

Difference between format of saved file in UG and actual run #4

Open ChoonSiang opened 10 months ago

ChoonSiang commented 10 months ago

In UG, it only indicate that the format of saved game is CURRENT_PLAYER_COLOR, GAME DIFFICULTY, HUMAN_MOVE_HISTORY and CPU_MOVE_HISTORY. But in actual saved file, the format is more complex than indicated, making it unable / hard for player to manually edit the saved game as indicated in the UG.

In UG:

image.png

Actual run:

image.png

nus-pe-script commented 9 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]

Lack of detail in UG regarding editing save file

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

The user guide expressed that users are able to edit the save file directly to achieve a desired game state, showing only 4 out of 6 components in the save file.

image.png

There was no explanation for the remaining 2 purpose, so the user can only assume that changes to the shown 4 variables are sufficient. However, making changes to 1 piece location caused the file to not be recognised

image.png

Original Left, Edited Right. This suggests that there might be other factors for determining game state, which is not explained in the User Guide.

Since this editing feature is explicitly mentioned in the User Guide, it is safe to assume that average users will attempt to use this feature. However, being unable to utilize it would warrant a MEDIUM severity as it will likely affect all users who attempt this.

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

Their Response to the 'Original' Bug

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

No details provided by team.

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: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]