dsmrreader / dsmr-reader

DSMR-telegram reader & data visualizer for hobbyists. Free for non-commercial use.
https://dsmr-reader.readthedocs.io
Other
459 stars 95 forks source link

Maak data_retention_in_hours nauwkeuriger #1945

Closed RichieB2B closed 6 months ago

RichieB2B commented 6 months ago

I heb mijn retention settings op een jaar staan, maar was verbaasd dat de datagrammen van januari 2023 nu al niet meer allemaal beschikbaar zijn. Dit komt omdat in RetentionSettings de uren erg onnauwkeurig zijn gedefinieerd. Deze PR corrigeert dat.

Er is waarschijnlijk ook een migratie script nodig om de oude waardes in dsmr_datalogger_retentionsettings aan te passen:

672 -> 720
2016 -> 2160
4032 -> 4320
8064 -> 8760
dennissiemensma commented 6 months ago

Bedankt voor je suggestie. In dat geval zal ik het nog wat verder duwen naar de overdreven veilige kant: 31 dagen voor een maand en 366 dagen voor een jaar.

Het is immers puur ter voorkoming van de ophoping van de data. De beste oplossing zou zijn om timedelta te gebruiken, maar dat is wat complexer dus een bump van de waardes lijkt me prima.

dennissiemensma commented 6 months ago

Doorgevoerd in development