Dubbio sulla conversione da Lua number a integer in ValueBag

[arpa-simc]

Dubbio sulla conversione da Lua number a integer in ValueBag

[brancomat]

Ok, come si diceva i problemi sono due:

1. Se il troncamento sia un comportamento accettabile (rispetto ad es. all'arrotondamento) e questo mi pare al momento sia assodato: riguarda in realtà solo un subset degli odimh5, potrebbe essere rimesso in discussione ad esempio da eventuale introduzione dei netcdf e questo ci porta a...
2. Documentare la cosa. Sul dove però ho qualche dubbietto.

[spanezz]

ValueBag supporta solo valori interi e stringhe, quindi il troncamento c'è sempre stato. L'idea è, in fase di scan, scalare i valori; per esempio le latitudini non sono restituite in gradi ma in milionesimi di grado. Per esempio, in scan-grib2/01-common.lua:

area.latfirst = grib.latitudeOfFirstGridPointInDegrees * 1000000
area.lonfirst = grib.longitudeOfFirstGridPointInDegrees * 1000000

[edigiacomo]

Forse mi sono spiegato male: sono cosciente del fatto che il troncamento c'è sempre stato e che ValueBag gestisce solo stringhe e interi (ho scritto anche alcune parti in Lua dello scanner odim), il mio dubbio riguardava il passaggio e la gestione (documentata) di un valore non intero al ValueBag da Lua (troncamento, eccezione, arrotondamento, altro).

[spanezz]

Ah, scusa, avevo interpretato male il bug.

Lua non fa distinzione tra numeri interi e floating point, quindi per lanciare un'eccezione servirebbe un controllo su quanto piccola è la parte frazionaria di un numero, e come approccio non mi piace.

Considerando che chi scrive script di scansione spero guardi i risultati di quello che ha scansionato prima di metterli in produzione, propenderei per troncamento (perché mi aspetto che il codice Lua sia conscio dello star generando valori interi) e documentazione.

C'è documentazione relativa a questo aspetto di arkimet, al momento?

[edigiacomo]

Benissimo il troncamento. Per la documentazione, ho solo aggiunto una riga alla documentazione di ValueBag::load_lua_table in c9e282d (il messaggio di commit fa riferimento ad una issue diversa, è un refuso)

[edigiacomo]

Chiudo, per me come doc è sufficiente.