Skip to content

Feat/include nomenclatures in csv parser - #432

Draft
EliNoden wants to merge 7 commits into
mainfrom
feat/include-nomenclatures-in-csv-parser
Draft

Feat/include nomenclatures in csv parser#432
EliNoden wants to merge 7 commits into
mainfrom
feat/include-nomenclatures-in-csv-parser

Conversation

@EliNoden

@EliNoden EliNoden commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

🔎 Détails

Actuellement, les JSONschemas sont générés à partir des nomenclatures mais seules leurs valeurs restent sous forme d'enum.
Également, pour des nomenclatures plus permissives (code and label par exemple) on perd completement la notion de nomenclature. Pour notre travail sur le service specs, on souhaite avoir cette référence gardée dans nos schemas de sorte à ce que l'on puisse les récupérer à tout moment.

  • Fix des commandes de lancement du script csv_parser (il manquait des flags/arguments dans le readme et le arg parser)
  • Ajout d'un x-nomenclature dans les json schemas, qui viennent soit de la colonne "Détails de format" soit de la colonne "Nomenclature"
  • Modification du MDD pour ajouter une nouvelle colonne "Nomenclature" avant la colonne "CUT"

Validation

1. générer l'avant

On va d'abord générer un RC-EDA.old.schema.json pour qu'on puisse comparer les rendus.

  1. Se placer dans SAMU-Hub-Modeles/csv_parser
  2. Checkout le commit qui répare le script (mais n'ajoute pas encore le mécanisme de x-nomenclature)
git checkout a7c87eb3c09e249b28434607725393b5c6b76442

Lancer un csv_parser pour le RC-EDA, avec la commande :

python csv_parser.py -s RC-EDA -n RC-EDA -v 0.5 -m createCase -p models/model.xlsx -f 15-18

Puis renommer le fichier généré

mv ./out/RC-EDA/RC-EDA.schema.json ./out/RC-EDA/RC-EDA.old.schema.json

2. Générer l'après

Checkout sur la branche feat/include-nomenclatures-in-csv-parser (contient les 2 commits).

git checkout feat/include-nomenclatures-in-csv-parser

Relancer le parser :

python csv_parser.py -s RC-EDA -n RC-EDA -v 0.5 -m createCase -p models/model.xlsx -f 15-18
mv ./out/RC-EDA/RC-EDA.schema.json ./out/RC-EDA/RC-EDA.new.schema.json

3. Vérifier la différence

diff ./out/RC-EDA/RC-EDA.old.schema.json ./out/RC-EDA/RC-EDA.new.schema.json

Observer :

diff ./out/RC-EDA/RC-EDA.old.schema.json ./out/RC-EDA/RC-EDA.new.schema.json
178a179
>                     "x-nomenclature": "ISO 3166-ISO3166-2",
474a476
>                     "x-nomenclature": "HubSante.signalement",
663a666
>                     "x-nomenclature": "HubSante.nbVictimes",
679a683
>                     "x-nomenclature": "HubSante.patient",
716a721
>                     "x-nomenclature": "HubSante.idLieu",
945a951
>                     "x-nomenclature": "HubSante.systeme",
958a965
>                     "x-nomenclature": "HubSante.objetSource",
1089a1097
>                     "x-nomenclature": "HubSante.precision",
1152a1161
>                     "x-nomenclature": "HubSante.langue",
1345a1355
>                     "x-nomenclature": "HubSante.typeAppelant",
1384a1395
>                     "x-nomenclature": "HubSante.communication",
1539a1551
>                     "x-nomenclature": "HubSante.canal",
1555a1568
>                     "x-nomenclature": "HubSante.typeCom",

🔗 Ticket associé

Asana

@EliNoden
EliNoden force-pushed the feat/include-nomenclatures-in-csv-parser branch from 15f17aa to 9d80a4c Compare July 28, 2026 15:37
@EliNoden
EliNoden marked this pull request as draft July 30, 2026 09:19
@EliNoden
EliNoden force-pushed the feat/include-nomenclatures-in-csv-parser branch from 96da670 to 470a1d4 Compare July 30, 2026 13:51
@github-actions

Copy link
Copy Markdown

There is no coverage information present for the Files changed

Total Project Coverage 52.48% 🍏

@EliNoden EliNoden added the HOLD The branch is put on hold as it is waiting for something else label Jul 30, 2026
@EliNoden
EliNoden force-pushed the feat/include-nomenclatures-in-csv-parser branch 7 times, most recently from 6d5b8a5 to 173e35c Compare August 14, 2026 08:14
@EliNoden
EliNoden force-pushed the feat/include-nomenclatures-in-csv-parser branch 5 times, most recently from 3e8a2d3 to 4afd780 Compare August 14, 2026 12:00
@EliNoden
EliNoden force-pushed the feat/include-nomenclatures-in-csv-parser branch from 4afd780 to b7bc3ba Compare August 14, 2026 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

HOLD The branch is put on hold as it is waiting for something else

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants