search

r-map / rmap

rete monitoraggio ambientale partecipativo documentation at https://doc.rmap.cc
https://rmap.cc
53 stars 42 forks source link

Missing negative temperature data #329

Closed mbaldinetti closed 3 years ago

mbaldinetti commented 3 years ago

Missing negative temperature data

pat1 commented 3 years ago

avevo richiesto una pull request ... riapro per fare una revert , tag pull request etc

pat1 commented 3 years ago

f01cbab siamo in due in ARPAE a non comprendere il senso della patch le due variabili sono float e fare la differenza dei loro abs o l'abs della differenza dovrebbe essere abbastanza indifferente salvo forse situazioni di underflow e overflow @mbaldinetti sapresti spiegarci il senso della patch ?

pat1 commented 3 years ago

anzi l'ordine di esecuzione degli abs non è indifferente nel caso di temperature misurata e di conferma con segno differente, ma risulterebbe corretta la formula prima della patch e non dopo

mbaldinetti commented 3 years ago

dopo alcune verifiche confermo che la patch è errata. Il codice era già correttamente funzionante così senza bug delle temperature negative.