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

yes/no/maybe bicycle & walking values #774

Closed
nvkelso opened this issue Apr 30, 2016 · 5 comments
Closed

yes/no/maybe bicycle & walking values #774

nvkelso opened this issue Apr 30, 2016 · 5 comments
Milestone

Comments

@nvkelso
Copy link
Member

nvkelso commented Apr 30, 2016

We're always including yes, no & other values on the new walking, bicycling ways. But should we only include no or yes values contextually?

For instance, motor_vehicle: "no", is interesting here because tracks are assumed to be motor_vehicle: "yes",, but foot: "yes", I think is assumed to be yes, so why export that?

type: "LineString",
properties: {
kind: "path",
bicycle: "yes",
name: "Alta Tr.",
sort_key: 354,
landuse_kind: "national_park",
horse: "yes",
source: "openstreetmap.org",
motor_vehicle: "no",
foot: "yes",
tracktype: "grade2",
id: 239140918,
highway: "track"
},
@nvkelso nvkelso added this to the v1.0.0 milestone Apr 30, 2016
@nvkelso
Copy link
Member Author

nvkelso commented Apr 30, 2016

Another example, no isn't useful here for cycleway, we only want to know when it's yes.

@nvkelso
Copy link
Member Author

nvkelso commented Apr 30, 2016

Related to #453.

@rmarianski
Copy link
Member

Should we normalize it to yes if the value isn't no? It can be other values like "lane". Or do we want to pass it through as long as it isn't no?

@nvkelso
Copy link
Member Author

nvkelso commented May 1, 2016

Pass thru as long as it's not no and not none.

(I suspect some of those other values could probably be normalized (there look to be variants), but let's just pass them thru for now.)

@nvkelso
Copy link
Member Author

nvkelso commented May 4, 2016

Correctly updated. There's more work to be done to normalize all these, that's tracked in #792.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants