home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.43k stars 30.67k forks source link

Missing sensors from AIR_DETECTOR (yrr3eiyiacm31ski) #97563

Closed pacorreia closed 9 months ago

pacorreia commented 1 year ago

The problem

I use an Air Quality detector compatible with Tuya integration.

This device in Tuya IoT portal has the following Standard Status set

Code Type Values
co2_state Enum {"range": ["alarm", "normal"]}
co2_value Integer {"unit": "ppm", "min": 0, "max": 5000, "scale": 0, "step": 1}
alarm_volume Enum {"range": ["low", "middle", "high", "mute"]}
alarm_time Integer {"unit": "s", "min": 1, "max": 60, "scale": 0, "step": 1}
alarm_switch Boolean "{true,false}"
battery_percentage Integer {"unit": "%", "min": 0, "max": 100, "scale": 0, "step": 1}
alarm_bright Integer {"unit": "%", "min": 0, "max": 100, "scale": 0, "step": 1}
temp_current Integer {"unit": "\u2103", "min": -9, "max": 199, "scale": 0, "step": 1}
humidity_value Integer {"unit": "%", "min": 0, "max": 100, "scale": 0, "step": 1}
pm25_value Integer {"unit": "\u03bcg/m3", "min": 0, "max": 1000, "scale": 0, "step": 1}
voc_value Integer {"unit":"mg/m3","min":"0","max":"9999","scale":"3","step":"1"}
ch2o_value Integer {"unit":"mg/m3","min":"0","max":"9999","scale":"3","step":"1"}

And Instruction set:

Code Type Values
alarm_volume Enum {"range": ["low", "middle", "high", "mute"]}
alarm_time Integer {"unit": "s", "min": 1, "max": 60, "scale": 0, "step": 1}
alarm_switch Boolean "{true,false}"
alarm_bright Integer {"unit": "%", "min": 0, "max": 100, "scale": 0, "step": 1}

Sadly, in Home Assistant it this device is being recognized as having only:

So it's missing all othert sensors and actions

What version of Home Assistant Core has the issue?

core-2023.7.3

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

No response

Link to integration documentation on our website

https://www.home-assistant.io/integrations/tuya/

Diagnostics information

tuya-c7a83243205e3ca9260a803540f48447-Air detector-c5c5a2f0cf512cbb7d9d97d173c21240.json.txt

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

home-assistant[bot] commented 1 year ago

Hey there @tuya, @zlinoliver, @frenck, mind taking a look at this issue as it has been labeled with an integration (tuya) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `tuya` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign tuya` Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


tuya documentation tuya source (message by IssueLinks)

zevayo commented 1 year ago

Same problem here with the same device.

slavik73 commented 1 year ago

Same here (same device)

pacorreia commented 1 year ago

Same here (same device)

I reached out to @frenck in discord on this to ask if I could get some guidance to help fixing but he told me he couldn't.

I'm not sure if:

I really would like to have the remaining values to create an autonation with air purifier.

My guess is that there are other critical issues to deal with.

zevayo commented 1 year ago

With Local Tuya you can get all the sensors.

slavik73 commented 1 year ago

Localtuya somehow does not see my fingerbot with bluetooth gateway. But I will give it one more try as I changed HA installation to hassos from docker ....

вс, 1 окт. 2023 г. в 20:30, zevayo @.***>:

With Local Tuya you can get all the sensors.

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/core/issues/97563#issuecomment-1742146797, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABU7SKEKHJBIEYACDOBCDVTX5GSDVANCNFSM6AAAAAA27RRZLU . You are receiving this because you commented.Message ID: @.***>

pacorreia commented 1 year ago

With Local Tuya you can get all the sensors.

that's a good shout, but I'm curious why it doesn't work with built-in integration

pacorreia commented 1 year ago

With Local Tuya you can get all the sensors.

that's a good shout, but I'm curious why it doesn't work with built-in integration

@zevayo I did tried it, but it's giving an error trying to add the device, you had any issue? if so, could share any tips?

update : for some reason the device was not allowing connections, had to reset it, and then I was able to use with localtuya

pacorreia commented 1 year ago

In my case localtuya still does not give me all values like tvoc and hcho.

I guess I have to wait until this is sorted out

issue-triage-workflows[bot] commented 10 months ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

pacorreia commented 9 months ago

Not even a single response about this being fixable or not, and the result was being posed without any reply from main contributors

bricamac commented 9 months ago

I have same issue with AIR_DETECTOR (yrr3eiyiacm31ski) using module Tuya. I could see only co2, temperature , humidity but no PM messure .

igoramadas commented 8 months ago

Adding to the list of affected users. I have the same issue, only temp / humidity / co2 are visible.

pacorreia commented 8 months ago

I've a feeling that it won't get any kind of fix.

A segunda, 12/02/2024, 16:33, Igor Ramadas @.***> escreveu:

Adding to the list of affected users. I have the same issue, only temp / humidity / co2 are visible.

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/core/issues/97563#issuecomment-1939092265, or unsubscribe https://github.com/notifications/unsubscribe-auth/AM6OPV3NXA5BPQQGJH4ODADYTI75HAVCNFSM6AAAAAA27RRZLWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZZGA4TEMRWGU . You are receiving this because you authored the thread.Message ID: @.***>

igoramadas commented 8 months ago

I have created a ticket with Tuya directly asking them to validate (and apply) the following change: https://github.com/igoramadas/core/commit/95ee23747c22ddef7283b707de3d101ec5ef81d8

Hopefully this will come out on the next 2024.03 release.

igoramadas commented 8 months ago

Tuya is not very helpful. The device has a category "co2bj" which in theory should not support the extra sensors. The correct category is "dgnbj". But anyhow, the sensors are there and they even work in the Tuya / Smart Life apps. So I guess it should be fine to just extend "co2bj" with the new properties.

Anyone experienced with home-assistant development willing to test and possibly open a PR with the following changes: https://github.com/igoramadas/core/commit/95ee23747c22ddef7283b707de3d101ec5ef81d8

Unfortunately I won't have access to my hass Docker deployment at least for another 2 weeks, so I can't test this myself at the moment.