search

ARPA-SIMC / arkimet

A set of tools to organize, archive and distribute data files.
Other
15 stars 5 forks source link

Avoid grib1 scanning failure when bottomLevel is MISSING #330

Closed dcesari closed 5 months ago

dcesari commented 6 months ago

Correzione molto poco pitonica per la issue #326, non mi offendo se la migliorate.

dcesari commented 5 months ago

Mi date un feedback su questa PR?

edigiacomo commented 5 months ago

Scusa @dcesari, me l'ero persa... Anche io avrei risolto così, però anche io ho il tuo stesso dubbio (cioè se impatta con la gestione attuale e futura dei missing), quindi chiedo a @spanezz conferma.

spanezz commented 5 months ago

La riprenderei su io aggiungendo il test case, cosí quando riscriviamo i test nel linguaggio del 2031 non ci perdiamo questo caso.

Per il resto, ho appena verificato e per i GRIB1, l1 e l2 sono codificati con un byte, quindi 255 mi sembra corretto come dato mancante.

spanezz commented 5 months ago

Fatto! Ho anche aggiunto un parametro di default opzionale a grib.get_long, come con dict.get

brancomat commented 5 months ago

PR integrata in arkimet v1.51-1, in build ora