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

Oneway property should be true, false #1028

Closed
nvkelso opened this issue Sep 8, 2016 · 4 comments
Closed

Oneway property should be true, false #1028

nvkelso opened this issue Sep 8, 2016 · 4 comments

Comments

@nvkelso
Copy link
Member

nvkelso commented Sep 8, 2016

We've normalized all the other yes related properties in v1 to be True or False, but oneway is still yes.

Example tile:

yes should be True:

properties: {
kind: "major_road",
name: "Stockton St.",
sort_key: 377,
kind_detail: "tertiary",
source: "openstreetmap.org",
oneway: "yes",
sidewalk: "both",
id: 417400763
},

If it's no, should we even export that as False?

properties: {
source: "openstreetmap.org",
kind: "minor_road",
service: "alley",
oneway: "no",
sort_key: 356,
kind_detail: "service",
id: 332383249
},
@nvkelso nvkelso added this to the v1.0.0 milestone Sep 8, 2016
@nvkelso
Copy link
Member Author

nvkelso commented Sep 9, 2016

This is a bug fix for tiles in our SemVer doc – and will be an additive change to house styles (which don't show one-way arrows yet). Moving to v1.1

@nvkelso nvkelso modified the milestones: v1.1.0, v1.0.0 Sep 9, 2016
@nvkelso
Copy link
Member Author

nvkelso commented Oct 5, 2016

Related / duplicate:

@nvkelso
Copy link
Member Author

nvkelso commented Apr 4, 2017

We're prepping to show oneway arrows on all Mapzen house styles.

@nvkelso
Copy link
Member Author

nvkelso commented Apr 20, 2017

Well, turns out there are valid values besides yes and no:

Neither have very many uses in OpenStreetMap today. But thinking about the lanes on I-395 in the Washington, D.C. area that do this, those exceptions are exceptional.

Propose we close this, but still act on some / all of the issues related in #1028 (comment).

@nvkelso nvkelso modified the milestones: v1.2.0, v1.1.0 Apr 21, 2017
@nvkelso nvkelso modified the milestones: v1.5.0, v1.4.0 May 25, 2017
@nvkelso nvkelso modified the milestones: v1.6.0, v1.7.0 Sep 25, 2018
@nvkelso nvkelso closed this as completed Feb 13, 2019
@ghost ghost removed the nextnext label Feb 13, 2019
@nvkelso nvkelso removed this from the v1.9.0 milestone Feb 20, 2019
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

1 participant