Any chance LG Refrigerator will be integrated?
1 Like
Just noticed that, works great!
One welcome implementation detail was handling of defective LG design of Washer when it disconnects.
Not yet…
All,
I just released a new version with refrigerator support.
For the moment refrigerator sensor are very limited and I need feedback because I don’t have one.
Important:
I renamed the repository to ha-smartthinq-sensors.
I also renamed the component to smartthinq_sensors, so you shoud de-configure and remove previous version and install and configure the new one.
Please provide your feed-back directly on github component repository.
Hi, @ollo69—have you considered contributing your improvements to the wideq library back to the main repository so other projects can use them? As it stands, because you have copied and modified the original source code, the changes you make there are not accessible to other tools—for example, integrations for smart home software beyond Home Assistant. And bug fixes we make in the central wideq repository won’t (automatically) be available to your component.
I am also a little disappointed that the documentation for your component doesn’t contain any attribution about where you took the original code from. You have also changed the license, which is not really allowed. I imagine that this may be an early foray into open-source development for you, which is really exciting! But you might want to read up on some OSS etiquette advice, for example.
1 Like
Hi @samps,
no problem from my point of view to contribute to the original WideQ library, problem is that I changed the structure to better manage devices in HA and merge the code id the library seems quite complex. Anyway I will be happy if you have some suggestion on how to proceed.
Related to documentation, you are right, I honestly do not have time to update it, but I will add asap reference to WideQ project.
For the license, yes I’m not so expert in open-source rules. Should I change to different one?
My objective was just to provide a working componet to the comunity and enjoy myself in code developing, there are no other interest insight.
I’m senior developer wih more than 25 years experience but write code still make me happy, but I normally work on non open-source projects…
1 Like
Awesome! It would be really fun to see how we can incorporate the changes—I think the best way to start would be seeing what parts are easiest to merge back without wholesale restructuring and open a pull request. Or if you think the reorganization is an important improvement, maybe you can open an issue to describe the idea for how the new organization should go?
A good place to start understanding a given license is choosealicense.com’s bullet-point explanations. For MIT, there is basically only one requirement: you can do literally anything you want with the code, but you have to include the attribution and license notice with any redistribution. So that means one easy option is to just include the same LICENSE file as for the original code. Or you can pick any license you like for your code as long as you just add the attribution statement for the code that has been reused.
Thanks for your awesome efforts on making all this stuff better and cover more device types!! It’s truly great and the best kind of outcome of the open-source development model.
1 Like
This is amazing. Thank you!
Great work @ollo69 ! This works great for my washer and dryer. My LG refrigerator also shows now in HA. The door open sensor works, however the fridge and freezer temps are incorrect.
Actual Fridge Temp = 36 F
Home Assistant Sensor = 51.8 F
Actual Freezer Temp = 0 F
Home Assistant Sensor = 46.4 F
I thought maybe there was just an offset for some reason, but when I lowered the Freezer temp 3 degrees, the Home Assistant sensor reported an increase of 5 degrees. So it isn’t just a simple offset unfortunately.
I need some more information because every LG device have different working mode.
Are you able to provide me some debug log as decribed here so that I can check the right way to detect temperature?
Thanks
@ollo69, I updated my smartthinq integration in HACS to v0.3.8 and now the temperatures show correctly for the freezer and fridge!
How do you want to track issues and/or features request? Here or github?
Also you did amazing job and are very thankful!
I noticed that my washing machine outputs symbols that are not recognized:
* ThinQ: received unknown WASHER status '@WM_MX_OPTION_SPIN_HIGH_W' of type 'spin_option'
* ThinQ: received unknown WASHER status '@WM_MX_OPTION_TEMP_EXTRA_HOT_W' of type 'water_temp'
Can you please add them?
In HA log file you should have an information log like this:
2020-04-30 01:46:54 INFO (SyncWorker_0) [custom_components.smartthinq_sensors.wideq.core_v2] Loading model info for LAVATRICE. Model: WTWN3, Url: <the url>
Can you please post the url indicated in your log so that I can download json file related to your model and map missing state?
Thx
I think is better to track issue using github repository and here discuss about usage with HA.
Sounds good, will do it next time, here are the log entries:
2020-05-04 23:13:29 INFO (SyncWorker_1) [custom_components.smartthinq_sensors.wideq.core_v2] Loading model info for WASHER. Model: F3L2CYV5W_WIFI, Url: https://aic.lgthinq.com:46030/api/webContents/modelJSON?modelName=F3L2CYV5W_WIFI&countryCode=WW&contentsId=JS1217232703654216&authKey=thinq
2020-05-04 23:13:29 INFO (SyncWorker_1) [custom_components.smartthinq_sensors.wideq.core_v2] Loading model info for DRYER. Model: RV13B6ES_D_US_WIFI, Url: https://aic.lgthinq.com:46030/api/webContents/modelJSON?modelName=RV13B6ES_D_US_WIFI&countryCode=WW&contentsId=JS1126002523643728&authKey=thinq
Thank You
With last version status should be there…
Hi there!
I have a split system AC that doesn’t seem to appear at all using the hass-smartthinq integration. It doesn’t give any errors at all, the module is setup correctly but it doesn’t seem to see the device. Is the current one using the V2 API? My AC model is WH34SR. When I use the included script with wideQ it doesn’t show any devices either, but a forked wideq shows my AC unit.
Is there any work I can do to assist? i’ve been attempting to engineer this myself but am not to grips with python at all.
1 Like