Brighten built-up areas on z12 #3225
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This resolves 2777
|
Thanks! I have to test brightness progression on the map, but this change looks like a proper tuning of midzoom rendering. |
|
It looks good for me. Colors are well balanced and details are more visible. The downside is that we loose a feeling that built-up areas are coherent for big cities, but we gain more intuitive, gradual color change. If nobody objects, I plan to merge it tomorrow. Warsaw, z12 Before After SW of Warsaw, z12 Before After |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This resolves #2777
The brightness sequence from z10 up used to be 212-216-211-223, now it is 212-216-221-223.
Test rendering with links to the example places:
Before
After
