add exposes to Sinope thermostats and fix energy report on Sinope Plugs #5376
Conversation
|
Thanks! |
|
Hi MattL0, thanks for all the great work on the Sinope devices! =) is there a hardware related issue why "energy" is not exposed on the TH1300ZB?
Would you mind confirming / Any chance to add it ? Let me know what you think, PS: guess you are the same MattL0 who used to run Homeseer ? Cheers! |
|
Hi, Yan, yes I still use HomeSeer, changed my name to PistacheL0.
I do not own the th1300zb, but it should be easy to add this.
I'll check today or Tomorrow!!
…________________________________
De : homeseer666 ***@***.***>
Envoyé : mardi, février 14, 2023 10:53 a.m.
À : Koenkk/zigbee-herdsman-converters ***@***.***>
Cc : MattL0 ***@***.***>; Author ***@***.***>
Objet : Re: [Koenkk/zigbee-herdsman-converters] add exposes to Sinope thermostats and fix energy report on Sinope Plugs (PR #5376)
Hi MattL0, thanks for all the great work on the Sinope devices! =)
is there a hardware related issue why "energy" is not exposed on the TH1300ZB?
I'm no expert but did some testing based on another user feedback which would confirm data availability?
[image]<https://user-images.githubusercontent.com/56667580/218788067-a965dc27-bcbe-410c-9684-c32b27f2ca93.png>
Would you mind confirming / Any chance to add it ?
I have a number of these thermostats installed so am available for testing.
Let me know what you think,
PS: guess you are the same MattL0 who used to run Homeseer ?
Thx! Yann aka 123qweasd from Montreal / Homeseer forums.
Cheers!
—
Reply to this email directly, view it on GitHub<#5376 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AGRZIWJRRQ2PVG2PPEV425TWXOS5VANCNFSM6AAAAAAUB5HZVE>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
|
Great! Thanks for the prompt response, looking forward to testing it :) Thought you had moved to Home assistant... Salutations de la Rive-Sud Pistache ;) |
|
Can You try to add the attribute currentsumdelivered to the semetering cluster in the reporting page?
Do you get any error in the log? |
|
Seems to be OK, see below.
no errors in the log, anything else I should look for in the log?
[image: image.png]
…On Tue, Feb 14, 2023 at 8:10 PM MattL0 ***@***.***> wrote:
Can You try to add the attribute currentsumdelivered to the semetering
cluster in the reporting page?
[image: 618EE3EE-354C-495A-95FC-4755D9EEEF1A]
<https://user-images.githubusercontent.com/27497561/218899076-18d00d7a-1d18-45c8-a5c9-7083d6a0dcb1.jpeg>
Do you get any error in the log?
—
Reply to this email directly, view it on GitHub
<#5376 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ANQK3PB3QHL7X42BZBBYOOTWXQUGXANCNFSM6AAAAAAUB5HZVE>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
|
|
|
Ref to my original post. Efforts from @cloutierjo https://discord.com/channels/931570222475907083/1073757763311771719/1073807675265122374 |
|
pretty sure the pull request from @cloutierjo can be merged like that |
|
Do you know the update interval for "energy"? The updated Sinope.js ( @cloutierjo ) has been running since yesterday evening and all thermostats still show N/A... is it a cumulative value or hourly, daily, other,... ? Thx
|
|
Based on the doc from another Sinope, seems like it's cumulative: Energy (numeric) so something must be wrong unless the value gets updated every day/24h ? Let me know what you think, Thx |
|
could you try to set the report interval to 300 , and the "min rep change" to 0,1 ( not 0.1). yes this is cumulative. I do not know a method to reset it. Setting theses value into the dev console is suposed to reset the value to 0 according to zigbee3.0 specs.., but here it is soft resetting the thermostat as a hole. |
|
Testing. stay tuned. manipulating the data to get the desired values is not overly complicated but would be nice to have some type of MQTT command to reset it. |
|
300 for minimum or maximum report interval? |
|
the "min rep change" field will only let me use period "." not comma "," |
|
Ok here's what I'm testing now. Energy still shows N/A
|
|
Ok will test this morning, stay tuned. |
|
is the dev build available under windows ? not running Linux.... https://www.zigbee2mqtt.io/advanced/more/switch-to-dev-branch.html#linux Or any workaround for me to test under windows? |
|
Worst case I will install a portable Linux; let me know what the options are. Thanks again! |
Yes there is one (replacing the sinope.js file directly). But, this should be done only after a backup of the zigbee2mqtt folder. First , what is your zigbee2mqtt version installed? http://yourIP:port/#/settings/about |
|
Version 1.30.1 |
|
can you try with this code ? remove ".txt" Didn't see the divisor readind was not in the code. |
|
testing. stay tuned. thx |
|
Back with the original log error/issues .... Zigbee2MQTT:info 2023-02-20 21:14:38: Currently 14 devices are joined: Zigbee2MQTT:warn 2023-02-20 21:15:27: Received message from unsupported device with Zigbee model 'TH1300ZB' and manufacturer name 'Sinope Technologies' |
|
Just to ensure I'm not missing something: -Stopped Z2M other Zigbee devices work normally. |
|
BTW I don't if this can help troubleshoot, but once I restart with your modified sinope.js, I can't roll back to the original sinope.js; it returns the same errors "Not supported (Router)"; I have to call "npm ci" from command line to make it work again. |
|
could you test this one ? https://github.com/MattL0/zigbee-herdsman-converters/blob/master/devices/sinope.js This should at least set your thermostats in a working state. |
|
Also what is the value of the divisor attribute for the seMeteting cluster? |
|
File test : same errors. |
|
1000
|
|
Do you have any active external converter in your zigbee2mqtt folder?
…________________________________
De : homeseer666 ***@***.***>
Envoyé : Tuesday, February 21, 2023 12:20:44 AM
À : Koenkk/zigbee-herdsman-converters ***@***.***>
Cc : MattL0 ***@***.***>; Author ***@***.***>
Objet : Re: [Koenkk/zigbee-herdsman-converters] add exposes to Sinope thermostats and fix energy report on Sinope Plugs (PR #5376)
File test : same errors.
checking value hold on
—
Reply to this email directly, view it on GitHub<#5376 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AGRZIWNTU3T276BWAV3LXR3WYRGCZANCNFSM6AAAAAAUB5HZVE>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
|
Not that I know of. everything is stock. how can I double check? |
|
checked my configuration.yaml using Zemismart rollerblinds and Yale locks. that's it |
|
I think it’ll be easier to wait for the release (7 days), as I do not see what is the issue here.
I’ll make a pr tomorow, to add the divisor. It’ll be in the release version.
…________________________________
De : homeseer666 ***@***.***>
Envoyé : Tuesday, February 21, 2023 12:52:58 AM
À : Koenkk/zigbee-herdsman-converters ***@***.***>
Cc : MattL0 ***@***.***>; Author ***@***.***>
Objet : Re: [Koenkk/zigbee-herdsman-converters] add exposes to Sinope thermostats and fix energy report on Sinope Plugs (PR #5376)
checked my configuration.yaml
no external converters.
using Zemismart rollerblinds and Yale locks.
Frontend.
homeassistant: false
availability: true
permit_join: false
device_options:
expose_pin: true
that's it
—
Reply to this email directly, view it on GitHub<#5376 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AGRZIWLBVV6Z6ZYO2WIZ67DWYRJ3VANCNFSM6AAAAAAUB5HZVE>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
|
Thanks for the efforts, will report back once the release is available. |
|
BTW since the release is not out yet, I've noticed this other issue with the TH1300ZB integration: for "Enable outdoor temperature display": "on" "true" "false" work but "off" doesn't. Not sure if this is an easy fix or not but it also affects the frontend control (can't be turned off) so maybe it's worth fixing it. Let me know what you think. Cheers, Yann |
|
Hi Matt, Just installed Z2M latest release and unfortunately, energy still doesn't show ... |
|
Hi Yan, I have to prepare for an important interview I have this monday(13th). I will come back to you shortly after that. |
|
No problem, keep me posted |
|
Hello, Thank you very much for this! I had twelve TH1300ZB running on ZHA. Those on firmware 1.0.8 ( I guess 1008 in Zigbee2MQTT) were not reporting energy. I updated them all to current firmware 1.1.9 (1109) via Sinope GT130 (took a few days for all to receive the update) then added them to Zigbee2MQTT 1.30.1. No energy reported. When 1.30.2 was released, I applied the update. Still no energy reported. I tried to reconfigure and re-interview to no avail (maybe me, I'm new to Zigbee2MQTT/learned a lot during that week). @homeseer666 : would be be able to test deleting/adding back one to your Zigbee2MQTT and see if energy gets reported after a few minutes? @MattL0 : Is there a smarter way to do this without deleting/adding them back? (I'm quite sure that was not the smartest move on my end :-P ) @MattL0 : I have two TH1300ZB connected to an auxiliary output TR1310 :
Thanks again for this update! |
|
Thanks for the heads up! You are right, after a while, it does start logging energy =) Only bug left on my end is minor / detailed here: Cheers, |
|
Do your "energy" values get updated? If yours work, I guess I'll try do deinstall/reinstall all of them. Keep me posted, thx |
Yes, they do. |
|
Ok thank you, I'll try do delete/add one of the thermostats to see if it resolves the issue. Would you also mind confirming your reporting parameters (portal-device-reports):
|
|
Ok got it, first issue was the missing "min rep change". Still, can you confirm 2 last items: 1: reporting parameters you are using: 2: can you reproduce this bug (Ie: switching "enable_outdoor_temperature" to OFF doesn't work and generates an error / manually sending "false" does work.) |
|
I will come back tomorrow. It seems part of the issues are resolved now? |
|
Yes, will gather some more data but so far so good.
|
|
Hi @MattL0, Quick question, still related to TH1300 exposed features. Unless there is hope to get them added/merged, is there a way to poll unsupported features via MQTT/Z2M? I'd like to create temperature sensors in Home assistant for current floor and room temperatures. Currently Z2M only exposes "LocalTemperature" (either floor or room , based on active mode)
This is achievable, by polling 2 other clusters (separate temperatures)
They are accessible with ZHA but not sure how to access it with Z2M. Any help would be greatly appreciated. Thank you and happy new year! |
…inope plugs