search

joshualeeky / pe

0 stars 0 forks source link

No limit in recipe list #2

Open joshualeeky opened 2 years ago

joshualeeky commented 2 years ago

image.png

Having no limit for the recipe list can cause the user to key in too many recipes resulting in the program not working for a very long time as the output will keep printing on the screen, (I had to ctrl+C to continue)

nus-pe-script commented 2 years ago

Team's Response

No limit of modules in the recipe should be imposed because this will reduce the usability of the program. Also, no proper real-world use of building recipes will contain an extremely large number of modules, which can be considered as improper use of the recipes. Recipes with a reasonable number of modules will run relatively quickly (depending on the size of the inputted data and the specifications of the hardware).

Also, refer to this for more details: https://github.com/nus-cs2113-AY2122S1/forum/issues/141

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]