Track GPU usage and temperature for nvidia gpus #183
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds tracking for nvidia gpus of usage and temps, it also renames the existing temperature to be CpuTemperature (I know it was mobo temp before but it was more or less the same as cpu, I also fixed that to properly read the temps for my system and presumably others with amd cpus as it didnt have "acpitz temp1".
It changes the icon for cpu usage and adds one for gpu usage, maybe not the best icon but couldn't find a more fitting one.
It uses nvidia-smi command line tool if available (afaik all nvidia gpu systems have this on linux by default with the drivers).
If you'd prefer it to use a proper rust crate I'm happy to get that working but all I could find added a lot of dependencies including C binding ones so needed gcc to build too etc, and I don't believe nvidia-smi has any performance worries at all, and it's a very stable api.
These seem like the best options for nvidia and amd that I could find. https://github.com/rust-nvml/nvml-wrapper https://github.com/Umio-Yasuno/libdrm-amdgpu-sys-rs