search

farice9 / pe

0 stars 0 forks source link

Estimate takes in "watchlist.txt" as well even though it is not a script #3

Open farice9 opened 4 years ago

farice9 commented 4 years ago

image.png

I believe it'll be better if there's a dedicated folder for script so that files like "watchlist.txt" won't be considered as well.

nus-pe-bot commented 4 years ago

Team's Response

Hello, the reason why we do not have a dedicated folder for script is because that can create a long nesting of folders which could create further confusion and annoyance to the user, and having them place the script files in the workspace folder would make things easier for the user as well. We have allowed the user to specify "watchlist.txt" if they want to as we do not restrict the script file names that the user intends to use so long they can name it in their operating system, so they could name a script file as "watchlist.txt" if they want to and use that as input to the estimate command. While it is true that if they do so they be overwriting their data, but we believe that the file overwrite prompt by the operating system would suffice to warn the user of the consequences behind their action. Moreover, I do not think there is a problem with considering "watchlist.txt" regardless of the scenario as it does not affect the usage or causes any confusions or problems.

I hope this helps with the understanding!

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]