vlebourl / custom_vesync

Custom VeSync component for Home Assistant
96 stars 59 forks source link

Warning about native unit of measurement for sensors #164

Closed aIexus closed 9 months ago

aIexus commented 9 months ago

The problem

The problem

Warning about native unit of measurement for sensors received

What version of this integration has the issue?

1.1.1

What version of Home Assistant Core has the issue?

2023.9.2

Diagnostics

Diagnostics ```json { "home_assistant": { "installation_type": "Home Assistant Supervised", "version": "2023.9.2", "dev": false, "hassio": true, "virtualenv": false, "python_version": "3.11.5", "docker": true, "arch": "aarch64", "timezone": "Europe/Kiev", "os_name": "Linux", "os_version": "6.1.21-v8+", "supervisor": "2023.09.2", "host_os": "Debian GNU/Linux 12 (bookworm)", "docker_version": "24.0.5", "chassis": "", "run_as_root": true }, "custom_components": { "vesync": { "version": "1.1.0", "requirements": [ "pyvesync==2.1.10" ] }, "smartir": { "version": "1.17.7", "requirements": [ "aiofiles>=0.6.0" ] }, "hacs": { "version": "1.32.1", "requirements": [ "aiogithubapi>=22.10.1" ] }, "reversotts": { "version": "1.0.2", "requirements": [ "pyttsreverso==0.4" ] }, "tapo_control": { "version": "5.2.4", "requirements": [ "pytapo==3.2.14" ] } }, "integration_manifest": { "domain": "vesync", "name": "VeSync", "codeowners": [ "@markperdue", "@webdjoe", "@thegardenmonkey", "@vlebourl" ], "config_flow": true, "dhcp": [ { "hostname": "levoit-*", "macaddress": "*" } ], "documentation": "https://www.home-assistant.io/integrations/vesync", "iot_class": "cloud_polling", "issue_tracker": "https://github.com/vlebourl/custom_vesync", "requirements": [ "pyvesync==2.1.10" ], "version": "1.1.0", "is_built_in": false }, "data": { "fans": [ { "device_name": "LR-Levoit Air Purifier 300S", "device_image": "https://image.vesync.com/defaultImages/deviceDefaultImages/wifibtonboardingnotify_airpurifier_core300s_eu_240.png", "cid": "**REDACTED**", "connection_status": "online", "connection_type": "WiFi+BTOnboarding+BTNotify", "device_type": "Core300S", "type": "wifi-air", "uuid": "**REDACTED**", "config_module": "WiFiBTOnboardingNotify_AirPurifier_Core300S_EU", "mac_id": "**REDACTED**", "mode": "sleep", "speed": 2, "extension": { "airQuality": -1, "airQualityLevel": 1, "mode": "sleep", "fanSpeedLevel": "255" }, "current_firm_version": null, "device_region": "EU", "pid": null, "sub_device_no": null, "config": { "display": true, "display_forever": true }, "device_status": "off", "enabled": false, "config_dict": { "module": "VeSyncAirBypass", "models": [ "Core300S", "LAP-C301S-WJP", "LAP-C302S-WUSB" ], "modes": [ "sleep", "off", "auto", "manual" ], "features": [ "air_quality" ], "levels": [ 1, 2, 3, 4 ] }, "features": [ "air_quality" ], "modes": [ "sleep", "off", "auto", "manual" ], "air_quality_feature": true, "details": { "filter_life": 91, "mode": "manual", "level": 0, "display": false, "child_lock": false, "night_light": "off", "air_quality": 1, "display_forever": false, "air_quality_value": 10 }, "timer": null } ] } } ```

Home Assistant log

Logs ```py 2023-09-14 13:26:21.401 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.lr_levoit_air_purifier_300s_air_quality () is using native unit of measurement ' ' which is not a valid unit for the device class ('aqi') it is using; expected one of ['no unit of measurement']; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author. 2023-09-14 13:26:21.410 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.lr_levoit_air_purifier_300s_air_quality_value () is using native unit of measurement ' ' which is not a valid unit for the device class ('aqi') it is using; expected one of ['no unit of measurement']; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author. ```

Additional information

No response

vlebourl commented 9 months ago

Unfortunately the fix causing this warning is necessary to keep trigger showing up correctly in automations (cf. https://github.com/home-assistant/core/issues/100361). So for now, although I agree it's bug, it will stay that way. Sorry for the inconvenience.