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

Adding new tuya / Moes TRV #932

Merged
merged 1 commit into from
Jun 21, 2021
Merged

Adding new tuya / Moes TRV #932

merged 1 commit into from
Jun 21, 2021

Conversation

MattWestb
Copy link
Contributor

Adding the second type of the _TYST11 tuy /Moes TRV.
PR is made for adding them in ZHA for getting the extended functions Moes TRV is having.

Not tested by my or some user but #931 is some gave testing the first version but im not 110% that all is working but i think its better then no quirk for the device.

Adding the second type of the _TYST11 tuy /Moes TRV.
PR is made for adding them in ZHA for getting the extended functions Moes TRV is having.

Not  tested by my or some user but #931 is some gave testing the first version but im not 110% that all is working but i think its better then no quirk for the device.
@coveralls
Copy link

Pull Request Test Coverage Report for Build 958058968

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 82.547%

Totals Coverage Status
Change from base Build 952628130: 0.0%
Covered Lines: 3798
Relevant Lines: 4601

💛 - Coveralls

@MattWestb
Copy link
Contributor Author

MattWestb commented Jun 21, 2021

zigpy/zigpy#653 (comment) updated but picture is coming later then have finding one OK.

Edit: Picture added and deleting the TRV from not known model.

@Adminiuga Adminiuga merged commit b0049a9 into zigpy:dev Jun 21, 2021
@MattWestb
Copy link
Contributor Author

Thanks !!

@MattWestb
Copy link
Contributor Author

The Becca BRT-100 reporting and also then setting have the setpoint temperature with different multiple as the "normal" Moes TRVs and is one fix for it #981 that is being added for the _TZE200_b6wax7g0 TS0601 variant but if you is getting one _TYST11_b6wax7g0 6wax7g0 it need adding one more device typer in the quirk for fixing it (the extra functions in ZHA is in place) like is being done in the PR linked in the issue.

Then the setpoint is sent with one decimal lesser in preposition its also only possible changing it in 1°C and not 0.5°C as with other tuya TRVs.

@miklosbagi miklosbagi mentioned this pull request Aug 28, 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

Successfully merging this pull request may close these issues.

3 participants