pnx-si / gn_module_suivi_habitat_territoire Goto Github PK

En étant logué en tant qu'admin, lorsqu'on veut modifier seulement le mois d'une date d'une visite nos obtenons un message d'erreur. Ce message indique que la maille est déjà attribuée pour cette année.

Il faudrait éviter de prendre en compte la visite en cours de modification lors de la vérification de la présence d'une visite pour l'année correspondant aux champ date de visite.

Dans les datatables, les boutons d'action sont à droite. Or, lorsque le menu de GeoNature est ouvert, ces boutons se retrouvent cachés et inaccessibles.

• Il serait intéressant de les basculer sur les colonnes les plus à gauche sur le principe du module Suivi Flore Territoire.
• Il faudrait aussi utiliser les mêmes icônes et couleurs de bouton.

Lors de l'ajout d'une nouvelle visite, le champ "rechercher" en haut de liste des observateurs ne semble pas fonctionner (ni dynamiquement ni avec entrée).

Le contenu détaillé d'une maille ne se met pas à jour, exemple :
Consultation des sites / commune Villaroger -> 3 mailles avec dernière visite en 2021.
Mais lorsqu'on clique sur la maille -> "aucune visite sur ce site" dans informations générales ni détails.

Le module SHT utilise le plugin Leaflet.Deflate pour améliorer l'affichage des points sur la carte quelque soit le niveau de zoom. Or avec la version 2.3 de GeoNature (Angular 7) ce plugin en version 1.3.0 provoque l'erreur suivante :

ERROR in ../external_modules/sht/frontend/node_modules/Leaflet.Deflate/types/index.d.ts(14,34): error TS2694: Namespace '"/home/jpm/workspace/geonature/web/geonature/frontend/node_modules/@types/leaflet.markercluster/index"' has no exported member 'MarkerClusterGroupOptions'.

Les dépendances définies dans le package.json étaient:

"dependencies": {
    "@types/leaflet": "^1.4.3",
    "@types/leaflet.markercluster": "^1.4.0",
    "leaflet": "~1.3.1",
    "leaflet.markercluster": "^1.4.1",
    "Leaflet.Deflate": "^1.3.0"
  }

L'installation de "@types/leaflet" et "@types/leaflet.markercluster" suppriment des erreurs liées à TypeScript mais il reste malgré tout l'erreur ci-dessus.

La seule solution supprimant l'erreur et l'ajout directement dans GeoNature du plugin Leaflet.Deflate à l'aide de la commande: npm install Leaflet.Deflate

Nous allons ajouter le plugin comme dépendance de GeoNature en attendant de trouver une meilleur solution au problème ci-dessus.

Le filtre est à ordonner par année.

Dans la fenêtre modale contenant le formulaire de "Saisie d'un relevé", la liste des "Espèces présentes" n'est pas triées alphabétiquement. Il faudrait afficher les noms des taxons classé alphabétiquement par ordre croissant.
Pour le moment, les taxons sont renvoyés dans l'ordre d'insertion dans la table pr_monitoring_habitat_territory.cor_habitat_taxon.

Lors de l'export des données d'une maille en csv (mais pas vérifié si ça le faisait aussi pour les autres formats), la ligne correspondante est dupliquée à l'identique 3 fois.

Il semble que la création de référentiel géographique pour les mailles M50m et M100m soit inutile pour plusieurs raisons:

• il semble impossible d'ajouter l'ensemble des mailles de 50m ou 100m sur l'ensemble du territoire considéré dans le projet Flore Sentinelle par exemple (plusieurs départements). Cela représente déjà plusieurs millions de géométrie à stocker dans la table ref_geo.l_areas pour un seul département...
• dans SHT, les mailles correspondent à des sites, la géométrie de la maille est donc sotckée dans la table gn_monitoring.t_base_sites. La stocké à nouveau dans ref_geo.l_areas fait doublon.
• les mailles ne sont pas le seul format des sites SHT, pour l'habitat des messicoles c'est un transect (ligne) qui est utilisé. Dans ce cas là, il est inutile de la stocker dans le référentiel géographique.
• Les mailles (M50m, M100m mais aussi M25m) ou les transects ne sont jamais réutilisés que cela soit dans ce module ou les autres modules Conservation.

Pour ces diverses raison, il semble nécessaire de:

• supprimer l'ajout des mailles du référentiel géo lors de l'intégration des données via les scripts d'import.
• s'assurer que le module n'utilise plus les mailles M50m et M100m depuis le référentiel géographique.

Les perturbations concernent uniquement celles que l'on constate sur le terrain (à préciser dans l'interface ?). Il faudrait aussi pouvoir associer les perturbations à des alertes ou un changement de coloration au niveau de l'interface (?).

Les Taxons sont tous dans les même colonnes il faut les avoir dans plusieurs colonnes

Il faut aussi un export sur l’écran MAP/list avec prise en compte des filtres

Issue générale afin de pouvoir discuter et échanger des documents (maquettes, CRs, etc) concernant la création du module Suivi Habitat Territoire.

Dans un premier temps, vous trouverez en pièce jointe le compte rendu de la réunion du 20 février ainsi que le MCD.

CR_NS_CBNA_200218-CM.pdf

Vincent P. (Natural Solutions)

A noter que @gildeluermoz a avancé sur la partie générique du MCD de suivi (tables en jaune) : https://github.com/PnX-SI/GeoNature/blob/develop/data/core/monitoring.sql

Et @amandine-sahl a commencé à travailler sur un backend/API pour cette même partie : https://github.com/PnCevennes/projet_suivis_backend_core

CCTP initial : http://geonature.fr/documents/cctp/2017-11-CCTP-Suivi-Territoire-Habitats.pdf

Lier les habitats sur la carte avec leur fiche descriptive à gauche, au click sur un habitat de la carte.

Lors d'une consultation, si on filtre à la commune (par exple) puis que l'on regarde les détails d'une maille puis que l'on souhaite revenir en arrière avec "retour à la liste des sites", le filtre précédemment appliqué est remis à 0. Il serait pratique qu'il reste actif pour éviter de le remettre à chaque fois.

La vue des exports des visites retournent plusieurs lignes par visite ce qui ne devrait pas être le cas. En outre, les géométrie retournées correspondent aux mailles 10km, 5km et 1km alors que cela devrait être une maille de 50m...

Il faudrait que le message d'erreur qui s'affiche lors de l'enregistrement d'une fiche indique le problème qui a réellement lieu plutôt qu'un message générique.
Par exemple, l'erreur User "10209" cannot update visit number 930 affiche "Une erreur est survenue lors de l'édition de votre relevé". Cela devrait afficher "Vous n'avez pas les droits de modification pour cette fiche.".
Idéalement, le bouton de modification devrait être désactivé et un tooltip devrait indiquer que l'utilisateur n'a pas les droits d'édition sur la fiche !

Lors de l’export des données, les mailles exportées ne correspondent pas aux mailles Combes à neiges (50m) et caricion (100m).

Problèmes rapporté par @jmg-cbna.

L'export des visites affiche autant de colonne que de taxons présents dans les habitats gérés dans le module. Avec l'augmentation du nombre d'habitat géré le nombre de colonnes peu fortement augmenter et rendre le fichier CSV difficile à importer dans un tableur.

Il faudrait voir si un autre format ne serait pas envisageable afin d'éviter cet éventuel problème.

Voir SFT : https://github.com/PnX-SI/gn_module_suivi_flore_territoire/blob/master/docs/import-data.rst

@LeaCBNA remonte le problème suivant:

Sur les premiers imports des passages sur les mailles combes à neige, il y a des mailles sur lesquelles Alopecurus gerardi (cd_nom = 81638) n'est pas coché. Alopecurus alpinus (cd_nom = 81614) est un de ses synonymes.
Apparemment, le changement de cd_nom n'a pas fonctionné : il y a des mailles dans lesquelles A. alpinus apparaît dans les "Taxons non compris dans la liste d'espèces caractéristiques" , alors que A. gerardi n'est pas coché dans la liste.

Bref, s'il y a moyen de cocher A. gerardi dans toutes les mailles où il y a A. alpinus ce serait top !

• Backend :
  • Remplacer l'utilisation d'id dans les paramètres de config par des codes
  • Revoir les chemins des web services pour qu'ils soient plus REST (voir PRIORITY_FLORA).
  • Mettre en place une gestion des droits d'accès aux web services avec retour de message d'erreur et gestion de l'appartenance pour chaque type d'action (voir PRIORITY_FLORA, dont le fichier models.py et la classe ZpCruvedAuth).
  • Définir s'il vaut mieux lever des Exception pour renvoyer des erreurs dans les web services (raise Forbidden("Vous n'avez pas les droits pour éditer cette AP")) ou gérer le retour d'erreur avec un code HTTP (return { "message": "No organisms found !" }, 404). Mettre à jour les web services en conséquence.
    • => Privilégier l'utilisation des Exceptions car elles permettent d'être lancé depuis n'importe quel niveau du code. Werkzeug définit autant d'exceptions que de code HTTP, nous pouvons donc ainsi contrôler parfaitement le code HTTP à retourner.
  • Reprendre entièrement la gestion des données d'exemple.
  • Mettre à jour l'impression écran du module dans la doc.
  • Créer une fichier Postman décrivant et permettant de tester les web services du module. L'ajouter au dossier doc/.
• Frontend :
  • Trier les colonnes contenant les taxons (noms scientifiques) par ordre alphabétique dans l'export des visites.
  • Ajouter des icones pour les champs des formulaires (filtres) ou du rendu des données.
  • Ajouter un fichier package.json pour qu'il contienne Leaflate.Deflate et @types/geojson (#21).
  • Renommer les noms des composants pour qu'ils ne contiennent pas le nom du module et soit plus significatif.
  • Revoir les chemins des accès aux différentes parties de l'application (voir PRIORITY_FLORA).
  • Ajouter un bouton aux cartes pour pouvoir recentrer sur la couche geojson courante.
  • Utiliser des composants utilisant Angular Material. Remplacer les datatables par leur équivalant Material.
  • Paginer les datatables côté serveur.
  • Vérifier que toutes les services et fichiers dans shared/ sont bien utilisés dans l'application.
  • Amélioration la gestion des paramètres de config à l'aide d'un token injecté.
  • #45

Lors d'un filtre (organisme, année de visite, commune, habitat) le filtre se fait sur le tableau de gauche et la carte se déplace sur le centroïde des observations correspondantes mais en conservant l'intégralité des points. Ce serait pratique que le filtre s'applique également sur la carte.

Install et comment administrer les données

Dans l'export devrait figurer le code INSEE des communes et non pas l'id présent dans la base.

Sur le site 334 (https://geonature.floresentinelle.fr/#/sht/listVisit/334), Sophie V. n'a pas réussi à ajouter une visite.
Un message d'erreur "Une erreur est survenu" s'affiche à chaque fois.

La visite de ce site indique aussi 10/16 taxons alors qu'en l'affichant, nous pouvons voir seulement 8/16 taxons.

Il faudrait ajouter des icônes devant les différents champs des différentes interface.
Unifier les icônes avec ceux utilisés dans Suivi Flore Territoire.

Est il pertinent d'exporter la géométrie des sites dans l'export ?
Ne peut on pas mettre une info plus pertinente ?

Actuellement, les observateurs associables aux visites sont gérés via une liste "Observateurs Flore Sentinelle" qui n'est modifiable que par l'interface d'UsersHub. À moins qu'il y ait une autre solution que je ne connaisse pas ?
Ceci pose problème car UsersHub n'est accessible qu'aux administrateurs. Du coup, les utilisateurs ne peuvent pas ajouter facilement de nouveaux observateurs, ils doivent passer par une demande à un administrateur.
Toutefois, avec l'ajout de la possibilité de création de compte depuis la page d'accueil de GeoNature, il est possible de demander aux observateurs que l'on souhaite ajouter de s'inscrire.

Il faudrait trouver une solution pour rendre accessible l'ajout de nouveaux observateurs.

La liste "Observateurs Flore Sentinelle" en question comprend en fait 2 groupes d'utilisateurs :

• les "Observateurs Flore Sentinelle" : personnes n'ayant pas créé de compte et qui n'auront pas à le faire (=> sinon doublon avec la liste "Utilisateurs Flore Sentinelle").
• les "Utilisateurs Flore Sentinelle" : personnes ayant créé un compte sur Flore Sentinelle.
  Ainsi, tous les nouveaux utilisateurs sont automatiquement ajoutés a cette liste.

Pour une meilleure vue d'ensemble, ou peut-être modifier un peu les couleurs de maille, car la différence entre les rouges des an+1, an+2, an+3 n'est pas très visible lorsque l'on n'a pas les différentes couleurs côte à côte.

Un des membres de la liste "Observateurs de Flore Sentinelle" qui comprend 2 groupes d'utilisateurs "Utilisateurs Flore Sentinelle" et "Observateurs Flore Sentinelle", contient l'utilisateur "Marie DAU...GNE". Ce compte inactif fait partie du groupe "Observateurs Flore Sentinelle" qui contient les comptes de personnes ajoutées via le mécanisme d'importation, par exemple.

Ce compte n'est pas sélectionnable au niveau des formulaires.