search

PnX-SI / Nomenclature-api-module

Flask module for Nomenclature API
GNU General Public License v3.0
3 stars 10 forks source link

Feat/group3 inpn taxonomie #53

Closed mvergez closed 9 months ago

mvergez commented 1 year ago

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 :

Elle dépend de https://github.com/PnX-SI/TaxHub/pull/433 pour valider le groupe 3 Inpn (taxonomie.check_is_group3inpn)

camillemonchicourt commented 1 year ago

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 https://github.com/PnX-SI/GeoNature/issues/343

mvergez commented 1 year ago

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 commented 1 year ago

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 https://github.com/PnX-SI/TaxHub/pull/433

codecov[bot] commented 9 months ago

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 :warning:
src/pypnnomenclature/routes.py 66.66% 1 Missing :warning:
Additional details and impacted files ```diff @@ 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](https://app.codecov.io/gh/PnX-SI/Nomenclature-api-module/pull/53/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=PnX-SI) | Coverage Δ | | |---|---|---| | [pytest](https://app.codecov.io/gh/PnX-SI/Nomenclature-api-module/pull/53/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=PnX-SI) | `50.12% <50.00%> (+0.12%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=PnX-SI#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.