-
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
remove leftover from background line for intermittent waterways #2636
Conversation
The plan is to look at this tomorrow and release 3.3.1 and 4.0.0. |
If you release 4.0.0 then please bump the carto requirement to the latest released version. Planned 0.18 has only one improvement in it which decreases parsing time for osm carto. Unfortunately, it is detrimental for OSM Bright. If I can fix this today there will be 0.18. Otherwise we would be fine with 0.17.3. |
I can confirm this fixes the stream problem, however it does not fix the waterway bridge problem mentioned in #2633 (comment) See also http://www.openstreetmap.org/#map=16/52.3037/8.9295 |
If i move after the bridgecasing block this seems to be fixed. As said the code is fairly difficult to read and understand with the mixture of different things and drawing order sensitivity. There could be other side effects we have not yet spotted. |
…ix bridge drawing order, fixes gravitystorm#2633
63c559a
to
102fb19
Compare
@imagico Yes, you are right. This was the wrong drawing order. Sorry about that. Fixed now. |
FYI. just released carto 0.18. |
Please confirm that the waterway=streams, intermittent=yes black lines rendering of issue #2633 is fixed. I still view this problem in tiles drawn just now. See: Best regards, Javier. |
As i have mentioned in https://lists.openstreetmap.org/pipermail/dev/2017-May/029904.html 3.3.1 has not yet been rolled out. Just look when openstreetmap/chef#124 is closed. |
Waiting anxious :) |
fixes #2633