pcm-dpc / COVID-19

COVID-19 Italia - Monitoraggio situazione
Other
3.87k stars 2.25k forks source link

"sigla_provincia" Napoli assente #863

Closed RiccardoBorchi closed 2 years ago

RiccardoBorchi commented 3 years ago

Tipo di issue:

Riassunto

La provincia Napoli non ha valorizzato il campo "sigla_provincia" nel file dpc-covid19-ita-province.json.

RiccardoBorchi commented 3 years ago

Il campo "sigla_provincia" per Napoli è ancora null, non potete valorizzarlo a "NA"?

miccoli commented 3 years ago

Mi sono accorto anche io dell'errore, dovuto probabilmente ad un parsing errato dei file .csv per la conversione in formato JSON. Infatti molti SW interpretano la stringa "NA" come "not available" da cui l'errore.

Ho aperto la PR #931 che risolve l'errore per il passato, ma fino a quando non si risolve l'errore nella procedura di conversione continuerà a ripresentarsi.

Sto mettendo in piedi una procedura di QA automatica: è possibile vedere i risultati in https://github.com/miccoli/COVID-19/actions?query=branch%3ACI++, il codice python è nella PR #847

miccoli commented 3 years ago

Vedi anche #755.

miccoli commented 3 years ago

@umbros Se può aiutare, l'errore è (ri)comparso nel commit 67f3d5b del 15 agosto: infatti:

$ git diff 67f3d5b~ 67f3d5b -- dati-json/dpc-covid19-ita-province.json | head -41
diff --git a/dati-json/dpc-covid19-ita-province.json b/dati-json/dpc-covid19-ita-province.json
index fd72951..6e15216 100644
--- a/dati-json/dpc-covid19-ita-province.json
+++ b/dati-json/dpc-covid19-ita-province.json
@@ -214,7 +214,7 @@
         "denominazione_regione": "Campania",
         "codice_provincia": 63,
         "denominazione_provincia": "Napoli",
-        "sigla_provincia": "NA",
+        "sigla_provincia": null,
         "lat": 40.83956555,
         "long": 14.25084984,
         "totale_casi": 0,
@@ -1878,7 +1878,7 @@
         "denominazione_regione": "Campania",
         "codice_provincia": 63,
         "denominazione_provincia": "Napoli",
-        "sigla_provincia": "NA",
+        "sigla_provincia": null,
         "lat": 40.83956555,
         "long": 14.25084984,
         "totale_casi": 0,
@@ -3542,7 +3542,7 @@
         "denominazione_regione": "Campania",
         "codice_provincia": 63,
         "denominazione_provincia": "Napoli",
-        "sigla_provincia": "NA",
+        "sigla_provincia": null,
         "lat": 40.83956555,
         "long": 14.25084984,
         "totale_casi": 0,
@@ -5206,7 +5206,7 @@
         "denominazione_regione": "Campania",
         "codice_provincia": 63,
         "denominazione_provincia": "Napoli",
-        "sigla_provincia": "NA",
+        "sigla_provincia": null,
         "lat": 40.83956555,
         "long": 14.25084984,
         "totale_casi": 3,

etc.

miccoli commented 3 years ago

È quasi un anno che questo issue è aperta, e l'errore (banale da correggere), è ancora presente:

{
  "data": "2021-09-19T17:00:00",
  "stato": "ITA",
  "codice_regione": 15,
  "denominazione_regione": "Campania",
  "codice_provincia": 63,
  "denominazione_provincia": "Napoli",
  "sigla_provincia": null,
  "lat": 40.83956555,
  "long": 14.25084984,
  "totale_casi": 268396,
  "note": null,
  "codice_nuts_1": "ITF",
  "codice_nuts_2": "ITF3",
  "codice_nuts_3": "ITF33"
}

possibile che sia così difficile produrre un JSON con "sigla_provincia": "NA" invece che null?

RiccardoBorchi commented 3 years ago

È quasi un anno che questo issue è aperta, e l'errore (banale da correggere), è ancora presente:

{
  "data": "2021-09-19T17:00:00",
  "stato": "ITA",
  "codice_regione": 15,
  "denominazione_regione": "Campania",
  "codice_provincia": 63,
  "denominazione_provincia": "Napoli",
  "sigla_provincia": null,
  "lat": 40.83956555,
  "long": 14.25084984,
  "totale_casi": 268396,
  "note": null,
  "codice_nuts_1": "ITF",
  "codice_nuts_2": "ITF3",
  "codice_nuts_3": "ITF33"
}

possibile che sia così difficile produrre un JSON con "sigla_provincia": "NA" invece che null?

È quello che mi chiedo anche io.

miccoli commented 2 years ago

Un aggiornamento: dopo alcuni tentennamenti, sembra che da 42c8cecf il problema sia risolto :tada:

Un grazie ai gestori della repo!

RiccardoBorchi commented 2 years ago

Un aggiornamento: dopo alcuni tentennamenti, sembra che da 42c8cec il problema sia risolto 🎉

Un grazie ai gestori della repo!

Finalmente, ottimo! 🎉