search

nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

When loading in TXT data and then saving it again, the data format changes #343

Open nus-pe-bot opened 1 day ago

nus-pe-bot commented 1 day ago

Similar to JSON, loading in and then re-saving the sample test.txt data leads to the resulting allocation_results.txt file to have the data in a different format. Before:

image.png

After:

image.png

This causes a similar error when loading back the resulting TXT file.

image.png

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

ehz0ah commented 1 day 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

Duplicate status (if any):

Duplicate of #569