etalab / schema-irve

TableSchema pour les Infrastructures de Recharge de Véhicules Electriques (IRVE)
10 stars 10 forks source link

Choisir une stratégie "identifiants des schémas" pour la mise en place du dynamique #26

Closed thbar closed 9 months ago

thbar commented 1 year ago

Je crée un ticket spécifique pour qu'on puisse clairement avancer là dessus, en lien avec :

Actuellement, en version 2.1.0, une ressource data gouv est marquée "ressource statique IRVE" dès lors que son schema est égal à etalab/schema-irve (car, si j'ai compris, le nom du repository où le schéma est déclaré est lui-même etalab/schema-irve).

Toutefois, la consolidation (https://github.com/etalab/notebooks/blob/master/irve-v2/consolidation-irve-v2.ipynb) brasse plus large, et il va falloir revoir cela pour pouvoir faire le tri entre le statique et le dynamique :

Je déduis une première tâche : il va falloir que la consolidation statique n'aille chercher que "par schéma", pour éviter de récupérer des jeux dynamiques au passage.

Ensuite, il faut préparer le merge de https://github.com/etalab/schema-irve/pull/21.

Je me demande si on peut maintenir une rétro-compatibilité de l'identifiant du schéma statique, pour conserver etalab/schema-irve. J'ai essayé de maintenir cette rétro-compatibilité ici :

Je déduis une deuxième tâche : confirmer avec @geoffreyaldebert que si on a le name schema-irve comme spécifié actuellement dans la PR, l'identifiant de schéma resterait si je comprends etalab/schema-irve.

Si c'est le cas, on pourrait aussi nommer le schéma dynamique schema-irve-dynamique (à modifier ici https://github.com/etalab/schema-irve/pull/21/files#diff-2621fa081e1f7d1d72b4097e61896f50e0de11f3cab99902b683725f90696e7aR3), ce qui donnerait un nom complet de etalab/schema-irve-dynamique.

En résumé, de cette façon:

Point à discuter avant la finalisation donc, poke @geoffreyaldebert !

geoffreyaldebert commented 1 year ago

Hello ! Du coup j'ai commenté ta PR actuel. En synthèse, c'est possible mais il faut faire quelques actions :

fchabouis commented 1 year ago

Après discussion avec @Benoit-MINT et @ChristinaLaumond, nous avons décidé d'avoir l'url existante etalab/schema-irve qui pointe vers le datapackage, puis etalab/schema-irve-statique et etalab/schema-irve-dynamique qui pointent vers les schémas enfants. Donc pas besoin de changer le nom du repo. L'article de loi sera adapté en ce sens, il devait être modifié de toutes les façons pour prendre en compte les données dynamiques.

thbar commented 1 year ago

etalab/schema-irve-statique

@fchabouis est-ce que ça peut être etalab/schema-irve à la place ? Pour des questions de rétro-compatibilité ?

fchabouis commented 1 year ago

Non, parce que etalab/schema-irve va pointer vers le datapackage qui sera à la racine des deux schémas. Donc ça ne va pas completement casser l'url, ça amenera vers la page qui liste les 2 schémas à la place.

thbar commented 9 months ago

Nous avons au final procédé au changement, et suivi ce qui est dans le commentaire de Francis plus haut

nous avons décidé d'avoir l'url existante etalab/schema-irve qui pointe vers le datapackage, puis etalab/schema-irve-statique et etalab/schema-irve-dynamique qui pointent vers les schémas enfants. Donc pas besoin de changer le nom du repo.

Vérifications effectuées ce jour:

Reste la question de l'adaptation du texte de loi - je ne sais pas si ça a été fait ou pas, donc j'ai lancé une discussion Mattermost avec @Benoit-MINT, je conserve donc ce ticket ouvert pour le moment.

L'article de loi sera adapté en ce sens, il devait être modifié de toutes les façons pour prendre en compte les données dynamiques.

thbar commented 9 months ago

Vu avec @Benoit-MINT, tout est bon sur le plan règlementaire, on peut clôturer (et avec joie).

thbar commented 9 months ago

Merci @geoffreyaldebert @fchabouis sur ces sujets ! 🥳