Feat Synthese: add status, red lists filters

[amandine-sahl]

cf - #1492

Prise en compte des statuts fournis par l'INPN dans le cadre de la bdc_statut :

• filtres de la synthèse
• fiche détail de l'observation
• export des statuts des taxons sélectionnés dans la synthèse
• clean : suppression de la mention des anciens statuts

[codecov]

Codecov Report

Base: 67.65% // Head: 67.63% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (10c0d82) compared to base (12e3334).
Patch coverage: 98.33% of modified lines in pull request are covered.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2173      +/-   ##
===========================================
- Coverage    67.65%   67.63%   -0.02%     
===========================================
  Files           77       76       -1     
  Lines         6424     6411      -13     
===========================================
- Hits          4346     4336      -10     
+ Misses        2078     2075       -3     

Flag	Coverage Δ
pytest	67.63% <98.33%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
backend/geonature/core/gn_profiles/routes.py	79.36% <ø> (-0.17%)	⬇️
...nature/core/gn_synthese/utils/query_select_sqla.py	75.13% <96.66%> (+4.83%)	⬆️
backend/geonature/core/gn_profiles/models.py	98.96% <100.00%> (ø)
backend/geonature/core/gn_synthese/models.py	89.29% <100.00%> (ø)
backend/geonature/core/gn_synthese/routes.py	78.97% <100.00%> (+0.31%)	⬆️
backend/geonature/utils/config_schema.py	91.83% <100.00%> (+0.10%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[amandine-sahl]

Remplace les pulls requests (qui sont à fermer sans merge) :

• Feat Synthese: add advanced status, red lists filters #1535
• Feat frontend/synthese: add BDC status text infos to observation modal #1536
• Feat backend/synthese: use new BDC status tables for status export #1537
• Fix: remove all references to old status and red lists #1538

[bouttier]

J’ai l’impression qu’on ajoute beaucoup de jointures, je me demande l’impact sur les performances de la synthèse ? Est-ce qu’il est toujours nécessaire de récupérer toutes ces informations ? Elles figurent toutes dans la liste des observations ?

Par ailleurs, on refait les jointures dans SyntheseQuery pour pouvoir filtrer ; est-ce que la BDCS se retrouve jointe 2 fois ? Il me semble qu’il y a du code qui vise à éviter de rejoindre lorsque déjà join mais est-ce qu’il est mis en œuvre ici ? L’utilisation de l’ORM simplifierait ceci mais c’est un autre sujet !

[jpm-cbna]

J’ai l’impression qu’on ajoute beaucoup de jointures, je me demande l’impact sur les performances de la synthèse ? Est-ce qu’il est toujours nécessaire de récupérer toutes ces informations ? Elles figurent toutes dans la liste des observations ?

Par ailleurs, on refait les jointures dans SyntheseQuery pour pouvoir filtrer ; est-ce que la BDCS se retrouve jointe 2 fois ?

Alors, j'ai pas mal travaillé les performances et l'on arrive à quelque chose de fonctionnel même sur plusieurs millions de données.
Mais effectivement, cela rajoute des jointures car nous sommes obligé à la fois d'aller vérifier que le taxon de l'observation et bien lié aux textes de protection choisi mais aussi que l'observation est bien lié à la zone géographique d'application du texte...
Et pour éviter la collision avec les tables du ref_geo liés aux filtres géographiques de la Synthese (communes, départements...), nous sommes obligé d'utilisé des alias.

Je ne pense pas que la BDC soit lié plusieurs fois.