-
Notifications
You must be signed in to change notification settings - Fork 16
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
Show battery level in device list #44
Comments
Actively working on including that in the next release |
Please try the latest release |
Hi, I have one door sensor with battery, I have other sensors from IKEA but because there was no information on the battery state I was using Z2MQTT instead. Since this release I do have a state for this sensor, but it looks like this: I know the state of the battery isn't 0 because: Now, beside the Door Sensor, I do have two blinds that also use battery, and the new integration shows this: And when the battery icon is clicked I can see this: What I would suggest is to show the percentage along with the icon and when you click the icon we can go to the history as per the sensor: Here is the log from my side. I hope it helps ... And again, Thank You for making this integration possible! 2024-05-11 09:06:53.387 INFO (SyncWorker_8) [custom_components.dirigera_platform] === START Devices JSON === |
Correct, the issue is with the base library. Have submitted a pull request as its merged will cut a new release to reflect the same |
Should work now, please use the latest release |
Hi @sanjoyg, Yes it does: Although it doesn't show the blinds battery on the list of devices, the entity is shown on the dashboard (Dwain's dashboard): Thank You very much! I'll be moving the motion sensors to the Ikea hub once again :) |
Hi, Thank you so much for the quick update. After upgrading to 1.7.1, I now see the battery level for my IKEA PARASOLL door sensor in the device list. Unfortunately, I do not see the battery levels in the device list for the two types of remotes I currently have connected to my Dirigera ("Remote control N2" and "Tradfri on/off switch"). I also noted that the battery is still reported under "sensors" rather than under "diagnostic" where I find it for Z2M. Not sure it makes any difference apart from aestethics though... Again: thank you! |
Thank you, lemme have a look for the remote controls... I assume they are included in the json dump |
Hi, Thanks sanjoyg. Really appreciate your help :-) I'm afraid the json dump was from vulcanoid1975. Would be happy to make a dump as well - but will need a littel pointer on how to create one :-) For now, the battery is reported for both remotes in the device page under "senors" (if that helps): //S |
I mis-understdood the issue at the first place. The battery level will appear as a sensor for the device. This is the general trends with most integration where there is one device representation and all relevant data points are represented as sensors. Example IKEA environment device, will expose as one device and multiple sensors for temperature, humidity, battery level etc. |
Sorry if I'm mis-unerstanding your last comment. The issue is that the battery level indicator is missing in the Home Assistant device listing for the remotes ... (on the right-hand column in the screenshot below) ... even though the battery level is available under sensors in the device view. I would love to be able to see the battery level of the remotes directly in the device list. |
My comment about the battery being listed under sensors was a separate (probably unimportant) observation. I noted that batteries of devices in the zigbee2mqtt integration are listed under "Diagnostic" rather than under "Sensors" (see screenhot below). My understanding is that these are also entities, but with a different category. I was wondering whether the category could explain why the battery entity for the Dirigera remotes are not shown in the device list. However, the door sensor has its battery entity under "Sensors", so I assume that is not the explanation. |
Try now |
Tried. Unfortunatly not working with 1.7.3 :-( Only dirigera integration device that has a battery level indicator in the device list is the PARASOLL door sensor |
Is the json dump same? then I can try it |
It should be the same, but I made some changes so ... here's another one: 2024-05-21 13:32:42.093 INFO (SyncWorker_39) [custom_components.dirigera_platform] === START Devices JSON === |
@sanjoyg Hi, I hope this issue can be be fixed quite easily. As you can see only Tradfri motion sensor has the battery status correct show in device list. The Tradfri motion sensor has battery level exposed via Diagnostics and not in the Sensors section which is correct. Other devices has the battery level exposed via Sensor and the Diagnostics shows nothing: |
Yup will have it in the next release |
Please try the latest change |
Please try the latest release |
I have tried with release 1.7.6 which is currently the latest version |
I was trying to co-relate the following entities to your JSON but I could find the entries. Can you point out the id of the entity from JSON dump in this thread for the following
|
The json in upper comment is from another user. I posted to this thread screenshots only. But my json is posted in another issue in last comment: |
Fixed this in the latest release, have tested and reviewed it. |
Can u please paste the JSON
…On Wed, 5 Jun 2024 at 15:46, Thomas Stieler ***@***.***> wrote:
I'm using this HA integration for my smarthome as well and I'm very
grateful for your work.
I can confirm that with the new release almost all of by Dirigera devices
are showing the battery level in the device overview, but there are two
exception: Two Fyrtur blinds
image.png (view on web)
<https://github.com/sanjoyg/dirigera_platform/assets/9194148/2c9d8687-ebd4-4d21-a307-4169a1f2f868>
The battery is shown in the diagnose section for the device, but unlike
other devices without value, the value is shown in a popup, if I click on
the battery icon or label:
image.png (view on web)
<https://github.com/sanjoyg/dirigera_platform/assets/9194148/82f4e838-502b-426e-9f36-d160c9dde885>
But it's not shown in the device list and I cannot see the level in an
enitity card on a dashboard, it only shows on click:
image.png (view on web)
<https://github.com/sanjoyg/dirigera_platform/assets/9194148/1b1ec024-5bf8-4ee5-95a1-4ff61e7ec9fe>
Are blinds something special? Would be great to see the battery level for
them as well...
—
Reply to this email directly, view it on GitHub
<#44 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB2X3CV5JEJGSANWQT2BOCLZF3QQ7AVCNFSM6AAAAABHO2I7TKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBZGQZDKNBQGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Here is the plain logfile: I also "beautified" the JSON (replaced single by double quotes, (True|False) by (true|false), then add linebreaks/identation). Here is the result: Here's a remote with working battery, the blind control, with label "Büro Rollos" in the screenshot above:
And here is one of the blinds:
Actually I can't see a difference in the JSON regarding battery level, but maybe you can :-) |
I tried testing... for some reason it just show up... Will try n fix it over few days |
Wow, sounds really good. |
I meant it doesnt show up :-( |
Ah, ok. How can I help tp reproduce? |
:-) I am able to reproduce as well but not able to fix :-(
…On Thu, Jun 6, 2024 at 6:44 PM Thomas Stieler ***@***.***> wrote:
Ah, ok.
How can I help tp reproduce?
I mean, the screenshots are real, I promise I didn't edit them ;-)
—
Reply to this email directly, view it on GitHub
<#44 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB2X3CWWLDUYYWH4SY5EQ2LZGBOEJAVCNFSM6AAAAABHO2I7TKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJSGQZTQNBWG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Ah, ok. |
I have the same problem that the battery icons in the dashboard are static. My guess: The entities are of type “binary_sensor”; shouldn't they be “sensor”? Binary means that it is not a numerical value, but an on/off or something similar? But I'm not a specialist, so maybe I misunderstood. Thank you very much for your efforts for this extension! |
The battery is actually of sensor type, Homeassistant has an issue with blinds % its reported in HASS already |
aah .. can u give me the json dump |
will you try this with the latest release once more |
This should be fixed now, have a look |
I got an error after installing and rebooting the 1.8.1; when I tried to reconfigure the integration, home assistant completely crashed. Error setting up entry IKEA Dirigera Hub : 192.168.23.48 for dirigera_platform |
There was bug left behind, try 1.8.2 |
Thank you, the integration now works again without errors. The battery entity is now created as a sensor (and no longer as binary_sensor). Nevertheless, the dynamic battery icon (with changing fill level) is not used, but a static icon. I don't know why this is the case. |
Is the correct battery % been shown? |
This is fixed now. |
With 1.8.7 everything is fine, thank you for your effort! |
Hi,
Thank you for all the effort you have (and are) putting into making this integration available to all of us! It is working like a charm.
One thing that I have been missing is the ability to see the battery level for remotes etc. in the device list. I suspect the reason it does not show up is that the "Battery" entity on dirigera_platform devices is reported under "Sensors" rather than under "Diagnostic". Is this something that would be possible to change without too much effort?
Again, thank you for providing this integration!
Best regards,
Soren
The text was updated successfully, but these errors were encountered: