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

FR: Support for Kogan 1.7l Kettle #47

Closed
xbmcnut opened this issue Oct 3, 2021 · 13 comments
Closed

FR: Support for Kogan 1.7l Kettle #47

xbmcnut opened this issue Oct 3, 2021 · 13 comments

Comments

@xbmcnut
Copy link

xbmcnut commented Oct 3, 2021

Hi there,

My Kogan panel heater is working great but was wondering if there was any intended support for the Kogan Smart Kettle https://www.kogan.com/nz/buy/kogan-smarterhome-17l-smart-glass-kettle/? Can't remember whether I'd asked that before or not.

I have the DeviceId and LocalKey using the 'Bleustacks' method but the integration just says unsupported device when I try and add.

@make-all
Copy link
Owner

make-all commented Oct 6, 2021

It will require more information about the DPS available on that device. If there is nothing in the HA log, then using a command line tool like tuyacli or tinytuya to query the dps is another method, and probably more convenient when it comes to experimenting with different settings.

@make-all make-all added the new device Unsupported device label Oct 6, 2021
@xbmcnut
Copy link
Author

xbmcnut commented Oct 6, 2021

Thanks. I have the preferences_global_keyazxxxxxx.xml file that contains an entry for dps along with the deviceID and local key etc that I pulled using the Bluestacks method with an old version of Smart Life. Will that contain the data you're after? I also have a Tuya developer account so I'm assuming some or all of the stuff I can pull from there?

@make-all
Copy link
Owner

make-all commented Oct 6, 2021

I think that XML file is the schema file for the device, which has all the useful information in it.

The Tuya developer portal can contain useful information about ranges and enum values, but is missing the DPS ID and does not always list all available settings. It is actually documentation for the server API, which often closely matches the device local API, but not always.

@xbmcnut
Copy link
Author

xbmcnut commented Oct 6, 2021

Happy to share the full XML with you. Can I email that?

@make-all
Copy link
Owner

make-all commented Oct 8, 2021

You can email it to jasonrumney at gmail.com

@xbmcnut
Copy link
Author

xbmcnut commented Oct 9, 2021

Done! Thank you.

@xbmcnut
Copy link
Author

xbmcnut commented Oct 9, 2021

Wow, that was fast Jason! Sadly, after updating to 0.11.1 this evening, I still get the 'device not supported' popup after entering the same data in the provided XML. I rebooted HA (2021.10.2) after updating the HACS integration.

image

2021-10-09 22:51:15 WARNING (MainThread) [custom_components.tuya_local.config_flow] Device matches None with quality of 0%. DPS: {'5': 96, 'updated_at': 1633773075.2028716}
2021-10-09 22:51:15 WARNING (MainThread) [custom_components.tuya_local.config_flow] Report this to https://github.com/make-all/tuya-local/issues/

@make-all
Copy link
Owner

make-all commented Oct 9, 2021

It is only reporting the current temperature, not the on/off state or the preset. You may need to add it while it is switched on so it reports everything.

If it really is only ever reporting current temperature locally, and requiring commands to go via the cloud, then you might be better setting it up as a temperature sensor in localtuya for now, as I haven't yet added sensor support.

@xbmcnut
Copy link
Author

xbmcnut commented Oct 12, 2021

Thank you!!!! I've been able to add it as a simple switch while it was boiling which is all I need. Now I can ask Google to turn it on!

image

@make-all
Copy link
Owner

make-all commented Oct 12, 2021 via email

@xbmcnut
Copy link
Author

xbmcnut commented Oct 12, 2021

Sorry, this is all I see in the HA log since the addition of the switch.

2021-10-12 19:29:56 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2021-10-12 19:30:26 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2021-10-12 19:30:56 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2021-10-12 19:31:08 ERROR (SyncWorker_12) [custom_components.tuya_local.device] Failed to refresh device state for Kogan Kettle.
2021-10-12 19:31:36 WARNING (MainThread) [homeassistant.helpers.entity] Update of switch.kogan_kettle is taking over 10 seconds
2021-10-12 19:32:36 WARNING (MainThread) [homeassistant.helpers.entity] Update of switch.kogan_kettle is taking over 10 seconds
2021-10-12 19:32:56 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2021-10-12 19:33:26 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2021-10-12 19:33:56 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2021-10-12 19:34:17 ERROR (SyncWorker_11) [custom_components.tuya_local.device] Failed to refresh device state for Kogan Kettle.
2021-10-12 19:34:36 WARNING (MainThread) [homeassistant.helpers.entity] Update of switch.kogan_kettle is taking over 10 seconds
2021-10-12 19:34:56 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30

@make-all make-all added partially working Needs more work to complete and removed new device Unsupported device labels Oct 14, 2021
@make-all make-all added the enhancement New feature or request label Feb 1, 2022
make-all added a commit that referenced this issue Jun 16, 2022
Based on info in issue #183, this dp is not returned even when the kettle is on.
It may be a set-only dp, which will need further work to support.

Should solve issue #47 also.
@make-all make-all added awaiting confirmation Wating for confirmation the issue is solved and removed partially working Needs more work to complete labels Jun 16, 2022
@make-all
Copy link
Owner

Based on info from issue #183, I have removed the preset from the config. It seems that this is a write-only command, or only appears in the output under limited circumstances. With just the switch and current temperature in the config, the kettle should now be correctly detected when it is on.

@make-all make-all added unreleased Will be in next release and removed enhancement New feature or request unreleased Will be in next release labels Jun 25, 2022
@make-all
Copy link
Owner

Please followup on issue #183 if there is any issue still.

@make-all make-all removed the awaiting confirmation Wating for confirmation the issue is solved label Jun 26, 2022
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

No branches or pull requests

2 participants