Skip to content
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

Adding SP-EUC01 temperature #2960

Merged
merged 7 commits into from
Sep 11, 2021
Merged

Adding SP-EUC01 temperature #2960

merged 7 commits into from
Sep 11, 2021

Conversation

Vodochnik
Copy link
Contributor

it works from GenDeviceTempCfg cluster.
See Koenkk/zigbee2mqtt#6861

it works from GenDeviceTempCfg cluster.
@Vodochnik Vodochnik changed the title Adding device temperature Adding SP-EUC01 temperature Aug 26, 2021
@islepnev
Copy link

I have tried this patch, sorry no luck, I could be missing something.
device_temperature appeared, but displays null for over 1 hour.
4 sockets of the same type, similar behavior.

Screenshot from 2021-08-26 19-19-41

@Koenkk
Copy link
Owner

Koenkk commented Aug 26, 2021

@Vodochnik I would expect you also need to bind the cluster: await reporting.bind(endpoint, coordinatorEndpoint, ['genDeviceTempCfg']);

@Koenkk
Copy link
Owner

Koenkk commented Sep 1, 2021

@islepnev can you check if it works now?

@Vodochnik
Copy link
Contributor Author

wait pls)

Add polling for device temperature since SP-EUC01 does not support reporting for it.
@Vodochnik
Copy link
Contributor Author

Okay, after some investigations... this plug does not support reporting for device temperature.
Temperature is reported only once on plugging in the wall outlet, but not afterward.
So we need polling for it.
Big thanks to @kirovilya for pointing me in right direction how to accomplish it.
withValueMin(-60) is for Russia :P

@Vodochnik
Copy link
Contributor Author

@islepnev please check if it works for you now) Polling time is set to 60 sec.

@islepnev
Copy link

islepnev commented Sep 9, 2021

@Vodochnik, it works perfectly!

History plot for my sockets:
Screenshot from 2021-09-09 21-14-42

@Vodochnik
Copy link
Contributor Author

Vodochnik commented Sep 9, 2021

@islepnev thanks!

@Koenkk done)

@Koenkk
Copy link
Owner

Koenkk commented Sep 10, 2021

Could you increase the poll duration? I'm not a fan of polling since it increases the stress on the network and I feel polling every 30 mins-1 hours is enough

update polling interval to 30 min.
added description explaining polling interval
@Vodochnik
Copy link
Contributor Author

Could you increase the poll duration? I'm not a fan of polling since it increases the stress on the network and I feel polling every 30 mins-1 hours is enough

Updated polling interval to 30 min, added explaining description.

@Koenkk Koenkk merged commit b4f7bff into Koenkk:master Sep 11, 2021
@Koenkk
Copy link
Owner

Koenkk commented Sep 11, 2021

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants