-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ajout d'une page au guide "Comment préparer des données à l'ouverture / la circulation ?" #85
Conversation
@AntoineAugusti @mathildehg Je vous laisse me faire un premier retour. On peut en reparler dans la semaine si vous voulez. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👏 @geoffreyaldebert pour le boulot accompli !
Je t'ai mis quelques remarques et suggestions, mais rien de bloquant. En revanche je pense qu'il est nécessaire que quelqu'un de moins technique que nous deux relise pour vérifier que le contenu est compréhensible et utile.
Une suggestion d'ajout : indiquer que la mise en place de référentiels fait parti d'une stratégie de montée en qualité de la donnée. Mettre en place ne suffit pas : il faut diffuser, former, vérifier que les données produites intègrent ces référentiels et n'en dérivent pas (contrôle humain ou tests automatiques).
Pourras-tu passer tes changements dans un correcteur orthographique pour vérifier l'absence de typos ?
Merci @AntoineAugusti pour le temps pris pour me faire part de tes remarques ! j'ai pris en compte l'ensemble de tes points a priori dans mon dernier commit. J'attends les remarques de @mathildehg puis je le ferai relire par une ou deux personnes moins techniques. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bravo pour cette page très complète !
J'ai fait des suggestions de forme directement dans le texte. Quelques remarques sur le fond :
- Peut-on considérer le cadastre comme un référentiel ?
- La BAN n'est pas citée dans le SPD, j'imagine que c'est parce qu'on en parle plus bas dans la partie adresse ?
- Dans le tableau, il conviendrait de mettre le lien du jdd dans la colonne "titre du jeu de données" et mettre le lien de l'organisation datagouv dans la colonne "producteur". Je ne sais pas si vous avez un moyen rapide de le faire ? Sinon je peux le faire à la main :)
Encore bravo, cette page me semble très utile !
Merci @mathildehg pour tes commentaires. Je les ai tous pris en compte. Pour tes trois remarques :
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello,
Ca me parait très bien, c'est clair, même pour moi qui est un néophyte 👍
J'ai proposé des petites corrections de formes et d'ortographe à la marge.
Merci beaucoup
|
||
|
||
Comme explicité dans la page [Préparer le jeu de données - *Cas 2 - La structure du jeu de données ne correspond à aucun schéma de données existant*](/qualite/preparer-le-jeu-de-donnees/#cas-2-la-structure-du-jeu-de-donnees-ne-correspond-a-aucun-schema-de-donnees-existant), il est important d'intégrer dans vos jeux de données des données pivots relevant d'un référentiel. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Peut-être qu'un encart du style "ne pas confondre schéma et référentiel" peut être pertinent ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@geoffreyaldebert À voir si tu veux ajouter un tel encart
Co-authored-by: Antonin Garrone <agarrone@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci à @mathildehg et @agarrone pour vos retours. Je pense que le texte de @geoffreyaldebert est maintenant bon pour aller en production 🚀
En son absence je merge la PR et on pourra faire des ajustements mineurs après avoir diffusé le contenu à la rentrée
Ajout de la page "Lier les données à un référentiel" suite à une demande ADEME.
Référence à l'issue #52