Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Route updates #272

Merged
merged 10 commits into from
Mar 4, 2017
Merged

Route updates #272

merged 10 commits into from
Mar 4, 2017

Conversation

doublestranded
Copy link
Contributor

@doublestranded doublestranded commented Mar 3, 2017

@kkowalsky
Copy link

The Vehicle Types section is linked, but doesn't have a path... Is that intentional? Also vehicle types 1702 and 405 are the best.

add 'a' before 'Douglas-Peucker algorithm' and link to a page (for those of us who don't know what that is).

In the Distance calculation heuristic algorithm section, 💯 for explaining the algorithm behind stop patterns, that's immensely useful. There's a spelling error after that paragraph, "Transitland logs inaccuraties" <-- should be inaccuracies

@kkowalsky
Copy link

After those two spelling checks, you're all set by the docs team 😄

@drewda
Copy link
Member

drewda commented Mar 3, 2017

This is looking good, @doublestranded. And thanks for your close read, @kkowalsky.

A few improvements we should do before merging:

I can work on the first two of those items.

@doublestranded
Copy link
Contributor Author

Thanks @kkowalsky and @drewda! Good eyes. I've made the edits (using the Douglas-Peucker Wikipedia page for stability. Plus it has a good graphic). Thanks for making the vehicle types section more complete @drewda !

@kkowalsky kkowalsky merged commit 567f452 into master Mar 4, 2017
@kkowalsky kkowalsky deleted the route-updates branch March 4, 2017 00:27
@drewda drewda removed the in progress label Mar 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants