search

mathoudebine / turing-smart-screen-python

Unofficial Python system monitor and library for small IPS USB-C displays like Turing Smart Screen or XuanFang
GNU General Public License v3.0
1.17k stars 196 forks source link

Incorrect log message when configuring manually in *nix environments. #582 #582

Open nhomar opened 1 month ago

nhomar commented 1 month ago

The way it is bieng debugged wire the name of the package that is failing instead being explicitly from the python logger.

The improvement is manage the message properly to be more explicit instead wire the:

Now:

╰─$ python3 configure.py
[ERROR] Tkinter dependency not installed. Please follow troubleshooting page: https://github.com/mathoudebine/turing-smart-screen-python/wiki/Troubleshooting#all-os-tkinter-dependency-not-installed

Expected:

All the names of packages failing avoiding to repeat yourself foloowing a nicer pythonistic apporach.

This messaging is better to avoid depend (for example) from brew installation and be able to work with pyenv (or any virtualenv manager).

For example (MESSAGE CAN BE IMPROVED):

╰─$ python3 configure.py
[ERROR] Tkinter dependency not installed. Please follow troubleshooting page: https://github.com/mathoudebine/turing-smart-screen-python/wiki/Troubleshooting#all-os-tkinter-dependency-not-installed
No module named '_tkinter'[ERROR] Python dependencies not installed. Please follow start guide: https://github.com/mathoudebine/turing-smart-screen-python/wiki/System-monitor-:-how-to-start
No module named 'psutil'