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

stop displaying really small zoos and theme parks #1635

Merged
merged 1 commit into from
Jul 10, 2015
Merged

stop displaying really small zoos and theme parks #1635

merged 1 commit into from
Jul 10, 2015

Conversation

@kocio-pl
Copy link
Collaborator

kocio-pl commented Jul 5, 2015

+1

This was referenced Jul 5, 2015
@imagico
Copy link
Collaborator

imagico commented Jul 7, 2015

Even the larger zoos seem very strong at the low zooms:

http://www.openstreetmap.org/#map=10/54.0586/10.2626

IMO these could well start at a later zoom level (like 12). The nature reserve boundaries can also be counterproductive for clarity at the lower zooms (size threshold here is 100 pixels). maybe unifying both with a size threshold of ~1000 pixel at z<12 and 100 pixel above would work.

@jojo4u
Copy link

jojo4u commented Jul 7, 2015

The example from imagico indeed looks like a scar on zoom 10 and 11.

@matthijsmelissen
Copy link
Collaborator

My bad, thanks for fixing.

@imagico @matkoniecz Feel free to open a new issue or PR if more changes are needed.

@matthijsmelissen matthijsmelissen merged commit c103272 into gravitystorm:master Jul 10, 2015
@matkoniecz
Copy link
Contributor Author

@math1985 It is partially my fault given that it was my idea to start rendering so early.

@kocio-pl
Copy link
Collaborator

My feeling is that zoom range adjusting is for general rendering, while way_area is for fine tuning it - they are just different tools, so no need to look who was wrong. =}

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.

5 participants