[BUG] Invalid temp values in database

[adripo]

Describe the bug
The temp values showed for one of my HDD is wrong. They are below 25C and this is not possible.
I debugged a little and found that the data in influxdb has invalid temp values of 0C. That changes the calculated median and generates wrong calculated values.

Expected behavior
Correct temp values

Screenshots

[adripo]

<style> </style>

2	2022-11-04T16:32:35.518660698Z	2022-11-05T16:32:35.518660698Z	2022-11-05T05:31:30Z	30	temp	temp
2	2022-11-04T16:32:35.518660698Z	2022-11-05T16:32:35.518660698Z	2022-11-05T05:32:30Z	0	temp	temp
2	2022-11-04T16:32:35.518660698Z	2022-11-05T16:32:35.518660698Z	2022-11-05T05:33:30Z	29	temp	temp
2	2022-11-04T16:32:35.518660698Z	2022-11-05T16:32:35.518660698Z	2022-11-05T05:34:30Z	29	temp	temp
2	2022-11-04T16:32:35.518660698Z	2022-11-05T16:32:35.518660698Z	2022-11-05T05:35:30Z	29	temp	temp
2	2022-11-04T16:32:35.518660698Z	2022-11-05T16:32:35.518660698Z	2022-11-05T05:36:30Z	29	temp	temp
2	2022-11-04T16:32:35.518660698Z	2022-11-05T16:32:35.518660698Z	2022-11-05T05:37:30Z	0	temp	temp
2	2022-11-04T16:32:35.518660698Z	2022-11-05T16:32:35.518660698Z	2022-11-05T05:38:30Z	29	temp	temp
2	2022-11-04T16:32:35.518660698Z	2022-11-05T16:32:35.518660698Z	2022-11-05T05:39:30Z	29	temp	temp
2	2022-11-04T16:32:35.518660698Z	2022-11-05T16:32:35.518660698Z	2022-11-05T05:40:30Z	29	temp	temp

[adripo]

I extracted the smartctl values for the temp history:

The 0 in influx are null in smartctl

  "ata_sct_temperature_history": {
    "version": 2,
    "sampling_period_minutes": 1,
    "logging_interval_minutes": 1,
    "temperature": {
      "op_limit_min": 5,
      "op_limit_max": 55,
      "limit_min": 5,
      "limit_max": 55
    },
    "size": 478,
    "index": 415,
    "table": [
      30,
      30,
      30,
      null,
      30,
      31,
      31,
      31,
      null,
      31,
      31,
      31,
      31,
      null,
      30,
      31,
      31,
      31,
      null,
      31,
      31,
      31,
      31,
      31,
      31,
      31,
      null,
      31,
      31,
      32,
      32,
      32,
      33,
      33,
      33,
      33,
      null,
      31,
      31,
      31,
      32,
      32,
      null,
      31,
      32,
      32,
      null,
      32,
      32,
      32,
      32,
      null,
      31,
      31,
      31,
      31,
      null,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      null,
      29,
      30,
      30,
      30,
      null,
      28,
      29,
      29,
      29,
      null,
      29,
      29,
      29,
      29,
      null,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      null,
      29,
      29,
      29,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      null,
      28,
      29,
      29,
      null,
      29,
      29,
      30,
      30,
      null,
      29,
      30,
      30,
      30,
      null,
      28,
      29,
      29,
      29,
      null,
      29,
      29,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      null,
      29,
      29,
      30,
      30,
      null,
      29,
      29,
      29,
      30,
      null,
      29,
      29,
      29,
      29,
      null,
      29,
      29,
      29,
      30,
      30,
      30,
      null,
      30,
      30,
      30,
      30,
      null,
      29,
      29,
      29,
      29,
      null,
      29,
      29,
      29,
      30,
      null,
      29,
      29,
      29,
      29,
      null,
      28,
      29,
      29,
      29,
      null,
      28,
      28,
      28,
      28,
      null,
      28,
      28,
      28,
      28,
      null,
      28,
      29,
      29,
      29,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      null,
      29,
      29,
      29,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      30,
      31,
      31,
      31,
      31,
      null,
      31,
      31,
      31,
      31,
      31,
      31,
      31,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      32,
      33,
      33,
      33,
      33,
      33,
      33,
      33,
      33,
      33,
      33,
      33,
      34,
      34,
      34,
      34,
      34,
      34,
      34,
      34,
      34,
      34,
      34,
      34,
      34,
      34,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      34,
      34,
      34,
      34,
      35,
      34,
      34,
      34,
      34,
      34,
      34,
      34,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      34,
      34,
      35,
      34,
      34,
      34,
      35,
      35,
      34,
      35,
      35,
      35,
      35,
      34,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      35,
      34,
      34,
      34,
      34,
      34,
      35,
      35,
      35,
      35,
      34,
      35,
      34,
      34,
      35,
      34,
      34,
      34,
      34,
      34,
      34,
      34,
      35,
      35,
      35,
      35,
      35,
      35,
      34,
      34,
      34,
      34,
      34,
      34,
      null,
      28,
      28,
      29,
      29,
      null,
      28,
      29,
      29,
      29,
      null,
      28
    ]
  },

[AnalogJ]

Definitely a weird issue. Thanks for bringing it to my attention. I've added a PR to skip the invalid temperature readings.

[adripo]

@AnalogJ Thanks, I checked and it solves the issue. I suggested a little change in your PR.

[posta246]

Hi, my webmonitor says temp is "infinite", even if smartctl shows a value (previous version of scrutiny show the right one). How can give you more info? Another problem, maybe related: temp history is not recorded, the graph only includes values obtained when container stop and start...