-
Notifications
You must be signed in to change notification settings - Fork 13
Add Entity Diagnostics Dashboard with Smart Suggestions #105
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
base: master
Are you sure you want to change the base?
Conversation
pnbruckner
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like some previous modifications that were made have been backed out. Can you explain why this has happened? Also, not sure why you changed the use of the EntitySelector into a SelectSelector. As far as the main point of the PR, I haven't had time yet to dig into that. I'm a bit preoccupied right now with some changes I'm working on to a couple of my other integrations. I'll try to spend some time on this in the near future.
Thanks for the suggestion!
| CONF_ENTITY_PICTURE, | ||
| CONF_MAX_SPEED_AGE, | ||
| CONF_REQ_MOVEMENT, | ||
| CONF_SHOW_UNKNOWN_AS_0, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are these changes being backed out?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was a mistake. I have reversed this hopefully!
| vol.Required(CONF_ENTITY_ID): EntitySelector( | ||
| EntitySelectorConfig( | ||
| include_entities=list(include_entities), multiple=True | ||
| vol.Required(CONF_ENTITY_ID): SelectSelector( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is wrong with the EntitySelector? It shows the friendly name, entity ID, and the entity picture if it has one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was having big problems with the EntitySelector - didn't find it showed the entityID (clearly?) and it was really hard to find the right entity due to a lack of information. SelectSelector makes it super clear both what you're selecting and what IS selected (when reviewing).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why the EntitySelector is working differently on these two (seemingly otherwise identical) systems, but I don't think that changing this selector should be part of this PR.
| self.options[CONF_END_DRIVING_DELAY] = user_input[ | ||
| CONF_END_DRIVING_DELAY | ||
| ] | ||
| self.options[CONF_END_DRIVING_DELAY] = user_input[CONF_END_DRIVING_DELAY] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why was this change backed out?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This formatting change was a mistake. I have reversed this formatting change, hopefully!
- Add comprehensive entity health analysis with reliability scoring - Add smart entity correlation suggestions based on state change timing - Improve UX with friendly names in entity selectors - Add integration source detection for better diagnostics - Add historical data analysis (7-14 day windows) - Add automated warnings for stale or unreliable entities This feature helps users monitor tracked entity health and discover new entities to track through intelligent correlation analysis. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
018b281 to
e4778b9
Compare
|
addressed concerns :-) |
|
There is still something very wrong with this branch. E.g., config_flow.py now has two definitions of the CompositeConfigFlow class. Also, it's missing the latest commit from the master branch. I think you need to rebase or something. |
|
Oh, also from the last workflow, an error from hassfest: [DEPENDENCIES] Using component recorder but it's not in 'dependencies' or 'after_dependencies' |


This PR adds a comprehensive Entity Diagnostics Dashboard to help users monitor the health of their tracked entities and discover new entities to track.
New Features
Technical Details
Benefits