Alkante / schema-dae

Schéma de validation de données pour Défibrillateur Automatique Externe
0 stars 2 forks source link

Correctifs en vue de la publication du schéma #1

Open johanricher opened 4 years ago

johanricher commented 4 years ago

Bonjour,

Bravo pour ce travail. J'imagine que la prochaine étape est la publication grand public de ce schéma ? Par exemple dans les catalogues de schéma Socle commun des données locales et schema.data.gouv.fr.

Les pré-requis pour publier sur schema.data.gouv.fr (https://schema.data.gouv.fr/documentation/validation-schemas) sont très similaires à ceux du SCDL (https://scdl.opendatafrance.net/docs/CONTRIBUTING.html), tout simplement car le premier a découlé du travail d'OpenDataFrance qui a été réalisé à l'occasion du projet Validata.

Les remarques suivantes sont donc générales et vous recevrez un retour similaire de la part d'Etalab si vous leur proposez le schéma :

La meilleure méthode pour tester la conformité d'un schéma reste de tester avec Validata en local pour avoir accès aux logs (docker-compose logs -f), ce qui est rendu facile grâce à Docker (https://git.opendatafrance.net/validata/validata-docker). Vous pouvez également tester le schéma en ligne avec la fonction "à la carte" de Validata (go.validata.fr), cela permet de s'assurer que tout fonctionne bien, comme le fait que la validation des fichiers d'exemple aboutisse, mais vous n'aurez pas accès aux logs. Une fois que vous êtes sûr que le schéma est correct et que les fichiers d'exemple passent la validation, vous pouvez faire une release.

Mon avis personnel :

Fonaine commented 4 years ago

Bonjour, et merci pour votre retour complet. Nous sommes en train de faire le nécessaire pour faire héberger le projet par une autorité publique. S'agissant des URL et du tag 1.0.0, ce sera corrigé lors du passage au dépôt public. S'agissant des fichiers non valides, c'était une volonté de notre part, pour donner des exemples non valides, en structure, comme en donnée, afin d'illustrer les vérifications faites par Validata. Mais, si je comprends bien votre remarque, ce n'est pas une bonne pratique dans le référencement du schéma ? Vous confirmez ? Si c'est le cas, nous les supprimerons. merci d'avance.

johanricher commented 4 years ago

Merci pour votre réponse.

Concernant les fichiers "non valides" je me suis mal exprimé ; je ne voulais pas dire qu'ils n'étaient pas conformes au schéma DAE. C'est en effet une bonne pratique, que vous avez suivie, de proposer des fichiers comportant des erreurs que pointera le validateur, à fin d'exemple à ne pas suivre pas les utilisateurs.

Je voulais signifier que ces fichiers ne semblent pas conformes au format CSV. En l'occurence ils ne peuvent même pas être chargés par Validata qui renvoie respectivement ces erreurs :


Vous pouvez constater ce problème si vous essayez de valider ces fichiers dans Validata, vous recevez cette erreur dans l'interface : image

Et dans les logs :

image

image

johanricher commented 4 years ago

Pour information, ces erreurs dans les fichiers CSV sont relevés par Github :

Le service de validation en ligne CSVLint vous informe également :

Fonaine commented 4 years ago

Les 2 fichiers incorrects ont été mis à jour.