Skip to content
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

First (not definitive) proposition of SCDL schema evolution #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gcarra
Copy link

@gcarra gcarra commented Sep 16, 2024

Salut @Pierlou,

j'ai fait les premières évolutions du format SCDL. Celles-ci ne sont pas définitives, en effet les entretiens avec les utilisateurs/producteurs sont encore en cours.

Je suis preneuse de tes éventuels retours et questionnements.
Je me demande également si on souhaite créer une branch pour travailler sur l'évolution et la pusher sur la master que quand l'évolution est bien validée ? Si oui, je n'ai pas la main sur le github OpenDataFrance, donc si tu peux la créer ça serait parfait.

Je reste à disposition,
Bien à toi
Giulia Carra

@Pierlou
Copy link
Collaborator

Pierlou commented Sep 17, 2024

Bonjour Giulia, très cool que ça avance ! Je regarde ça dans les prochains jours pour faire une review propre.
Tu as une préférence pour une branche plutôt que le fork actuel ? Ca ne me choque pas de rester comme ça en tout cas

Copy link
Collaborator

@Pierlou Pierlou left a comment

Choose a reason for hiding this comment

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

Ca prend forme 🚀

"homepage": "https://gitlab.com/opendatafrance/scdl/subventions",
"path": "https://gitlab.com/opendatafrance/scdl/subventions/raw/v2.1.1/schema.json",
"homepage": "https://github.com/OpenDataFrance/subventions",
"path": "https://github.com/OpenDataFrance/subventions/raw/v2.1.1/schema.json",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Il faudra changer le numéro de la version avant la release pour que ce soit cohérent

"role": "author"
},
{
"title": "Pierre Dittgen",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Je ne sais pas si on veut retirer les contributeurs de la première version du schéma, pour laisser les traces de l'évolution de la governance

Copy link
Author

Choose a reason for hiding this comment

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

ok, j'ai remis ça :)

{
"name": "idSubvention",
"title": "Identifiant de la subvention",
"description": "id interne d'identification d'une subvention permettant de faire le lien avec les actions et permettant de relier les informations sur les différents exercices budgetaire d'une subvention pluriannuel. En cas de financement pluriannuel, cet idéntifiant est le même pour chaque exercice budgetaire.",
Copy link
Collaborator

Choose a reason for hiding this comment

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

C'est un id unique par an du coup ?

Suggested change
"description": "id interne d'identification d'une subvention permettant de faire le lien avec les actions et permettant de relier les informations sur les différents exercices budgetaire d'une subvention pluriannuel. En cas de financement pluriannuel, cet idéntifiant est le même pour chaque exercice budgetaire.",
"description": "id interne d'identification d'une subvention permettant de faire le lien avec les actions et permettant de relier les informations sur les différents exercices budgétaires d'une subvention pluriannuelle. En cas de financement pluriannuel, cet identifiant est le même pour chaque exercice budgétaire.",

Copy link
Author

Choose a reason for hiding this comment

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

Oui c'est bien ça :)

@@ -81,7 +72,7 @@
{
"name": "idAttribuant",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Si c'est toujours un SIRET, peut-être renommer le champ siretAtribuant ?

Copy link
Author

Choose a reason for hiding this comment

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

j'ai fais des changements à ce sujet pour pouvoir y mettre également ridet et thaiti. J'ai ajouté du coup un attribut "type identifiant". J'espère te montrer les évolutions bientôt!

"example": "2017-06-27",
"name": "nomServiceInstructeur",
"title": "Nom du service instructeur",
"description": "Nom du service qui instruit la demande. Si possible, reprendre le titre du service tel qu'affiché sur l'annuaire des services publics. https://lannuaire.service-public.fr/gouvernement/c6ef7a55-69ea-4c15-b6c8-addcde7bdb63",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Pas sûr de comprendre pour l'URL est ici ?

Copy link
Author

Choose a reason for hiding this comment

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

Je pense que c'était pour essayer d'avoir une nomenclature claire et bien défini i.e. celle reportée annuaire services publiques. Mais je suis d'accord que nous n'avons pas besoin de spécifier l'URL ici .

"title": "Clé de versement",
"description": "Si un numéro d'Engagement Juridique n'existe pas renseigner l'identifiant permettant de relier la demande de subvention aux versements.",
"type": "string",
"constraints": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Pattern ?

Copy link
Author

Choose a reason for hiding this comment

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

pas te pattern unique

{
"name": "conditionsVersement",
"title": "Conditions de versement de la subvention",
"description": "Choix unique parmi plusieurs valeurs possibles : 'unique', 'échelonné' ou 'autre'. La valeur 'autre' correspond à une description libre des modalités de versement de la subvention dans la limite de 256 caractères maximum.",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ce n'est plus d'actualité ? La précision était bienvenue

Copy link
Author

Choose a reason for hiding this comment

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

oui, j'ai du faire une erreur ici. Mais j'ai laissé les trois valeurs : unique, échellonné, autre et ajouter un champs "descriptionConditionsVersements" si conditionsVersements = autre

"name": "cofinancement",
"title": "Cofinancement",
"description": "Indiquer si le projet est cofinancé. Valeurs possibles : 'oui' ou 'non'.",
"type": "string",
Copy link
Collaborator

Choose a reason for hiding this comment

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

booléen

"title": "Attribuants cofinanceurs du projet",
"description": "Liste des noms des attribuants cofinanceurs du projet. Si plusieurs attribuants, les séparer par des virgules.",
"type": "string",
"constraints": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

pattern (pour certifier la séparation, et préférer un autre séparateur (| par exemple), la virgule pouvant être problématique si c'est aussi le séparateur du fichier)

"title": "Identification du pojet cofinancé",
"description": "Identification unique pour un projet donné, commun à tous les cofinanceurs du projet",
"type": "string",
"constraints": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

pattern ?

Copy link
Author

Choose a reason for hiding this comment

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

on ne sais pas trop :/ En vrai pour le moment je pense que cet identifiant n'existe souvent pas mais je trouve que le fait de le demander permets de faire prendre conscience que cela n'existe pas et cela pourrait être utile.

@gcarra
Copy link
Author

gcarra commented Oct 1, 2024

@Pierlou merci beaucoup pour ce retour, je vais les prendre en compte attentivement et les intégrer aux avancements fait de notre côté aussi. Je reviens vers toi dès que j'aurai bien avancé là-dessus. Bonne après-midi!

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.

2 participants