Feat/group3 inpn taxonomie

[mvergez]

Dans le cadre d'une prestation pour l'Agence Régionale de la Biodiversité en île de France, il a été demandé la possibilité d'afficher différentes valeurs de nomenclature dans la saisie Occtax.

Cette PR :

• ajoute une colonne group3_inpn à ref_nomenclatures.cor_taxref_nomenclature et modifie la contrainte d'unicité pour inclure cette colonne
• permet de filtrer les nomenclatures suivant le groupe 3 Inpn en mettant à jour la vue SQL ref_nomenclatures.v_nomenclature_taxonomie via l'API Rest

Elle dépend de PnX-SI/TaxHub#433 pour valider le groupe 3 Inpn (taxonomie.check_is_group3inpn)

[camillemonchicourt]

OK par contre je m'interroge sur un truc.
On avait déjà du mal à fournir une base de nomenclatures associés aux bons group2_inpn ou règnes pour que tout le monde en bénéficie.
Et ce n'est pas complet dans les nomenclatures qu'on fournit.
Donc ajouter un group3_inpn à ce niveau veut dire que chacun va se faire ça localement et pas le partager, donc tout le monde va galérer à associer les nomenclatures aux bons groupes/règnes ?

Le plus intéressant serait surtout de compléter les associations entre nomenclatures et règnes/group2_inpn, car @gildeluermoz s'en était fait une grosse partie y a plus de 5 ans, mais depuis ça n'a pas bougé ni été complété par personne.

Toute contribution bienvenue sur le sujet je pense.
Et surtout ça serait très dommage que certains se le fassent dans leur BDD et n'en fassent pas profiter les autres.

Voir PnX-SI/GeoNature#343

[mvergez]

Merci beaucoup pour ton retour @camillemonchicourt

Entièrement d'accord avec toi, il faudrait pouvoir mettre en place un moyen pour que les personnes puissent partager ces associations. Peut-être une issue / PR pour ajouter ces contributions.

Je vais déjà demander à l'ARB ce qu'ils ont et s'ils peuvent le partager.

Dans tous les cas ce n'est pas l'objectif de cette PR

[mvergez]

Les tests ne fonctionnent pas car cette PR dépend notamment de la fonction taxonomie.check_is_group3inpn(text) développée dans la PR PnX-SI/TaxHub#433

[codecov]

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (202a579) 50.00% compared to head (a7a3c5e) 50.12%.

Files	Patch %	Lines
src/pypnnomenclature/repository.py	0.00%	3 Missing ⚠️
src/pypnnomenclature/routes.py	66.66%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop      #53      +/-   ##
===========================================
+ Coverage    50.00%   50.12%   +0.12%     
===========================================
  Files            9        9              
  Lines          392      399       +7     
===========================================
+ Hits           196      200       +4     
- Misses         196      199       +3     

Flag	Coverage Δ
pytest	50.12% <50.00%> (+0.12%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.