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

Render junction=yes on areas #1253

Closed
sommerluk opened this issue Jan 24, 2015 · 15 comments
Closed

Render junction=yes on areas #1253

sommerluk opened this issue Jan 24, 2015 · 15 comments

Comments

@sommerluk
Copy link
Collaborator

Since #813 the name of junction=yes is rendered on points. This tagging is used for simple junctions. A tagging for complex junctions was in discussion when #813 was merged. In the meantime, discussion/voting has been terminated. So it would be nice to render the name of junction=yes also when it occurs on areas (same font/style like on points).

@sommerluk
Copy link
Collaborator Author

I suppose this means changing openstreetmap-carto.style?

@matthijsmelissen
Copy link
Collaborator

@sommerluk
Copy link
Collaborator Author

@math1985 Yes. I thought that it would be a problem for areas, because it’s only “linear”, not “polygon”? (I had tried to make a PR, but I failed due to a lack of knowledge about the import and a total lack of knowledge of SQL ;-)

@matthijsmelissen
Copy link
Collaborator

Hmm, maybe you're right, I guess it should indeed be polygon.

matthijsmelissen added a commit to matthijsmelissen/openstreetmap-carto that referenced this issue Jan 30, 2015
@matthijsmelissen matthijsmelissen added this to the Bugs and improvements milestone Feb 2, 2015
@matthijsmelissen matthijsmelissen modified the milestones: 3.x - Needs upgrade to openstreetmap-carto.style, Bugs and improvements Feb 3, 2015
@matthijsmelissen
Copy link
Collaborator

Better done in osm2psql.

@sommerluk
Copy link
Collaborator Author

@math1985 Should I open a ticket at osm2psql for this?

@matthijsmelissen
Copy link
Collaborator

@sommerluk No, I think we can solve it in our own osm2pgsql style/lua file.

@Codain
Copy link

Codain commented Dec 2, 2016

I'm interested in this feature. Is there any progress so far? I don't think I can help since I don't know the project code and have no test bench.

I think that simply displaying the junction name at the center of the polygon (as if it was a node) is enought, no need to draw the outline at the moment.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Dec 2, 2016

The milestone says it would need to change database layout - could someone confirm (or negate) it?

@pnorman
Copy link
Collaborator

pnorman commented Dec 3, 2016

The milestone says it would need to change database layout - could someone confirm (or negate) it?

We need junction=yes as areas.

@dieterdreist
Copy link

dieterdreist commented Dec 3, 2016 via email

@sommerluk
Copy link
Collaborator Author

Can be resolved without database reload by #2484

@jojo4u
Copy link

jojo4u commented Dec 5, 2016

For a feature I'd recommend a different tagging scheme, e.g. highway=junction

Yes this is usually valid but here we have colliding highway=traffic_signals and junction. But further discussion about tagging should be done elsewhere.

@dieterdreist
Copy link

dieterdreist commented Dec 5, 2016 via email

@kocio-pl
Copy link
Collaborator

Resolved by #2510.

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

Successfully merging a pull request may close this issue.

7 participants