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

Zones de sites d'individus #337

Open
wants to merge 8 commits into
base: dev
Choose a base branch
from

Conversation

xdidx
Copy link
Contributor

@xdidx xdidx commented Feb 7, 2022

Cette fonctionnalité correspond à l'ajout d'un nouveau type de programme (areas) permettant aux utilisateurs d'ajouter des zones, comportant des sites d'individus sur lequel on peut ajouter des observations. (cf #274)

Les observations peuvent être associées à des stades (ex: Feuillaison, floraison...etc) et chacun de ces stades peut comporter plusieurs étapes (ex pour la fleuraison: Aucune fleur n'est présente, 10% des fleurs, 50% des fleurs ou toutes les fleurs sont présentes)

J'en ai profité pour ajouter aussi quelques améliorations globales.

Même si c'est fonctionnel en l'état, il y a plusieurs améliorations possibles que j'ai déjà noté:

  • Vérifier si le programme est privé et s'il faut la connexion pour ajouter des éléments (area, species site et obs)
  • Vérifier si l'utilisateur a accès à la zone pour pouvoir ajouter un individu ou une observation
  • Faire la vérification de la possibilité d'ajout d'une étape coté API
  • Vérifier si un point est dans le polygone au clic sur la carte (par exemple en utilisant "turfjs booleanContains")
  • Faire hériter les nouveaux composants maps depuis la base

Edit: Ce n'est pas fonctionnel en l'état car je me basais sur Taxref et Refgeo

  • Enlever la ForeignKey en reference à Taxref pour cd_nom dans SpeciesSiteModel
  • Enlever la réference à LAreas pour municipality dans AreaModel

C'est une grosse fonctionnalité sur laquelle je travaille en temps partiel depuis quelques mois.
Je suis conscient qu'il aurait été plus pertinent de contribuer régulièrement plutôt que de faire une grosse PR aujourd'hui, j'espère cependant que les échanges et l'intégration de cette fonctionnalité n'en seront pas trop impactés.

@camillemonchicourt
Copy link
Member

OK intéressant.
C'est ce qui est déployé sur https://phenoclim.org/ ?
Cela semble assez spécifique à la flore et même à ce projet en l'état, mais si c'est bien mis à part, c'est en effet intéressant de l'intégrer dans GeoNature-citizen.

# Conflicts:
#	backend/gncitizen/core/commons/admin.py
#	backend/gncitizen/utils/init_data.py
#	frontend/src/app/programs/observations/form/form.component.ts
@xdidx
Copy link
Contributor Author

xdidx commented Feb 7, 2022

Effectivement, c'est lié à https://phenoclim.org/
Cependant, j'ai trié et enlevé pas mal de choses qui étaient spécifiques à Phénoclim. (il y a des éléments présent sur https://phenoclim.org/ qui ne sont pas dans cette PR)

On avait discuté un peu du lien à la flore, d'où les noms de species_sites qui permettent d'y associé aussi des sites lié à la faune, comme un nid.

@camillemonchicourt
Copy link
Member

OK super, merci pour les précisions.

@lpofredc
Copy link
Collaborator

lpofredc commented Mar 3, 2022

Analyse et intégration avec correctifs liés aux dernières évolutions du projet sur la branche wip-species-site-area

@xdidx
Copy link
Contributor Author

xdidx commented Aug 22, 2022

Bonjour !
Je reviens vers vous pour savoir quels seraient les conditions à réunir selon vous pour pouvoir faire l'intégration de cette fonctionnalité au sein de Geonature Citizen ?

@Lilya-NS
Copy link

Lilya-NS commented Aug 25, 2022

Bonjour, et merci @xdidx pour cette relance.

Je me permets d'identifier les plus grands contributeurs du projet (en dehors de Natural Solutions) pour avoir leurs retours à savoir @lpofredc @QuentinJouet @jolleon et @camillemonchicourt.

Vous pouvez observer la mise en place de zone sur le projet Phénoclim que nous avons réalisé pour CREA Mont-blanc ici : https://phenoclim.org/

Merci.

@camillemonchicourt
Copy link
Member

Je ne sais pas vraiment, il faudrait clarifier qui est mainteneur de ce projet.
Si @lpofredc a toujours le temps de maintenir ce projet ou non ?

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.

4 participants