paul-carteron / happign

Happign allows you to use the APIs provided by the IGN (France) to download their public data.
https://paul-carteron.github.io/happign/
GNU General Public License v3.0
27 stars 3 forks source link

Add CRS parameter #4

Closed paul-carteron closed 1 year ago

paul-carteron commented 2 years ago

From : Milo Monnier

Au final ce qui prend le plus de temps dans mon script c'est sa reprojection en Lambert93 pour l'afficher avec mes autres données. Le GetCapabilities des WMS ortho proposant beaucoup de CRS différents on pourrait peut-être passer cette étape de reprojection en demandant le flux WMS directement dans le bon système, qu'en penses-tu ? En rajoutant un paramètre "crs" à la fonction contruct_urls() et en modifiant cette ligne https://github.com/paul-carteron/happign/blob/997113ab1a90f5493a18fe4aa9734e000976022a/R/get_wms_raster.R#L216

paul-carteron commented 2 years ago

Pour les coordonnées, c'est possible d'implémenter un paramètre CRS comme tu l'indiques. Le choix d'utiliser des CRS fixe c'était pour éviter de faire deux requêtes à chaque téléchargement (une getCapabilities et une getMap). De plus, ajouter un paramètre CRS modifiera la bbox de la requête. Je me note ça et je vais voir si c'est implémentable sans trop de pb ! merci pour la remarque ;)

paul-carteron commented 1 year ago

crs parameter is now implemented in get_wms_raster().