EastCoastGreenwayAlliance / ecg-map

Interactive map and trip planner for the ECGA
https://map.greenway.org
7 stars 0 forks source link

Cropping of starting and ending segments #9

Closed gregallensworth closed 7 years ago

gregallensworth commented 7 years ago

This one is a lower priority, since we have other data cleanup and UI stuff to work out. Don't move on it, unless we end up with surplus budget.

Consider a case where the ending point was North 14th Street, on the right-center of this image.

image

The entire segment is used, despite not ending at the desired location. This "extra" comprises about a mile.

In practice, the expectations here are not too high and this is likely acceptable. But as a possible phase 2 improvement:

danrademacher commented 7 years ago

To be clear, only the clipping segment part is low priority, right? Finding a starting point is required. But our original plan of “find cue point nearest to the location user searched” is no longer valid, since we’re collapsing almost all cue points into the routing module's calculated values. So maybe we need a method of "find nearest segment" to user-supplied start and end?

Does this match your understanding, @clhenrick?

gregallensworth commented 7 years ago
nilesbarnes commented 7 years ago

This seems to have a relatively small impact on long searches but fairly significant impact on shorter searches.

nilesbarnes commented 7 years ago

@danrademacher I think if Greg could spend up to 4 hours of maintenance time to see if he can address this it would be helpful.

gregallensworth commented 7 years ago

Internal note: In the set of demos I have currently set up, some examples:

gregallensworth commented 7 years ago

Give this one a look. It looks to be spot-on with what we need: start/end segments now clip to the on-trail start and end points, and distances are recalculated for directions reporting.

danrademacher commented 7 years ago

confirmed locally: image

Was 169 miles.

Trenton to Bronx Path shaves off 4 miles: image

And Providence to Eastham: image

nilesbarnes commented 7 years ago

Looking good! @gregallensworth and @danrademacher Here is an example right at our HQ office (starting point) to downtown Durham. https://ecg-map.herokuapp.com/?loc=12,35.95948,-78.96371&route=35.92176,-78.93047,35.99598,-78.90156

danrademacher commented 7 years ago

And here's that with the fix in place: image

danrademacher commented 7 years ago

Just deployed: image

Try that link again, @nilesbarnes

nilesbarnes commented 7 years ago

Great! Thank you @danrademacher and @gregallensworth Much improved user experience.