-
Notifications
You must be signed in to change notification settings - Fork 822
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
Switch to ocean polygons #1982
Comments
Sounds reasonable. What did you have in mind with water styling / outlines of water? |
Other advantages would be:
Simplified water polygons have meanwhile been set up on openstreetmapdata.com - we were planning to roll this out together with some other improvements in coastline processing which are not quite finished yet but you can already get the simplified water polygons on http://data.openstreetmapdata.com/simplified-water-polygons-complete-3857.zip |
I want to keep that separate, but a fairly common convention is a darker blue outline. |
would doing this stop the issue of broken coastlines "flooding" the land? or could the reverse issue then happen, which areas of sea being shown as land? |
@donaldrnoble It makes no difference, "flooding" is still possible. |
I reverted the PR that fixed this, see #2101. |
The benefits of this change still remain important, as listed above:
Makes it possible to solve these issues: #621, #788, #1547, #1781, #2013, #2025, #2609, #3273, and PR #3670 To summarize the current status, issue #2101 caused the reversion to using land-polygons when some users had problems with the This problem was not easy to reproduce, so it is still not clear why this happened. We do not know if this problem would happen again with the current Options for moving forward:1) Try the rendering from #2066 again:
2) Use a different for low zoom water polygons
a) Raster mask for z0 to z6:
b) Reduced polygons for z0 to z6:
c) Generalized coastlines & water polygons for z1 to z8:
3) Create a new data source for low-zoom coastline/ocean rendering
Are there other options that I've missed? |
Ah, one more option: 4) Use ocean polygons at high zoom and land polygons at low zoom
|
Are there any other issues This might be a good case for "implement and regress if needed" Like was done for rendering of surfaces on roads. I don't see how it can be tested to see if it was fixed or not otherwise. Especially if it was a server anomaly like was suggested. |
Note option 1, 2c and 4 are for the coastlines only while 2a and 2b are for all waterbodies. The German style (https://github.com/giggls/openstreetmap-carto-de/) uses both the reduced water shapefiles and the simplified ocean polygons so any issues with those can be tested with that style. |
@imagico, I had been focusing on the coastline/ocean, but it would be very helpful to have a separate data source for low-zoom lakes and rivers. I'm currently downloading the reduced ocean shapefiles for testing, to see how they work when rendered at 2x resolution (e.g. for "retina" tiles or printing). But it would be great to have generalized rivers and lakes, to try with the generalized coastline. I believe your low-zoom demo shows an example of this? Are those generalized lake and river files proprietary? Would it be possible to make generalized river and lake files available on openstreetmapdata.com to go with the generalized coastlines? |
No, that is a much more complicated process and would also be questionable in terms of mapper feedback. Keep in mind that our goals require that the style needs to render the data in a way that allows mappers to understand how the data produces a certain rendering result based on basic observation. This is not the case if the data is processed with a complex, non-local algorithm. |
Was that "No" a specific response to the last line: "Would it be possible to make generalized river and lake files available"? Or are you also saying that the generalized coastlines are inappropriate to use at z1 to z6, due to mapper feedback? Are |
I don't think it is any more problematic than the currently used line simplification (probably less because it is more consistent) but i think both are questionable for this style and its function. If you follow comments of mappers regarding OSM-Carto rendering you can see that the simplified and outlined low zoom coastline rendering we have right now is quite frequently a source of confusion and misunderstandings.
The simplified shapefiles are only used for performance reasons, they are not meant to introduce any visual difference in the results. If you look very closely you can detect a tiny difference at z9 but that is really insignificant. |
We should switch from preprocessed land polygons on a water colour background to preprocessed water polygons on a land colour background.
This would
Cross-ref #621
Issues
cc @joto @imagico
The text was updated successfully, but these errors were encountered: