-
Notifications
You must be signed in to change notification settings - Fork 819
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
Leisure=track should not be an area #574
Comments
No it doesn't, nor would changing leisure to |
According to Wiki it should be rendered as area if there is also area=yes. However I'm not able to prepare PR resolving this issue, because I don't know what changes should be done exactly. |
This could be fixed by osm2pgsql-dev/osm2pgsql#346. If osm2pgsql doesn't accept ths change we could also create our own .lua file. However, I'm still not fully sure what the desired behaviour is? |
For me by default it should be rendered as a line and as an area when a closed line is tagged with area=yes. |
... and as an area when it is a multipolygon. |
@math1985 as Lonvia said in osm2pgsql-dev/osm2pgsql#346 (comment) if you're going to move to lua processing in OSM's "standard" style, surely you're going to use custom lua processing for that style, and not whatever standard/example "style.lua" is included with osm2pgsql? There are places where using lua instead of SQL to do processing would vastly simplify the resulting SQL that needs to be written - I think it would be very beneficial to make use of that capability where appropriate. (naturally "moving to lua processing" assumes that the drawbacks of that, such as database reloads if e.g. tag consolidation is done have been considered and the change is thought to be beneficial overall - that's a much larger issue than just one style change) |
I haven't thought of that yet really. The process of moving to hstore is going extremely slow (it's already on the agenda for multiple years), so at the moment I'm inclined towards rolling out a solution that is as simple as possible first, so at least we can get something done, rather than keep discussing without ever accomplishing something. Also, some of our users (for example @gravitystorm himself) use a single PostGIS database for multiple rendering styles, which also suggests not making to complex / style-specific decisions in the lua file. |
Related to #2238 (lua branch). |
I think this should be renamed and reopened. Or we need a new issue. https://www.openstreetmap.org/way/144179903 is not rendered on master and lua, so we have no style definition. |
Opened new issue #2632. |
Leisure=track is currently rendered as area. However, it should be rendered as way.
This requires a change to the osm2pgsql style sheet.
See also https://trac.openstreetmap.org/ticket/2339
The text was updated successfully, but these errors were encountered: