JulianJason / The-Travelling-Man

0 stars 0 forks source link

Word distance needs some tweak #10

Closed JulianJason closed 7 years ago

JulianJason commented 7 years ago

In WordDistance.java, I changed @MatthewNeo's Algo to do it word by word so that it could capture Sentozza to sentosa.

However, running the itinerary planner through the word distance function causes it to go through all the list returned.

e.g. Searching for park causes the itinerarysolver to go through east coast park, fort canning park, etc.

I suggest either a pop up to fix the user's mistake or if that is not possible just return one String with the lowest lev distance.

MatthewNeo commented 7 years ago

Noted.

Do you want me to add the popup window?


From: Jason Julian notifications@github.com Sent: Sunday, November 20, 2016 10:26:52 PM To: JulianJason/The-Travelling-Man Cc: Student - Matthew Neo; Mention Subject: [JulianJason/The-Travelling-Man] Word distance needs some tweak (#10)

In WordDistance.java, I changed @MatthewNeohttps://github.com/MatthewNeo's Algo to do it word by word so that it could capture Sentozza to sentosa.

However, running the itinerary planner through the word distance function causes it to go through all the list returned.

e.g. Searching for park causes the itinerarysolver to go through east coast park, fort canning park, etc.

I suggest either a pop up to fix the user's mistake or if that is not possible just return one String with the lowest lev distance.

You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/JulianJason/The-Travelling-Man/issues/10, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AV5k8UV1nFCzEg5jFL6qpaJbzMJN5rdiks5rAFisgaJpZM4K3jM2.

The above message may contain confidential and/or proprietary information that is exempt from disclosure under applicable law and is intended for receipt and use solely by the addressee(s) named above. If you are not the intended recipient, you are hereby notified that any use, dissemination, distribution, or copying of this message, or any attachment, is strictly prohibited. If you have received this email in error, please inform the sender immediately by reply e-mail or telephone, reversing the charge if necessary. Please delete the message thereafter. Thank you.

JulianJason commented 7 years ago

Works like a charm @MatthewNeo

Implemented