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

Wrongly imported humidity sensor #5

Closed
Puebllo opened this issue Mar 15, 2024 · 10 comments · Fixed by #6
Closed

Wrongly imported humidity sensor #5

Puebllo opened this issue Mar 15, 2024 · 10 comments · Fixed by #6

Comments

@Puebllo
Copy link

Puebllo commented Mar 15, 2024

Hi,
Amazing job, reviving this integration project! I've just migrated from original integration to your's and wanted to check if the wireless thermostat data such as battery level and signal strength are available. Unfortunately they're not. In emodul I can see battery level and signal strength (represent with "bars")
Please tell me, are those features are already in your fork or are they in development ?

I've TECH: L4-WiFi (v.1.1.10) controller and TECH R-8b Wireless thermostats for each area. Below is example of attributes from thermostat:

  - heat
  - "off"
min_temp: 7
max_temp: 35
current_temperature: 22.6
temperature: 21.5
current_humidity: -2
hvac_action: idle
friendly_name: Biuro
supported_features: 385```
@anarion80
Copy link
Owner

Hello.

Not sure what you mean exactly. Battery level for wireless sensors is extracted as separate sensor under device:
image

Signal strength is currently not extracted.

PS. I was added as collaborator to the upstream repo, so this repo will become obsolete and everything will happen in the upstream:
mariusz-ostoja-swierczynski/tech-controllers#55

@Puebllo
Copy link
Author

Puebllo commented Mar 15, 2024

Oh, now I see them in UI. I was too quick with issue. Thanks!

I've one extra sensor for humidity that shows -2% in all areas.

@anarion80
Copy link
Owner

Ok, for humidity it would be great if you could provide a json dump of what emodul is returning for your module.
Go to emodul.pl while logged in, developer tools (F12), network tab, refresh page and then look for XHR type request to your module id. Click on it and get the JSON response:
image

@Puebllo
Copy link
Author

Puebllo commented Mar 15, 2024

JSON response is quite long, so I've putted it here: https://pastebin.com/TmseCEUb

(I've redacted controller id, rest of the response is untouched)

@anarion80
Copy link
Owner

Ok, so the API is reporting "-2" for each of 8 sensors. What does that translate into in emodul itself?

@Puebllo
Copy link
Author

Puebllo commented Mar 15, 2024

emodul doesn't show anything regarding humidity

image

@anarion80
Copy link
Owner

All right, then it seems like some error value on their side as well. Right now I only check if humidity is reported and if it is different than zero. I will need to then check if it is greater than zero, and then you will not have such sensor entity.

@Puebllo
Copy link
Author

Puebllo commented Mar 15, 2024

Perfect, let me know when it's ready then I'll test it

@anarion80 anarion80 changed the title Reporting battery level and signal strength of Wireless TECH Thermostat R-8b Wrongly imported humidity sensor Mar 15, 2024
anarion80 added a commit that referenced this issue Mar 15, 2024
🐛 Fix humidity sensor creation. Fixes #5
@anarion80
Copy link
Owner

Ok, it is fixed now on the main branch. You can download and test.

@Puebllo
Copy link
Author

Puebllo commented Mar 15, 2024

Works good, no humidity sensor and no errors in logs 👌

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 a pull request may close this issue.

2 participants