Skip to content

Update instructions to install SenseHat in HassOs #15503

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
denics wants to merge 9 commits into from
Closed

Update instructions to install SenseHat in HassOs #15503

Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
fc208f6
Update sensehat.markdown
denics Oct 31, 2020
f7bf8b5
Update sensehat.markdown
denics Oct 31, 2020
a718806
Update sensehat.markdown
denics Oct 31, 2020
e3c2dbe
Update sensehat.markdown
denics Oct 31, 2020
de43c66
Update sensehat.markdown
denics Oct 31, 2020
86540aa
Update sensehat.markdown
denics Oct 31, 2020
aa7c251
Update sensehat.markdown
denics Oct 31, 2020
7fc3acc
Update sensehat.markdown
denics Nov 3, 2020
7676db2
Update sensehat.markdown
denics Nov 3, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 30 additions & 1 deletion source/_integrations/sensehat.markdown
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,36 @@ group:
- group.sense_hat
```

### Directions for installing on Raspberry Pi Raspbian Based installation:
## Install on:

### Home Assistant Operating System

_SenseHAT_ is not going to work straing in Home Assistant Operating System. To enable it you can follow these steps.

- Follow instructions [here](/hassio/enable_i2c/) to enable i2c on Home Assistant Operating System
- Install "Custom deps deployment" add-on. To do so
- add ``https://github.com/home-assistant/hassio-addons-development`` to your add-ons repositories
- select *Custom deps deployment* and enable it
- Configure "Custom deps deployment" add-on
Copy link
Copy Markdown
Member

@frenck frenck Oct 31, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Custom deps is not recommend, we should not document that and/or writing instructions to use that.

Copy link
Copy Markdown
Contributor Author

@denics denics Oct 31, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can add a warning if you think it is better, but this is the only way to make the integration works. Otherwise, we can remove the integration as it cannot be installed.

Copy link
Copy Markdown
Member

@frenck frenck Nov 3, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A warning makes no sense.

If it doesn't work without these workarounds, I suggest filing an issue and get the problem fixed.

Copy link
Copy Markdown
Contributor Author

@denics denics Nov 3, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am filing an issue in "home-assistant/operating-system" to include support for RTIMUlib. Can we leave it as "workaround" ? or we will have change this text to something like:

You will not be able to use your SenseHat until issue 953 will not be solved.

What do you think?

Copy link
Copy Markdown
Member

@frenck frenck Nov 4, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we don't document issues.

Copy link
Copy Markdown

@AgalmicVentures AgalmicVentures Nov 8, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@frenck

Custom deps is not recommend, we should not document that and/or writing instructions to use that.
we don't document issues.

The larger issue with the documentation is listing SenseHat as a supported device when it's not.

No need to be curt with @denics for trying to address the problem. Three tickets have already been filed and closed as stale. He's proposing two reasonable solutions to make this a better experience for your users.

If you honestly don't want to make it work, that's fine -- in that case, please remove SenseHat from your documentation so others don't waste time like I (and @denics) did.

Copy link
Copy Markdown
Member

@frenck frenck Nov 8, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm sorry to hear about your issues. However, this is how it is. Feel free to contribute a fix @AgalmicVentures.

I'm fine with removing integrations if that is a better solution, especially if they are not maintained.

Copy link
Copy Markdown
Contributor Author

@denics denics Nov 10, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dear @frenck , the issue here is not with the SenseHat integration, I even made a PR to make the integration more universal and I am ready to add a workflow to it. The problem is that HassOS does not provide a required library. In my "solution" I am suggesting a workaround if HassOS is not integrating the required library. I opened a ticket on HassOS to point that out and you moved it to core (I do not understand why).

Copy link
Copy Markdown
Member

@frenck frenck Nov 11, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because integration defines the installation requirements, not the OS.

If it can be installed by the custom deps add-on, it can be handled by defining dependencies in the integration. Both do the same: Installing Python packages.

Thus this is an integration issue (core), not the OS.

Copy link
Copy Markdown

@QuantumStar QuantumStar Sep 21, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gee, I sure wish I'd found this thread six hours ago. I wasted a ridiculous amount of time, thinking "Surely if it's in the documentation, the SenseHAT must work?". I appreciate that this is all done by volunteers, but like others have said, it would be ideal if we could just figure out how to get it working again, but if not REMOVE THE DAMN THING FROM THE DOCUMENTATION or list it as deprecated, something to save people like me from wasting their time.


```yaml
Comment thread
denics marked this conversation as resolved.
pypi:
- >-
git+https://github.com/RPi-Distro/RTIMULib#egg=rtimulib&subdirectory=Linux/python
apk:
- gcc
- g++
- py3-sensehat
- i2c-tools-dev
- python3-dev
- raspberrypi-dev
- raspberrypi-libs
```

- Start the plugin and check the logs to make sure everything is fine. Set it to "Start at boot"
- Reboot and enjoy your lights

### Raspberry Pi Raspbian Based installation:

Here are the steps to make the _SenseHAT_ sensor work _successfully_ with the virtual environment versions.

Expand Down