jurkovic-nikola / OpenLinkHub

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

missing cpu device and to be able to name them #17

Closed iocc-git closed 2 months ago

iocc-git commented 2 months ago

Describe the bug

Not sure what version I used about 1-2 months ago but I miss to be able to name them. Because my fans isnt in the right order.

Also the cpu itself is missing now. Maybe it thinks "iCUE LINK H150i" is the cpu? Speed 0 rpm but change of color-theme changes all fans instead.

To Reproduce

installed OpenLinkHub-0.2.1-1.x86_64.rpm, no changes iCUE LINK System Hub iCUE LINK H150i iCUE LINK QX RGB fan iCUE LINK QX RGB fan iCUE LINK QX RGB fan stdout.log

Expected behavior

See describe the bug.

Screenshots

Screenshot_2024-09-19_22-22-37

Desktop (please complete the following information):

AlmaLinux 9.4 with 6.10.8 kernel 1.6.336 firefox-115

Additional context

Nope, but I'll add a feature report separately.

jurkovic-nikola commented 2 months ago

Click on Not Set bellow the device name. CPU device is on Dashboard page

jurkovic-nikola commented 2 months ago

Regarding 0 speed and no temperature, I can't reproduce that with same firmware levels on hub. Screenshot from 2024-09-20 07-02-02

You can try to set "refreshOnStart": true once for devices to be re-enumerated, but temps and speed from the pump should be reported back.

jurkovic-nikola commented 2 months ago

Also, replace the binary from this zip and run it for a 30-60 seconds. Send me stdout.log file after that. There was packet validation implemented in those releases, so debug data is needed. OpenLinkHub.zip

iocc-git commented 2 months ago

Hello.

Thanks, it wasnt obvious that "Not set" is something that you can click on and set.

Replaced OpenLinkHub binary and restarted. Still 0 speed and no temperature.

changed refreshOnStart to true and restarted. it clearly refreshed and the cpu unit now have speed and temp.

Here is about 10 mins of debug data. stdout.log

It still hasnt changed the order of the devices. I would suggest that the order would be:

iCUE LINK System Hub, iCUE LINK H150i CPU, iCUE LINK QX RGB FAN 1, iCUE LINK QX RGB FAN 2, iCUE LINK QX RGB FAN 3

I also think it should be included in the dashboard by default. Like the row below the Temperature Sensors. See screenshot...

Screenshot_2024-09-23_20-15-45 Screenshot_2024-09-23_20-16-03

jurkovic-nikola commented 2 months ago

Did you update package to 0.2.2 ? You should have arrows bellow the fan / pump icon to move. Try to clear browser cache.

image

jurkovic-nikola commented 2 months ago

If you have Discord, join. It will be much faster to explain where would you like to see CPU info. https://discord.gg/ywtSAerx

iocc-git commented 2 months ago

No, I didnt know it was a new version available. But now I have. Yes, it works after clearing browser cache.

Ok, I'll be there quite soon :P Screenshot_2024-09-23_20-40-43

jurkovic-nikola commented 2 months ago

Changes to dashboard index will be implemented either in 0.2.3 or 0.2.4.