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

remove leftover from background line for intermittent waterways #2636

Merged
merged 1 commit into from
May 22, 2017

Conversation

nebulon42
Copy link
Contributor

fixes #2633

@pnorman
Copy link
Collaborator

pnorman commented May 21, 2017

The plan is to look at this tomorrow and release 3.3.1 and 4.0.0.

@nebulon42
Copy link
Contributor Author

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.

@imagico
Copy link
Collaborator

imagico commented May 21, 2017

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

@imagico
Copy link
Collaborator

imagico commented May 21, 2017

If i move

https://github.com/nebulon42/openstreetmap-carto/blob/63c559ab405826c03332aabec5f56f2198cf84f6/water.mss#L106-L109

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.

@nebulon42
Copy link
Contributor Author

@imagico Yes, you are right. This was the wrong drawing order. Sorry about that. Fixed now.

@nebulon42
Copy link
Contributor Author

FYI. just released carto 0.18.

@pnorman pnorman merged commit 102fb19 into gravitystorm:master May 22, 2017
@nebulon42 nebulon42 deleted the intermittent_fix branch May 24, 2017 20:26
@javiersanp
Copy link

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:
http://c.tile.openstreetmap.org/18/119298/109368.png/status
http://www.openstreetmap.org/#map=16/28.5444/-16.1554

Best regards, Javier.

@imagico
Copy link
Collaborator

imagico commented May 29, 2017

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.

@javiersanp
Copy link

Waiting anxious :)
Thank you Christoph

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

Successfully merging this pull request may close these issues.

Rendering of intermittent streams broken
4 participants