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

New OpenAir export format for aerial sensitive areas (#2372) #3349

Merged
merged 67 commits into from
Sep 28, 2023

Conversation

lpofredc
Copy link
Contributor

@lpofredc lpofredc commented Dec 7, 2022

Replace #2666

As mentionned in #2372, there are expectations for the OpenAir file format in the context of free flight activities. This merge request adds the ability to export sensitive zonings in the OpenAir format, either individually or for all sensitive areas.

This new feature stands on PyOpenair lib, specially developped for this.

@camillemonchicourt
Copy link
Member

Salut.
Il serait bien d'intégrer cette PR maintenant que les pré-requis au niveau de la libraire Python sont OK.
Mais il semble que cette PR a toujours une couverture de tests insuffisante pour pouvoir l'intégrer.

@lpofredc
Copy link
Contributor Author

Salut, Oui, je suis justement sur l'écriture des tests manquants.

@babastienne
Copy link
Member

babastienne commented Sep 19, 2023

Hello. Absolument, il manque des tests unitaires pour qu'on puisse valider la PR. Il faudrait tester l'API (un paramètre a été ajouté pour ne pas paginer les résultats), et les vues dans le module sensitivity.

Lorsque c'est bon ne pas hésiter à nous pinger pour qu'on regarde.

@lpofredc
Copy link
Contributor Author

Tests unitaires ajoutés, j'attends vos retours.

@camillemonchicourt
Copy link
Member

J'avais pas capté que ce développement ajoutait la dépendance shapely, ce qui n'est pas anodin.
En effet à ma connaissance c'est une librairie assez conséquente.
Cela pose la question d'ajouter une telle dépendance à tout Geotrek, uniquement pour gérer un format d'export particulier, seulement dans l'API et uniquement pour un module assez particulier (Zones sensibles) et une seule pratique (parapente).

Bon pourquoi pas, vu qu'on est parti sur cette évolution et qu'elle a été validée.
Mais il faudra se réserver la possibilité de désactiver ou enlever cette dépendance, si elle pose des soucis de mise à jour ou de maintenance de Geotrek-admin.

@submarcos submarcos merged commit 2f12183 into GeotrekCE:master Sep 28, 2023
15 checks passed
@lpofredc
Copy link
Contributor Author

Merci !

@babastienne babastienne changed the title New OpenAir export format for aerial sensitive areas New OpenAir export format for aerial sensitive areas #2372 Sep 28, 2023
@babastienne babastienne changed the title New OpenAir export format for aerial sensitive areas #2372 New OpenAir export format for aerial sensitive areas (#2372) Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants