astro-pi / python-sense-hat

Source code for Sense HAT Python library
https://sense-hat.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
505 stars 253 forks source link

RTIMULib copy should be optional #88

Open teuteuguy opened 5 years ago

teuteuguy commented 5 years ago

Copying the RTIMULib to the home folder requires access to the home folder, which in some cases, ie. running as root can become complex.

This should be a setting on load of the lib, to choose whether or not to copy the RTIMULib file or not.

Event, be able to provide path to file directly.

XECDesign commented 5 years ago

In general, you should be guaranteed access to your home directory, if nowhere else. But yes, flexibility wouldn't hurt.

J-Hurtta commented 4 years ago

Indeed. I'm trying to use python-sense-hat on on a limited access case (Home Assistant on Hass.io).

As the RTIMULib is not listed as a dependency, it is not installed on my device and thus I cannot use my sense hat.

rzr commented 4 years ago

or at least requierement should be mentionned somewhere

Hints: https://github.com/astro-pi/python-sense-hat/issues/58#issuecomment-624958208 https://github.com/rzr/sense-hat-webthing/blob/master/requirements.txt