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

feat(add): PO-THCO-EAU #8729

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

gbrunois
Copy link

@gbrunois gbrunois commented Feb 4, 2025

Add new device PO-THCO-EAU according to this issue : #8245

@gbrunois gbrunois marked this pull request as draft February 4, 2025 12:57
@gbrunois gbrunois marked this pull request as ready for review February 4, 2025 13:19
@reguiny
Copy link

reguiny commented Feb 4, 2025

I have 2 models :
_TZE284_tbgecldg.
_TZE200_tbgecldg

@gbrunois
Copy link
Author

gbrunois commented Feb 4, 2025

I have 2 models : _TZE284_tbgecldg. _TZE200_tbgecldg

This code has been tested only with _TZE284_tbgecldg

@reguiny
Copy link

reguiny commented Feb 4, 2025

I use both with the same code and they work exactly the same.

@reguiny
Copy link

reguiny commented Feb 4, 2025

If i am reading it correctly, In your code it appears to be _TZE204_tbgecldg. 0 not 8.

@gbrunois
Copy link
Author

gbrunois commented Feb 4, 2025

If i am reading it correctly, In your code it appears to be _TZE204_tbgecldg. 0 not 8.

You're right, it's _TZE204_tbgecldg. I used the generated external definition to obtain the fingerprint.

@Koenkk
Copy link
Owner

Koenkk commented Feb 4, 2025

@gbrunois you can safely add these: _TZE284_tbgecldg, _TZE200_tbgecldg, these should be the same devices (as the postfix is the same)

@reguiny
Copy link

reguiny commented Feb 5, 2025

I have tried your code and the battery level is reported at over 100%.
I use this setting which works for me:
'battery', tuya.valueConverterBasic.scale(0, 100, 50, 150)],

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 this pull request may close these issues.

3 participants