-
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
Rendering of intermittent streams broken #2633
Comments
Did something go wrong with the hstore merge perhaps? |
I don't think this is due to the hstore changes, i think this is caused by 970df74, specifically https://github.com/gravitystorm/openstreetmap-carto/blame/master/water.mss#L195-L197 Side note: the water-lines code is quite difficult to understand meanwhile with all the tunnel/background/bridge stuff mixed fairly wildly. |
Does this also impact 3.3.0? |
Yes, that looks like an error. some background/line-* properties are set, while background/line-color is unset except when int_tunnel = yes |
Well spotted. This was somehow an leftover I have missed. This also impacts 3.3.0. I will cherry pick the commit to the 3.x branch once committed. As I have seen that 3.3.0 is now live we should probably release 3.3.1. |
I noticed something similar with a large black line on the map - is it related to this issue? https://www.openstreetmap.org/way/4301350 ok on z18/19 though |
…ix bridge drawing order, fixes gravitystorm#2633
@boothym Thanks for reporting this. It is now also fixed in #2636. |
…ix bridge drawing order, fixes gravitystorm#2633 (cherry picked from commit 102fb19)
It also happens in channel bridges: https://www.openstreetmap.org/#map=17/41.62288/-0.90528 |
@Robot8A thanks, this was also fixed. |
Not sure if you're aware but this is still visible/an issue on osm.org, see the black lines which are intermittent streams: |
You have old tiles here - either browser cache or server side caching. |
No worries, I'll let it sort itself out over time then. |
New way https://www.openstreetmap.org/way/501481012 and it's still being rendered black: https://www.openstreetmap.org/#map=17/-28.96073/-52.65422 It can't be a cache issue with new data. |
The fixed version of the style has not yet been rolled out. Just look when openstreetmap/chef#124 is closed. |
Rendering of intermittent streams seems broken in master.
Screen shot:
The text was updated successfully, but these errors were encountered: