search

r-lidar-lab / ALSroads

Road corrections and measurements from ALS data
19 stars 4 forks source link

S'assure d'un CRS projeté pour les routes #13

Closed jfbourdon closed 2 years ago

jfbourdon commented 2 years ago

S'assure que la couche de chemin utilise un CRS projeté et non géographique car sinon la mesure de tolérance n'a plus de sens. J'ajoute la vérification, car si logiquement une couche de chemin a surtout du sens avec un CRS projeté, il m'est déjà arrivé d'en voir avec un CRS géographique. Le message suivant est affiché en cas d'erreur:

Error: Expecting a projected CRS for 'road'

J'ai aussi changé quelques messages d'avertissement utilisant m comme unité de distance pour que ce soit plutôt la valeur de st_crs(road)$units qui soit utilisé. Comme ça, le message sera cohérant si le CRS utilise des pieds comme unité.

Jean-Romain commented 2 years ago

Ha sinon je t'ai ajouté comme collaborateur. T'es pas obligé de travailler dans ton fork. Tu as les droits d'écriture sur ce depôt

jfbourdon commented 2 years ago

Ha sinon je t'ai ajouté comme collaborateur.

J'ai vu (et merci) et j'ai accepté, mais je ne me fais pas assez confiance. Puisque tu es le Créateur, tu es bien mieux placé que moi pour savoir si ma modif est sensée ou si elle aura des conséquences ailleurs dans le code. À moins que valider et merger mes PR te tanne, je vais continuer comme ça un bout.

Jean-Romain commented 2 years ago

J’étais sûr que tu répondrais ça. Tu peux faire des PR dans le dépôt principal si tu veux que je révise. Et tu peux pousser sans PR pour les petites bricole. Après c'est comme tu veux.