nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

lots for motorcycles and heavy vehicles #566

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

i skimmed through the text file of all the carpark lots, and it seems that all (or at least majority of) the carparks don't have lots for motorcycles and heav vehicles. if most carparks have 0 lots for motorcycles and heavy vehicles, perhaps it is better not to display when the user finds it, to reduce clutter. or if the case is that the data from LTA does not specify for motorcycles and heavy vehicles, it can be removed entirely

image.png

image.png


[original: nus-cs2113-AY2223S1/pe-interim#590] [original labels: type.FeatureFlaw severity.Low]

malcolmang commented 2 years ago

Team's Response

Hello,

Thank you for your bug report.

While the LTA API sometimes provides incomplete data, the team feels it is useful to still show this information in a breakdown, since a car user necessarily can't use a parking lot from the other types involved. Therefore, even if a majority of carparks only contain car lots, a motorcyclist would still find it useful to use the find command and see carparks that have the motorcycle lot type. We feel it is better to be transparent about the data we are receiving. We thus reject this as intended behaviour, and one that provides value to a portion of users.

Duplicate status (if any):

--