Skip to content
This repository has been archived by the owner on Feb 10, 2022. It is now read-only.

Liste des données validables #31

Closed
camillemonchicourt opened this issue Jun 6, 2019 · 5 comments
Closed

Liste des données validables #31

camillemonchicourt opened this issue Jun 6, 2019 · 5 comments

Comments

@camillemonchicourt
Copy link
Member

Actuellement il semblerait que le module ne liste que les données qui ont un enregistrement correspondant dans gn_commons.t_validations, donc principalement les données ayant été saisies dans Occtax, qui stockent une valeurs par défaut lors de leur insertion.

Par contre pour toutes les autres données insérées dans la Synthèse directement, elles ne remontent pas dans le module Validation. Hors, cela devrait être le cas, à partir du moment où elles ont un UUID_SINP.

A voir aussi si on prend en compte un champ validable au niveau des sources ou JDD ?

@camillemonchicourt
Copy link
Member Author

En lien avec #16

@camillemonchicourt
Copy link
Member Author

Le module liste désormais toutes les données présentes dans la Synthèse qui ont un UUID_SINP, pas seulement celles qui ont un enregistrement dans t_validations.
Fait dans GeoNature (PnX-SI/GeoNature@f426557), en même temps que d'autres compléments comme l'ajout des filtres géographiques sur la carto, la prise en compte des nomenclatures actives uniquement...

@DonovanMaillard
Copy link

DonovanMaillard commented Jun 18, 2019

Sans avoir un avis bien tranché (mais camille était intéressé pour un retour ici), le paramètre validable ou non ne me choquait pas dans la t_sources. En tous cas, il ne me semble pas plus adapté au niveau du jdd...

La t_source est technique et est propre à chaque instance, comme le choix de pouvoir valider ou non les données. La t_datasets a plutôt vocation a stocker les infos sur l'acquisition des données en elles-mêmes, mais surtout à contenir des infos qui sont échangeables et pas propres à une instance.

Du coup, pas d'avis tranché car globalement, le mettre au niveau du dataset ne pose pas de soucis particuliers. Mais niveau mcd, je trouvais plus cohérent d'avoir cette info au niveau de la t_source...

Reste que la t_source n'est pas forcément remplie dans les imports manuels (rendre l'id_source obligatoire?). Le module d'imports rempli la t_source pour sa part, et on peut envisager un booléen au moment de l'import pour définir si les données sont validables ou non. Mais en soit, avec le fonctionnement actuel ca se choisit de la même manière dans le module MTD....

@sig-pnrnm
Copy link

mais camille était intéressé pour un retour ici

Bien que commanditaire du développement initial, je dois vous avouer que je ne maitrise pas suffisamment les tenants et aboutissants du MCD pour avoir un avis sur la question (si jamais un avis de ma part était attendu 😉 )

@camillemonchicourt
Copy link
Member Author

camillemonchicourt commented Jun 26, 2019

Jointure sur les données de la synthèse et remontée de toutes celles qui ont un UUID_SINP, sans forcément déjà avoir d’enregistrement dans gn_commons.t_validations.

validable dans meta.t_datasets : PnX-SI/GeoNature@ebd4f7a

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants