[BUG] App keeps scanning and fields are not available

alexVinarskis / dell-powermanager

Cross-Platform Dell Power Manager re-implementation

GNU General Public License v3.0

47 stars 1 forks source link

[BUG] App keeps scanning and fields are not available #22

Closed Scridgeon closed 8 months ago

Scridgeon commented 9 months ago

All of the sections under Battery Settings and Thermal Management have a blue scanning bar that goes across them. The scanning persists and the fields never become available to use.

To Reproduce: Steps to reproduce the behavior:

Open App
Go To Battery Settings section
Blue scanning indicator constantly scrolls over fields on the right.

Expected behavior: The fields should be available to use when entering that section of the app.

Device Info:

OS: [6.5.0-13-generic, Ubuntu 23.10]
Platform [Dell Inc. XPS 13 9360]
App Version [0.8.0+20231126-193826]

alexVinarskis commented 9 months ago

Hi, thanks for the report. It seems like app cannot get response from Dell's CCTK. A few things to check:

I assume you have, but just to make sure: have you installed all of the dependencies? Upon start of the app, bottom left corner would have yellow warning indicator that CCTK is missing, and for ubuntu it can be installed automatically
Can you run Dell's CCTK manually? If installed via integrated installer, path would be sudo /opt/dell/dcc/cctk, and it shall be run as root/sudo. Sample output:
```
$ sudo /opt/dell/dcc/cctk --version
```


* Could you please run app via terminal, and attach the log file? via `dell-powermanager > log.txt`. Fyi log will contain all battery info as seen on the first page, including battery's serial number, in case you would like to redact it

Scridgeon commented 9 months ago

cctk: Dell Command Configure Version 4.11.0 6 (Linux - Aug 9 2023, 10:29:26)

log file is attached. log.txt

alexVinarskis commented 9 months ago

Seems problem is that your machine does not support ThermalManagement:

$ sudo /opt/dell/dcc/cctk --ThermalManagement --PrimaryBattChargeCfg

The option 'ThermalManagement' is not available or cannot be configured
through this tool.

App should've detect this, and show appropriate message, but it was not implemented yet. On bright side your machine may still support PrimaryBattChargeCfg, but due to combined command it does not show it, ill have to re-implement that.

Could you please provide a complete output of these commands: sudo /opt/dell/dcc/cctk -H sudo /opt/dell/dcc/cctk -H --ThermalManagement sudo /opt/dell/dcc/cctk -H --PrimaryBattChargeCfg

This should help me understand output format from unsupported device, and adjust app's logic accordingly

Scridgeon commented 9 months ago

cctk -H : cctk.txt

cctk -H --ThermalManagement : cctk_thermal.txt

cctk -H --PrimaryBattChargeCfg : cctk_battcfg.txt

alexVinarskis commented 8 months ago

fixed by https://github.com/alexVinarskis/dell-powermanager/pull/26