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

issue adding BEOK thermostat TOL47 - wrong device profile mapping #1930

Closed
apf-doit opened this issue May 18, 2024 · 2 comments
Closed

issue adding BEOK thermostat TOL47 - wrong device profile mapping #1930

apf-doit opened this issue May 18, 2024 · 2 comments

Comments

@apf-doit
Copy link

apf-doit commented May 18, 2024

Hi,

i have added successfully 6 devices of that type, but the 7. doesn't wont to be recognized correctly. Instead of using the thermostat template beok_tgm50_thermostat.yaml like the 6 devices before, it comes up with idea_heatingbelt_airconditioner.yaml. The correct template is not in the list to choose.

How can I change the mapping during the import process or maybe after? The device in question works just like the others in the Smart Life App, so i consider the device to work as expected and not broken.

I unfortunately don't have a debug of a successfull import of any of the other devices, but I can just delete one and add it again if needed.

I will attach the debug during the import as well.

What other informations can I provide?

cheers
alex

debug:
home-assistant_tuya_local_2024-05-18T12-03-07.200Z.log

diagnostic successful import:
th-u7_config_entry-tuya_local-5385a9e4efd19a85b12e74e653374038.json

diagnostic unsuccessful import:
th-u8_config_entry-tuya_local-c244cd96f07b73ceb957a34c633f2e49.json

@apf-doit apf-doit added the bug Something isn't working label May 18, 2024
@apf-doit
Copy link
Author

apf-doit commented May 18, 2024

i just want to add some screenshot of 2 same devices in comparison after import: th-u7 is imported correctly and th-u8 (named Air Conditioner) as an air conditioner.

th-u7
th-u8

@make-all make-all added device improvement Improvement to an existing device config and removed bug Something isn't working labels May 19, 2024
make-all added a commit that referenced this issue May 19, 2024
Some PRs slipped through with the `scale: 0` copied from Tuya cloud
docs.

This integration uses a multiplier for scale, not a decimal place
shift like the Tuya cloud uses. This allows temperature scales of 2
that some thermostats use to represent 0.5 degree steps to be
represented.

In any case, unscaled values do not need to include a scale mapping.

Issue noticed in the log attached to issue #1930, though this is not
the device being reported.
@make-all make-all added awaiting confirmation Wating for confirmation the issue is solved unreleased Will be in next release labels May 19, 2024
make-all added a commit that referenced this issue May 19, 2024
It seems some devices do not report dp 110.

Issue #1930
@apf-doit
Copy link
Author

It works now - thank you. I don't even have to choose the device after import, the template pops up correctly. I guess this fix will go as an improvement into next release, so I will close this issue.

@make-all make-all reopened this May 20, 2024
@make-all make-all removed awaiting confirmation Wating for confirmation the issue is solved unreleased Will be in next release device improvement Improvement to an existing device config labels May 20, 2024
timlaing pushed a commit to timlaing/tuya-local that referenced this issue Aug 8, 2024
Some PRs slipped through with the `scale: 0` copied from Tuya cloud
docs.

This integration uses a multiplier for scale, not a decimal place
shift like the Tuya cloud uses. This allows temperature scales of 2
that some thermostats use to represent 0.5 degree steps to be
represented.

In any case, unscaled values do not need to include a scale mapping.

Issue noticed in the log attached to issue make-all#1930, though this is not
the device being reported.
timlaing pushed a commit to timlaing/tuya-local that referenced this issue Aug 8, 2024
It seems some devices do not report dp 110.

Issue make-all#1930
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

2 participants