search

amanusk / s-tui

Terminal-based CPU stress and monitoring utility
https://amanusk.github.io/s-tui/
GNU General Public License v2.0
4.06k stars 140 forks source link

s-tui doesn't seem to work on armhf #102

Closed highvoltage closed 5 years ago

highvoltage commented 5 years ago

Step 1: Describe your environment

Step 2: Describe the problem:

Observed Results:

Forwarded from Debian bug report 914194:

I tried running s-tui on my Olimex OLinuXino LIME2 device, running Debian armhf.

It failed with a stacktrace:

Traceback (most recent call last): File "/usr/bin/s-tui", line 11, in load_entry_point('s-tui==0.8.2', 'console_scripts', 's-tui')() File "/usr/lib/python3/dist-packages/s_tui/s_tui.py", line 927, in main graph_controller = GraphController(args) File "/usr/lib/python3/dist-packages/s_tui/s_tui.py", line 728, in init self.view = GraphView(self) File "/usr/lib/python3/dist-packages/s_tui/s_tui.py", line 233, in init urwid.WidgetPlaceholder.init(self, self.main_window()) File "/usr/lib/python3/dist-packages/s_tui/s_tui.py", line 552, in main_window self.controller.temp_thresh) File "/usr/lib/python3/dist-packages/s_tui/Sources/TemperatureSource.py", line 54, in init self.update() File "/usr/lib/python3/dist-packages/s_tui/Sources/TemperatureSource.py", line 146, in update update_func(sensor, 0) File "/usr/lib/python3/dist-packages/s_tui/Sources/TemperatureSource.py", line 93, in update_func Source.update(self) File "/usr/lib/python3/dist-packages/s_tui/Sources/Source.py", line 28, in update self.eval_hooks() File "/usr/lib/python3/dist-packages/s_tui/Sources/Source.py", line 68, in eval_hooks if self.get_edge_triggered(): File "/usr/lib/python3/dist-packages/s_tui/Sources/TemperatureSource.py", line 190, in get_edge_triggered return self.last_temp > self.temp_thresh TypeError: '>' not supported between instances of 'float' and 'NoneType'

Step 3: Reproduce the problem:

Steps to reproduce:

  1. Install a Debian armhf instance
  2. apt install s-tui
  3. s-tui
amanusk commented 5 years ago

Hi, Thanks for opening the issue, and for porting s-tui to Debian! I believe the issue is supposed to be solved in the latest version (0.8.3),

amanusk commented 5 years ago

Seems this fixes the bug. Closing. Thanks @highvoltage