jurkovic-nikola / OpenLinkHub

Open source interface for iCUE LINK Hub and other Corsair AIOs, Hubs for Linux
GNU General Public License v3.0
84 stars 7 forks source link

Add git tag argument to Dockerfile, update Readme.md #11

Closed kondas closed 3 months ago

jurkovic-nikola commented 3 months ago

Whats the reasoning behind pciutils and usbutils in docker file ? Those packages are not required to run this. Also, AMD temps are monitored via hwmon, and I dont think zenpower will work in such a way.

kondas commented 3 months ago

I've recieved an error indicating missing lspci command while trying out the latest version. I think its something regarding VGA detection. Usbutils, was just preventive measure so its indeed not necessarry.

I'm not using k10temp module to monitor AMD temperatures via hwmon, instead i'm using v2.0 of the zenpower module. It gives more detailed information:

zenpower-pci-00c3
Adapter: PCI adapter
SVI2_Core:   925.00 mV
SVI2_SoC:    994.00 mV
Tdie:         +39.4°C  (high = +95.0°C)
Tctl:         +39.4°C
Tccd1:        +40.5°C
SVI2_P_Core:   3.07 W
SVI2_P_SoC:    3.51 W
SVI2_C_Core:   3.29 A
SVI2_C_SoC:    3.53 A

Since using k10temp in the configuration did not yield any temperature values, i've provided zenpower in the config instead, and it is working properly. Althrough possibly not intentional then, but it works:

screenshot
jurkovic-nikola commented 3 months ago

Okay, that's good. Software will try to iterate thru hwmon with value you specify in config file. If value is left empty, it will iterate over k10temp or coretemp for CPUs. Good to know its actually integrated in hwmon.