-
-
Notifications
You must be signed in to change notification settings - Fork 27
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.
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
Home Assistant Data Science no longer works via this add-on #634
Comments
See this issue for more details on what's broken: robmarkcole/HASS-data-detective#157 (comment)
**EDIT?UPDATE: ** Where does the example |
Note: this was previously reported back in August 2023, #491 - Example "GETTING STARTED.ipynb" is broken but closed by Frenck who mentioned:
"provided by the Home Assistant project" is a little ambiguous. It is clear the example isn't coming fro the Hass Data Detective repo. Nor does it come from https://github.com/home-assistant/data.home-assistant. After a bit of searching I finally found the home-assistant-notebooks repo under the home-assistant organization: https://github.com/home-assistant/home-assistant-notebooks The add-on does have the version of While that notebook was updated, I'm not sure it worked/was tested since it appears that HASS-Data-Detective and therefore the use of it in this add-on has been broken for at least 18 months and is probably due to a chance in SQLAlchemy starting early in 2.x. As discovered by others: db.perform_query() fails if you give it a string. |
I appreciate the detective work - sounds like a couple of PR are in order |
I'm not sure what you want me to do with this?
Cool, but you are in the wrong project for a comment on that expectation.
🤷 Feel free to provide an upstream fix for issues.
Agree. Let me put it this way: Not here :) I'm closing it on this end, as there is nothing I can do from this end to solve this issue. If upstream fixes are applied, I'm happy to ship/tag an updated release of this add-on. ../Frenck |
Problem/Motivation
Home Assistant Data Science which is linked from home-assistant.io page footers, it no longer works and appears to have not worked since spring of 2023. The instructions (Quick Start Guide) say to use this add-on to run https://github.com/robmarkcole/HASS-data-detective/. The example notebook,
GETTING_STARTED.ipynb
fails at the first DB query under "Popular entries"See robmarkcole/HASS-data-detective#157 and home-assistant/data.home-assistant#292 for bug reports others have opened.
Expected behavior
I would expect an example linked from the main home-assistant.io pages to work.
Actual behavior
Database query fails -- See robmarkcole/HASS-data-detective#157 and home-assistant/data.home-assistant#292 for bug reports others have opened.
Steps to reproduce
Follow Home Assistant Data Science Quick Start Guide say to use this add-on to run https://github.com/robmarkcole/HASS-data-detective/
Proposed changes
If Home Assistant Data Science via @robmarkcole's HASS-data-detective can still work with current versions of Home Assistant, please update this add-on to include current working modules.
If not consider:
Thank you.
The text was updated successfully, but these errors were encountered: