Skip to content

Add Brest Métropole 2023/2024 WMTS orthophoto #2673

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

Merged
merged 6 commits into from
Apr 22, 2025

Conversation

XioNoX
Copy link
Contributor

@XioNoX XioNoX commented Apr 10, 2025

First contribution, please let me know if there is anything I can improve.
Unfortunately there is only a WMTS feed. Hopefully a WMS feed that can be used in ID once osm-fr/infrastructure#586 is done.

Thanks

@andrewharvey
Copy link
Collaborator

andrewharvey commented Apr 17, 2025

The WMTS source only provides EPSG:2154 which limits to editors which can't reproject.

I suggest to use the URL

https://geo.brest-metropole.fr/arcgis/rest/services/public/PhotoAerienne/MapServer/export?f=image&format=png&transparent=true&imageSR={wkid}&bboxSR={wkid}&bbox={bbox}&size={width},{height}

Since that will request Web Mercator tiles which allows use in iD and other editors.

@andrewharvey
Copy link
Collaborator

Sorry I'm not across the licensing, is there a page somewhere documenting this license as permissible to OSM?

@XioNoX
Copy link
Contributor Author

XioNoX commented Apr 17, 2025

I suggest to use the URL [...]
Thanks so much !! We spend quite a lot of time figuring out if that was doable !

Sorry I'm not across the licensing, is there a page somewhere documenting this license as permissible to OSM?

It's the main license used by the french government and local authorities. You can see that most orthophoto in /sources/europe/fr/ use it.

I was able to find this for example (in french) : https://wiki.openstreetmap.org/wiki/FR:OpenData#Licences_compatibles_avec_celle_d'OpenStreetMap_(ODbL)
That says that at long as you show the paternity (source?) you can use it.

@andrewharvey
Copy link
Collaborator

Thanks so much !! We spend quite a lot of time figuring out if that was doable !

The only reason I knew the URL is I'd previously spent a lot of time working it out for other source.

You might prefer to use format=jpg, since you're not using it as an overlay you don't need the transparency from png.

@XioNoX
Copy link
Contributor Author

XioNoX commented Apr 17, 2025

I also changed best to false as the tiles take a significant amount a time to generate (up to ~25s). Let me know if there is a way to optimize this.

@andrewharvey
Copy link
Collaborator

Thanks. You can add

"available_projections": [
            "EPSG:3857"
],

which should fix the check error.

I don't think you can do much about the slowness. In JOSM it may be able to use the service without reprojection, so the JOSM index may want to do that instead of use this export URL.

@andrewharvey
Copy link
Collaborator

I added the privacy_policy_url based on following the links from the URL.

@andrewharvey andrewharvey merged commit a0433c1 into osmlab:gh-pages Apr 22, 2025
1 check failed
@andrewharvey
Copy link
Collaborator

I've just merged this one, but thinking about the included logo, I'm not sure it's okay to include it directly like you have.

The imagery index is licensed CC BY SA 3.0 and while we can include other licensed content, for example with scripts being under their own licenses, it's complicated to manage that for logo's given they are unlikely to be released under an open license.

How about we instead just use https://brest.fr/sites/default/files/medias/brestfr/images/accueil/Groupe%2029.svg?

@XioNoX
Copy link
Contributor Author

XioNoX commented Apr 22, 2025

Thanks a lot ! Any idea when it will be usable on ID ?

About the logo, I don't mind, but the risk is that this URL becomes a 404 with time.

An alternative could be to use the (outdated) logo from Wikipedia : https://fr.wikipedia.org/wiki/Brest_M%C3%A9tropole#/media/Fichier:Brest_M%C3%A9tropole_logo.svg

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.

2 participants