-
Notifications
You must be signed in to change notification settings - Fork 3
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
release v0.1? #87
Comments
|
both /install and /devices cover parts of this, but it can be better, yes.
very good idea. I'll make a nice PR adding some semi-simple entity type and use it as an example of how to add widgets, because that's the major non-refactoring work that's next.
For both of these I feel expectation management is most important :) |
contribute and discuss!
|
Having now actually tried to use voorkant (main + #89 + #91 + #92) on a Toon1 against my real HA, clearly some more work is needed. Startup takes several minutes at 70%-80% CPU, just to consume all HA state (and debug log extensively about it). Then, with just 8 entities shown, we use 15-30% CPU. Part of that likely is the scrolling of long titles. So, I think we want:
Also, I'm unsure (because startup took so long) but I think I once got caught trying to do things after the Toon software blanked the screen. Should investigate this and if necessary unblank on startup. (And once we have that, we can do auto-blank and wakeup-on-touch) |
it just happened again while voorkant was running. See also #83 |
The scrolling can be set with When you say - maybe don't actually get all entities. Do we need some code that gets entities on demand? so while parsing the dashboard we go and get the entities? |
That's the idea, yes. Need to check if the WS API can do it. First I'll see how much faster I can get all of this without making that change. |
it turns out this goes down to 3 seconds if UIDummy doesn't try to log entire state JSONs. |
What if we introduce a |
nice one. Just printing the unit from the state might be enough of a start in fact, but UIDummy would have to special case that. UISensor makes sense I think. |
this is #104 now |
Nice/needed (please add):
The text was updated successfully, but these errors were encountered: