-
Notifications
You must be signed in to change notification settings - Fork 664
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
Use different Tuya cluster for _TZE200_yjjdcqsq
temp sensor
#3244
base: dev
Are you sure you want to change the base?
Conversation
When using Var04, my temp sensor cannot read any values. With Var03 the main values (temp, humidity and battery can be read).
@Jonas-Meyer97 Since you added the device to variant 04 in the first place, can you check and see if your devices still work with Variant 03 as well? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #3244 +/- ##
=======================================
Coverage 88.18% 88.18%
=======================================
Files 301 301
Lines 9412 9412
=======================================
Hits 8300 8300
Misses 1112 1112 ☔ View full report in Codecov by Sentry. |
@Maddimax Sorry, I wasn't able to try it earlier. With variant 03, my device no longer reports the battery status. I tested it with two different devices and waited 24 hours. It seems there are two different variants for "signature": {
"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=4417, maximum_buffer_size=66, maximum_incoming_transfer_size=66, server_mask=10752, maximum_outgoing_transfer_size=66, 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": "0x0104",
"device_type": "0x0302",
"input_clusters": [
"0x0000",
"0x0001",
"0x0004",
"0x0005",
"0x0402",
"0x0405",
"0xef00"
],
"output_clusters": [
"0x000a",
"0x0019"
]
}
},
"manufacturer": "_TZE200_yjjdcqsq",
"model": "TS0601"
} |
Hmm, it's not ideal that there are apparently two almost identical versions of this sensor. zha-device-handlers/zhaquirks/tuya/ts0601_sensor.py Lines 50 to 95 in 915ff9b
We might be able to parse Tuya datapoints 3 and 4 and map them onto the same |
_TZE200_yjjdcqsq
temp sensor
Proposed change
When using Var04, my temp sensor cannot read any values. With Var03 the main values (temp, humidity and battery can be read).
Additional information
The raw signature of the device:
Checklist
pre-commit
checks pass / the code has been formatted using Black