elParaguayo / qtile-widget-laptopbattery

Qtile widget to display laptop battery status
MIT License
8 stars 1 forks source link

Widget/Qtile crash when reaching critical battery level #1

Closed vhsdream closed 4 years ago

vhsdream commented 4 years ago

I was using Arch Linux with the 5.81 kernel and running Lenovo Thinkpad T440p with an Intel i7-4702MQ processor, and Qtile crashes/becomes unresponsive once the battery reaches a critical level. The widget either disappears from the bar or becomes black, and a reload of Qtile is when the crash occurs (I am using Qtile version 0.15.1).

I have confirmed that it is indeed your widget (which is awesome otherwise by the way) misbehaving when battery reaches critical level: when my battery was at 13% I edited the widget config to consider 15% a critical level. Upon reload, Qtile crashed and I had to drop to a tty in order to undo the changes.

After undoing the changes, I waited until my battery hit the default critical level (10%) - that is when the widget went totally black/or disappeared from the bar, and when I reloaded the config Qtile crashed again.

A simple mitigation for me is to not allow my battery to reach critical level and edit the config to consider critical to be something real low, like 2%.

I have just updated kernel to version 5.8.2 but have not tried to reproduce the issue. Nor have I tried updating to Qtile 0.16.1 to see if that resolves it. In the next few days I will try these things and report back.

If you need any more info please just ask (this is my first time posting an issue on GitHub!) and I will do my best to supply it.

And keep up the good work! I really like the design of this widget and I want to keep using it.

elParaguayo commented 4 years ago

Thanks so much for trying the widget out. Glad you like it and sorry it crashes.

I have no doubt that the fault lies with my widget so would be happy to try and fix this.

As a start, is there anything in your log about this? The file should be in ~/.local/share/qtile/qtile.log That may have an error message.

I can also try to let my battery get to the critical level to test it too!

vhsdream commented 4 years ago

No need! I identified a mistake in my config and have the issue resolved! I was using variables for my colors (0-9) and missed one word (should have said "colors[6]" and I had it "[6]"). Fixing that and testing has resolved everything. I'm attaching the log snippet for posterity and will close this issue. qtile-log-snippet.log