soft-eng-practicum / Guber

0 stars 0 forks source link

Rider v Driver Proximity #25

Open Dave89rr opened 8 years ago

Dave89rr commented 8 years ago

Is Rider Close Enough for Ride?

Test Driver Rider Output
1 980 Colony Creek Dr, Lawrenceville GA 306 Sinclair Way, GA No
2 306 Sinclair Way, GA 980 Colony Creek Dr, Lawrenceville GA Yes
3 980 Colony Creek Dr, Lawrenceville GA 1000 University Center Lane, Lawrenceville No
4 1000 University Center Lane, Lawrenceville 980 Colony Creek Dr, Lawrenceville GA Yes
5 2148 Duluth Hwy, Duluth, GA 30097 980 Colony Creek Dr, Lawrenceville GA No
6 980 Colony Creek Dr, Lawrenceville GA 2148 Duluth Hwy, Duluth, GA 30097 Yes
7 1866 Buford Hwy #100, Duluth, GA 30097 6500 Sugarloaf Pkwy # 200, Duluth, GA 30097 Yes
8 6500 Sugarloaf Pkwy # 200, Duluth, GA 30097 1866 Buford Hwy #100, Duluth, GA 30097 No

Of the tests I did, I decided to record these 8 here, because some of the results were confusing.

Tests not shown here included opposite sides of GGC, in which case flipping the address didn't change the result. Given that the driver and rider were within certain threshold the system consistently gave "yes" and outside that threshold it gave "no"

Knagus commented 8 years ago

Do the same in google maps and you find the same oddities. Thanks for bringing it up.

Dave89rr commented 8 years ago

@Knagus I did those searches both in your app and in google, but I don't know what oddities you're referring to? Is there a specific link you have for google that I could test that says "yes" v "no"? When I looked it up I only saw miles and multiple options for routes.

mstackpo commented 8 years ago

The way the app is supposed to be coded is this: Three separate queries are made to google. A) Find the driving time from the driver to rider. B) Find the driving time from the rider to GGC. C) Find the driving time from the driver to GGC. If A + B <= C + 10 minutes, the result should be yes. Otherwise, no. One issue might be that you used "1000 University Center Lane," whereas I used "Georgia Gwinnett College." According to Google, they're a 2 minute walk apart. It's also possible that I have drivers and riders switched somewhere in the code. I'll double check. It also looks like there might be slight discrepancies checking against Google Maps because it rounds the results. So a driving time of 12.5 minutes might be displayed as 13 minutes.

Thanks for the testing. Very helpful!

Dave89rr commented 8 years ago

@mstackpo I think that might be the case that you do have the driver and rider switched. It would explain why Tests' 5 & 6 results (Yes v No) were reversed. Edit: Although it wouldn't explain why Tests' 7 & 8 results were correct. :/