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

Basic functionality for the zonnsmart TV01-ZG (Tuya product) #983

Merged
merged 13 commits into from
Aug 26, 2021

Conversation

geoffreylagaisse
Copy link
Contributor

I've managed to map some basic functionality of the following product:
https://www.zonnsmart.com/products/view.php?id=4
Bought on aliexpress.
This is a Tuya product.

I will add some more functionality later on, if that is okay.

@geoffreylagaisse
Copy link
Contributor Author

image

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1083473078

  • 22 of 58 (37.93%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.5%) to 82.022%

Changes Missing Coverage Covered Lines Changed/Added Lines %
zhaquirks/tuya/valve.py 22 58 37.93%
Totals Coverage Status
Change from base Build 1065257715: -0.5%
Covered Lines: 3983
Relevant Lines: 4856

💛 - Coveralls

@dmulcahey
Copy link
Collaborator

@Adminiuga are you good with this one?

Copy link
Contributor

@Adminiuga Adminiuga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@ress23
Copy link

ress23 commented Nov 18, 2021

I'm using this for my Moes TV01 TRV. Internal temp sensor is not working though, HA only receive a static value way off from the actual room temp. Using the built-in temp offset feature I’m able to affect the temp reading but it’s still static and still wrong (also the offset value is not 1:1 to the temp reading). Any idea on what might cause this issue?

{ "node_descriptor": "NodeDescriptor(logical_type=<LogicalType.EndDevice: 2>, complex_descriptor_available=0, user_descriptor_available=0, reserved=0, aps_flags=0, frequency_band=<FrequencyBand.Freq2400MHz: 8>, mac_capability_flags=<MACCapabilityFlags.AllocateAddress: 128>, manufacturer_code=4098, maximum_buffer_size=82, maximum_incoming_transfer_size=82, server_mask=11264, maximum_outgoing_transfer_size=82, descriptor_capability_field=<DescriptorCapability.NONE: 0>, *allocate_address=True, *is_alternate_pan_coordinator=False, *is_coordinator=False, *is_end_device=True, *is_full_function_device=False, *is_mains_powered=False, *is_receiver_on_when_idle=False, *is_router=False, *is_security_capable=False)", "endpoints": { "1": { "profile_id": 260, "device_type": "0x0301", "in_clusters": [ "0x0000", "0x0001", "0x0004", "0x0005", "0x0201", "0x0204", "0xef00" ], "out_clusters": [ "0x000a", "0x0019" ] } }, "manufacturer": "_TZE200_e9ba97vf", "model": "TS0601", "class": "zhaquirks.tuya.ts0601_trv.ZonnsmartTV01_ZG" }

@ress23
Copy link

ress23 commented Nov 21, 2021

Seems like this quirk is causing the same issues for multiple users: #1072

@MattWestb
Copy link
Contributor

All ZonnSmart TRV have one bug that is making them not sending normal update of the local temperature if reading ZHA and Z2M forums.
Until some one is sniffing how tuya is doing it we cant fixing it in ZHA side :-(((

@geoffreylagaisse
Copy link
Contributor Author

@MattWestb yeah I've been trying to figure out the reporting endpoint. But I've not been able to pinpoint it ... As for now I've seen that triggering the is_online endpoint or changing the local calibration triggers the update to the ZigBee controller. I will have to purchase a tuya controller to figure it out I guess

@MattWestb
Copy link
Contributor

For "Moes" TRVs many user is using sending one local calibration command with one automation for getting shorter interval reported local temperature but then the TRV is lighting up but i think it one OK work around for getting it reporting at all for the moment.

By the way welcome back to ZHA after doing little Z2M work !!

@geoffreylagaisse
Copy link
Contributor Author

Thanks! It was always my goal to maintain this device in both channels! But Z2M is easier to program in 😅.

@MattWestb
Copy link
Contributor

Different language and different system architecture both have good and bad sides and in the end all is better then deCONZ ;-))

Thanks for supporting our device in the community !!

@ress23
Copy link

ress23 commented Nov 21, 2021

@MattWestb yeah I've been trying to figure out the reporting endpoint. But I've not been able to pinpoint it ... As for now I've seen that triggering the is_online endpoint or changing the local calibration triggers the update to the ZigBee controller. I will have to purchase a tuya controller to figure it out I guess

Great info, thanks for your time and effort.

I've considered ditching the internal temp sensor and instead create a script that sends a temp reading from an external sensor to the TRV. Do you think that's possible? So far I've been unsuccessful, can't send values to 0x0218

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.

6 participants