Skip to content

[FEAT] Ajoute vue vers le SINP National#238

Open
k-patrinat wants to merge 27 commits intoPnX-SI:masterfrom
k-patrinat:vue_export_inpn
Open

[FEAT] Ajoute vue vers le SINP National#238
k-patrinat wants to merge 27 commits intoPnX-SI:masterfrom
k-patrinat:vue_export_inpn

Conversation

@k-patrinat
Copy link

Ajoute un script pour les transmissions vers le SINP National et une note d'utilisation détaillant le contenu de la requête et les limites

 Ajoute un script pour les transmissions vers le SINP National et une note d'utilisation détaillant le contenu de la requête et les limites
@camillemonchicourt
Copy link
Member

OK super, merci pour cette contribution.
Si je comprends bien, cette vue permet de prendre les données dans une Synthèse de GeoNature actuellement en V2 sur SOT et l'exporter au mieux selon la V3 du SOT.
Surement à préciser dans le nom de la vue et de l'export, car on a déjà un export de base v_synthese_sinp.
Je nommerai bien celle-ci v_synthese_sinp_sot_v2tov3 ou un truc du genre.

A voir aussi si on l'ajoute à tous dans une migration ou si on l'indique dans la doc pour ceux qui veulent.
Comme c'est une vue temporaire pour proposer de l'export SOT V3 alors que GeoNature est encore en V2, je l'indiquerai pour ceux qui veulent l'utiliser, sans l'ajouter à tous dans une migration Alembic, mais à discuter.

@k-patrinat
Copy link
Author

OK super, merci pour cette contribution. Si je comprends bien, cette vue permet de prendre les données dans une Synthèse de GeoNature actuellement en V2 sur SOT et l'exporter au mieux selon la V3 du SOT. Surement à préciser dans le nom de la vue et de l'export, car on a déjà un export de base v_synthese_sinp. Je nommerai bien celle-ci v_synthese_sinp_sot_v2tov3 ou un truc du genre.

A voir aussi si on l'ajoute à tous dans une migration ou si on l'indique dans la doc pour ceux qui veulent. Comme c'est une vue temporaire pour proposer de l'export SOT V3 alors que GeoNature est encore en V2, je l'indiquerai pour ceux qui veulent l'utiliser, sans l'ajouter à tous dans une migration Alembic, mais à discuter.

Je suis d'accord pour la renommer, d'autant plus que c'est bien une vue temporaire et qu'il faut l'identifier comme telle. Ta proposition me convient.
Pour la diffusion dans les Geonature, je te laisse juge, Après réflexion, c'est peut-être plus adapté à une diffusion restreinte. Cela dépend aussi du besoin dans la communauté et de l'avancement du passage de Geonature au SOTv3.

Copy link

@ahurabielle ahurabielle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci pour la proposition de vue. Cela me paraît vraiment souhaitable d'uniformiser nos pratiques.
Je préconise vraiment l'ajout du champ date_validation


Il a été décidé, dans la mesure où la synthèse ne comporte qu'un seul identifiant unique pour une donnée (le second
étant lié à un type de regroupement) que les trois niveaux `Evenement`, `SujetObservation` et `Descriptif` récupèrerait
ce même identifiant pour permettre une traçabilité de la donnée sur ces trois niveaux.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Autant, je comprends ce choix pour SujetObservation et Descriptif vu les précédentes discussions / choix de traduction par les organismes.
Autant, je trouve cela dommage pour Evenement car en pratique, dans les modules OccTax et monitoring, ce qui est renseigné dans l'attribut unique_id_sinp_grp correspond en pratique au concept d'évenement.
On pourrait donc faire le choix de le récupérer s'il est non NULL ou le mettre à la valeur d'unique_id_sinp si NULL pour ne pas laisse ce champ vide.

@codecov
Copy link

codecov bot commented Dec 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (baeeee8) to head (ad9b649).

Additional details and impacted files
@@      Coverage Diff      @@
##   master   #238   +/-   ##
=============================
=============================

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

camillemonchicourt and others added 17 commits December 9, 2025 07:20
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
camillemonchicourt and others added 7 commits December 9, 2025 07:25
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md

Co-authored-by: ahurabielle <adele@hurabielle.com>
@PaulLabruyere
Copy link

PaulLabruyere commented Dec 15, 2025

Le PN-Pyrénées avait fait un travail de projection GN -> SOTv3 assez intéressant, cité dans le ticket GeoNature#3285

On y trouvait notamment une fonction pour faire la traduction entre les nomenclatures GN (possiblement modifiées/enrichies selon les instances) et la nomenclature standard SOTv3. Cette fonction, qui prend la valeur source et le champ cible en argument, permet de gérer les cas où plusieurs champs GN peuvent alimenter le même champ SOTv3 (plusieurs appels dans un coalesce() avec le même champ cible et différentes valeurs sources), ainsi que celui où un champ GN peut alimenter plusieurs champs SOTv3 selon sa valeur (appels à différents endroits de la vue avec la même valeur source et différents champs cibles)

L'objectif n'est pas forcément le même cela dit. On avait une vue par niveau attendu dans le nouveau standard plutôt qu'une projection à plat. On aurait donc une vue événement, une autre SujetObs, une troisième descSujet... et donc autant d'exports / d'API différentes.

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.

5 participants