tuya / tuya-home-assistant

Home Assistant integration for controlling Powered by Tuya (PBT) devices using Tuya Open API, maintained by the Home Assistant Community and Tuya Developer Team.
MIT License
875 stars 206 forks source link

Ultrasonic level sensor #964

Open stefanobettipro opened 10 months ago

stefanobettipro commented 10 months ago

Device normal info

Device info (please complete the following information, which can be found in log): like this: { "active_time": 1699101903, "biz_type": 0, "category": "ywcgq", "create_time": 1698697618, "icon": "smart/icon/bay1610529967139OoVI/f4194654ecceabe31d6174fa1c51ff1c.png", "id": "bf7d88e7cb634381730xsl", "ip": "", "lat": "", "local_key": "", "lon": "", "model": "EPT_Ultrasonic level sensor", "name": "Serbatoio acqua San Lorenzo", "online": true, "owner_id": "71576573", "product_id": "wtzwyhkev3b4ubns", "product_name": "Tank A Level", "status": [ { "code": "liquid_state", "value": "upper_alarm" }, { "code": "liquid_depth", "value": 69 }, { "code": "max_set", "value": 90 }, { "code": "mini_set", "value": 10 }, { "code": "installation_height", "value": 800 }, { "code": "liquid_depth_max", "value": 120 }, { "code": "liquid_level_percent", "value": 100 } ], "sub": false, "time_zone": "+01:00", "uid": "***", "update_time": 1699585130, "uuid": "1437d065c7ef82dc" },

Device specifications (please complete the following information, which can be found in log): Same device's id, like this: 2023-11-12 18:15:05.494 DEBUG (SyncWorker_23) [tuya_iot] Request: method = GET, url = https://openapi.tuyaeu.com/v1.0/devices/bf7d88e7cb634381730xsl/specifications, params = None, body = None, t = 1699809305494 2023-11-12 18:15:05.583 DEBUG (SyncWorker_23) [tuya_iot] Response: { "result": { "category": "ywcgq", "functions": [ { "code": "max_set", "type": "Integer", "values": "{\"unit\":\"%\",\"min\":0,\"max\":100,\"scale\":0,\"step\":1}" }, { "code": "mini_set", "type": "Integer", "values": "{\"unit\":\"%\",\"min\":0,\"max\":100,\"scale\":0,\"step\":1}" }, { "code": "installation_height", "type": "Integer", "values": "{\"unit\":\"m\",\"min\":200,\"max\":2500,\"scale\":3,\"step\":1}" }, { "code": "liquid_depth_max", "type": "Integer", "values": "{\"unit\":\"m\",\"min\":100,\"max\":2400,\"scale\":3,\"step\":1}" } ], "status": [ { "code": "liquid_state", "type": "Enum", "values": "{\"range\":[\"normal\",\"lower_alarm\",\"upper_alarm\"]}" }, { "code": "liquid_depth", "type": "Integer", "values": "{\"unit\":\"m\",\"min\":0,\"max\":10000,\"scale\":2,\"step\":1}" }, { "code": "max_set", "type": "Integer", "values": "{\"unit\":\"%\",\"min\":0,\"max\":100,\"scale\":0,\"step\":1}" }, { "code": "mini_set", "type": "Integer", "values": "{\"unit\":\"%\",\"min\":0,\"max\":100,\"scale\":0,\"step\":1}" }, { "code": "installation_height", "type": "Integer", "values": "{\"unit\":\"m\",\"min\":200,\"max\":2500,\"scale\":3,\"step\":1}" }, { "code": "liquid_depth_max", "type": "Integer", "values": "{\"unit\":\"m\",\"min\":100,\"max\":2400,\"scale\":3,\"step\":1}" }, { "code": "liquid_level_percent", "type": "Integer", "values": "{\"unit\":\"%\",\"min\":0,\"max\":100,\"scale\":0,\"step\":1}" } ] }, "success": true, "t": 1699809311062, "tid": "0933b8fa817f11eeab249a6504bbc951" }

randvoorhies commented 7 months ago

I have a WIP branch for support here: https://github.com/home-assistant/core/compare/dev...randvoorhies:core:tuya-ywcgq-tank-level-sensor

I'm hoping I can get a friendly dev to review, as this will be my first contribution to HA.

Pro commented 6 months ago

@randvoorhies I took your changes and put them into a pull request: https://github.com/home-assistant/core/pull/114481

axute commented 6 months ago

I think you need to remove the draft status, but I'm not entirely sure.

Pro commented 6 months ago

I think you need to remove the draft status, but I'm not entirely sure.

Jep, will do that after I was able to test the changes locally and the necessary checklist done. Will get the sensor in a few days.

If you have the sensor already, feel free to test the changes and leave comments in the pull request.

stefanobettipro commented 6 months ago

What a wander!

s.

Dott. Stefano Betti Via 4 Novembre 11 20842 Besana in Brianza (MB)

Il giorno lun 1 apr 2024 alle 10:25 Stefan Profanter < @.***> ha scritto:

I think you need to remove the draft status, but I'm not entirely sure.

Jep, will do that after I was able to test the changes locally and the necessary checklist done. Will get the sensor in a few days.

If you have the sensor already, feel free to test the changes and leave comments in the pull request.

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

vmasmitja commented 5 months ago

Hi @stefanobettipro! Have you seen the PR updates? https://github.com/home-assistant/core/pull/114481. Maybe can be your solution. Seems the same issue as described here https://community.home-assistant.io/t/something-is-wrong-with-tuya-tank-level-sensors-with-the-new-official-integration/689321.