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

Support other OSM based online map sources (fix #8148) #8187

Merged
merged 1 commit into from
Apr 25, 2020

Conversation

moving-bits
Copy link
Member

@moving-bits moving-bits commented Apr 11, 2020

Extend our supported OSM based online maps by

  • openstreetmap.de and
  • cyclosm.org

Avaialble for NewMap only.
Also renamed "OpenStreetMap" to "OpenStreetMap.org" to distinguish from "OpenStreetMap.de".

Example screen shots:

OpenStreetMap.org (aka "OSM:Mapnik")
image

OpenStreetMap.de:
image

CyclOSM.org:
image

@moving-bits moving-bits self-assigned this Apr 11, 2020
@moving-bits
Copy link
Member Author

moving-bits commented Apr 11, 2020

"Help wanted":

Current implementation seems to work fine so far, with one caveat which I haven't been able to tackle yet: After switching between two of the three OSM online services the map stays blank until you close and reopen the map. (This does not happen when switching from an online based map to an offline based map and vice versa.) So there seems to be one additional step needed.

Solved

Copy link
Collaborator

Codacy Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- main/src/cgeo/geocaching/maps/mapsforge/TileSourceOsmde.java  1
- main/src/cgeo/geocaching/maps/mapsforge/TileSourceCyclosm.java  1
         

See the complete overview on Codacy

@moving-bits moving-bits changed the title WIP Support other OSM based online map sources Support other OSM based online map sources (fix #8148) Apr 12, 2020
@SammysHP
Copy link
Member

Do you have a permission for us to use these maps?

cyclosm.org:

A Tile Map Service is available at https://{s}.tile-cyclosm.openstreetmap.fr/cyclosm/{z}/{x}/{y}.png under a fairuse tile policy.

https://operations.osmfoundation.org/policies/tiles/:

Heavy use (e.g. distributing an app that uses tiles from openstreetmap.org) is forbidden without prior permission from the Operations Working Group.

openstreetmap.de has something similar:

https://www.openstreetmap.de/germanstyle.html

Das Einbinden der Kacheln in eigene Webseites können wir aufgrund der begrenzten Bandbreite leider nur für nichtkommerzielle Zwecke erlauben. Kleinere Webanwendungen, wie z.B. der Anfahrtsplan auf der Firmenwebseite dürfen die Kacheln selbstverständlich verwenden. Bitte beachten sie auch die Tile usage policy.

@moving-bits
Copy link
Member Author

Do you have a permission for us to use these maps?

Both new sources refer to the identical policy we already have for our current OSM online tile server, and attribution notes have been extended to list the two new sources.

@moving-bits
Copy link
Member Author

retest this please

@SammysHP
Copy link
Member

Both new sources refer to the identical policy we already have for our current OSM online tile server

And nobody said that it is OK what we do. ;)

@moving-bits
Copy link
Member Author

And nobody said that it is OK what we do. ;)

The comment openstreetmap/operations#371 (comment) can be read as a "temporarily ok, until we say otherwise".

This PR intends to reduce load on that server by distributing it over several different sources, hopefully resulting in low/acceptable load for all servers involved.

#8180 tries to help new users to configure offline map sources more easily to make more users of c:geo use offline maps.

@moving-bits
Copy link
Member Author

retest this please

@moving-bits moving-bits merged commit 33518ca into cgeo:master Apr 25, 2020
@moving-bits moving-bits deleted the issue_8148 branch April 25, 2020 18:30
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.

None yet

3 participants