Skip to content

Add disk temperature sensor and DISK/TEMPERATURE theme fields - #1073

Open
jeffreyjacques wants to merge 1 commit into
mathoudebine:mainfrom
jeffreyjacques:feat/disk-temperature
Open

Add disk temperature sensor and DISK/TEMPERATURE theme fields#1073
jeffreyjacques wants to merge 1 commit into
mathoudebine:mainfrom
jeffreyjacques:feat/disk-temperature

Conversation

@jeffreyjacques

Copy link
Copy Markdown

Report the temperature (°C) of the drive backing "/" as a new stat, rendered via the existing TEXT / GRAPH / RADIAL theme widgets under STATS > DISK > TEMPERATURE.

  • sensors.Disk gains an abstract disk_temperature(); implemented in all backends: Python (Linux drivetemp/NVMe via sysfs hwmon, with root-drive matching and a cached last-good value for SSDs that answer SMART intermittently), LibreHardwareMonitor (Windows), and both stubs.
  • stats.py renders the DISK/TEMPERATURE section when present. The section is optional: themes without it are unaffected (backward compatible).
  • theme_example.yaml documents the new fields, defaulting to SHOW: False.

Where no sensor is available the value is NaN and the fields stay blank (on Linux, SATA drives need 'sudo modprobe drivetemp').

@mathoudebine

Copy link
Copy Markdown
Owner

Thanks! Can you add the new field in res/themes/default.yaml? This file is used to get default values for themes that do not have the new keys

Report the temperature (°C) of the drive backing "/" as a new stat,
rendered via the existing TEXT / GRAPH / RADIAL theme widgets under
STATS > DISK > TEMPERATURE.

- sensors.Disk gains an abstract disk_temperature(); implemented in all
  backends: Python (Linux drivetemp/NVMe via sysfs hwmon, with root-drive
  matching and a cached last-good value for SSDs that answer SMART
  intermittently), LibreHardwareMonitor (Windows), and both stubs.
- stats.py renders the DISK/TEMPERATURE section when present. The section
  is optional: themes without it are unaffected (backward compatible).
- theme_example.yaml documents the new fields, defaulting to SHOW: False.

Where no sensor is available the value is NaN and the fields stay blank
(on Linux, SATA drives need 'sudo modprobe drivetemp').
@jeffreyjacques

Copy link
Copy Markdown
Author

Thanks for the review! Added DISK.TEMPERATURE (TEXT/GRAPH/RADIAL, all SHOW: False) to res/themes/default.yaml in e858645, matching the three display types the feature renders. Let me know if you'd like any other changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants