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

Humidity sensor #90

Closed
3 tasks done
joimaggz opened this issue Feb 28, 2024 · 4 comments · Fixed by #92
Closed
3 tasks done

Humidity sensor #90

joimaggz opened this issue Feb 28, 2024 · 4 comments · Fixed by #92
Assignees

Comments

@joimaggz
Copy link
Contributor

Checklist

  • I have filled out the template to the best of my ability.
  • This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
  • This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.

First off thank you for this great integration. Works like a charm. Currently it is missing reading data from the humidity sensor. It would be great to have that as well.

Describe the solution you'd like

Get current humidity percentage value from the humidity sensor.

Describe alternatives you've considered

I currently use dingusdk/ha-dukaone to get the humidity value. It is another integration for similar fans, but that one lacks some controls that your integration has.

Additional context

Possible to look at dingus/ha-dukaone to figure out how to get the data. Thanks again!

@hmn hmn self-assigned this Mar 28, 2024
@hmn hmn linked a pull request Mar 28, 2024 that will close this issue
@hmn
Copy link
Owner

hmn commented Mar 29, 2024

@joimaggz I created an updated integration with a device handler and a sensor, but I can't test it because I don't have a V2 fan so could you try it out?

if possible please enable debug in homeassistant and return any debug lines from the log if it does not work

@hmn
Copy link
Owner

hmn commented Mar 29, 2024

I created this beta release you can test https://github.com/hmn/siku-integration/releases/tag/v2.1.0-beta.1

@joimaggz
Copy link
Contributor Author

@hmn That worked like a charm! No bugs in the logger. I will write back a bit later regarding stability or any issues. I havent installed the fans yet so will be some days before.

Thank you for spending the time to implement this feature even though its not of use for you! I really tried to do it myself, it didnt work, but the good thing that came out of it is that I learned a tiny fraction of coding :)

My fan is V.3 - as I understand there is no difference from V.3 and V.2 when it comes to integration compatability. The product I am using is called Twinfresh (www.twinfresh.no) so you can add it to the list of compatable products if you want.

Screenshot 2024-03-30 at 00 05 01

@victori3d
Copy link

@hmn: I tested the integration as well. It works really well. Thanks a lot for enhancing the software.

@hmn hmn closed this as completed in #92 Mar 30, 2024
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.

3 participants