search

kennethszj / pe

0 stars 0 forks source link

unable to load json file despite not editing the file after saving #4

Open kennethszj opened 1 week ago

kennethszj commented 1 week ago

image.png after adding the data manually above and saving as a json file,

i was unable to load the file when i restarted the program

image.png

soc-pe-bot commented 3 days ago

Team's Response

Thanks for raising this issue. This is actually the expected behavior. File parsed in and file generated from the program are entirely different. Tester seem to stick with the idea of iP where data generated through the program will be saved and allowed to be pass through the program again (Closed Loop System). However, as mentioned at the start of the UG under Pre-Usage Guidelines, the allocation results file is NOT meant to be re-inputted into the program. The program is designed to have one input and one output where the output cannot be used as input (Open Loop System). The UG also provided some examples on what are the accepted file types and format and even provided test files for the users to experiment with. Our team believes that we handled this misuse case appropriately by prompting the error sufficiently to the user and allowing them to continue using the program.

image.png

image.png

The 'Original' Bug

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

uploading a csv file that was generated by the program fails to load without editing

image.png

below is the csv file without editing

image.png

[original: nus-cs2113-AY2425S1/pe-interim#290] [original labels: severity.Medium type.FunctionalityBug]

Their Response to the 'Original' Bug

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

Thanks for raising this issue. This is actually the expected behavior. File parsed in and file generated from the program are entirely different. Tester seem to stick with the idea of iP where data generated through the program will be saved and allowed to be pass through the program again (Closed Loop System). However, as mentioned at the start of the UG under Pre-Usage Guidelines, the allocation results file is NOT meant to be re-inputted into the program. The program is designed to have one input and one output where the output cannot be used as input (Open Loop System). The UG also provided some examples on what are the accepted file types and format and even provided test files for the users to experiment with. Our team believes that we handled this misuse case appropriately by prompting the error sufficiently to the user and allowing them to continue using the program.

image.png

image.png

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 response Team chose [`response.Rejected`] - [ ] I disagree **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]