search

dataforgoodfr / 12_bloom

24 stars 11 forks source link

[Backend] Fix coquille load_dim_zone_amp_from_csv.py #225

Closed marthevienne closed 1 month ago

marthevienne commented 1 month ago

benificiaries to beneficiaries ! Il faut repeupler la table dim_zone après correction.

marthevienne commented 1 month ago

@SebM42 la coquille a empêché que la colonne beneficiaries soit intégrée dans la table dim_zone donc pour moi, c'est du P1. Ça va bloquer votre travail en data.

SebM42 commented 1 month ago

ah oui en effet c est du P1 !

njouanin commented 1 month ago

c'est quoi le pb au juste ?

marthevienne commented 1 month ago

Il y a une coquille dans le code, il me semble que c'est toi qui l'avait écrit. Je peux m'en occuper mais il faudra repeupler la table dim_zones pour réintégrer le champ manquant beneficiaries

njouanin commented 1 month ago

OK, je regarde dès que possible.

njouanin commented 1 month ago

Bien vu pour la coquille, c'est corrigé. Par contre le traitement actuel ne prévoit pas la mise à jour ... il faut que je vois comment intégrer la correction dans la table dim_zone

njouanin commented 1 month ago

Correction proposée pour la table dim_zone:

update dim_zone set json_data = json_data || '{"beneficiaries": "NA"}';
update dim_zone set json_data = json_data || '{"beneficiaries": "GB, BE, NL"}' where id=711;
update dim_zone set json_data = json_data || '{"beneficiaries": "GB, BE, NL, DE"}' where id=712;
update dim_zone set json_data = json_data || '{"beneficiaries": "BE, NL, DE"}' where id=713;
update dim_zone set json_data = json_data || '{"beneficiaries": "BE, NL"}' where id=714;
update dim_zone set json_data = json_data || '{"beneficiaries": "ES"}' where id=715;
update dim_zone set json_data = json_data || '{"beneficiaries": "ES"}' where id=716;