StyraHem / ShellyForHASS

Shelly smart home platform for Home Assistant
MIT License
620 stars 111 forks source link

[BUG] Shelly Gas concentration (ppm) sensor entity not showing #540

Open emanueleprina opened 3 years ago

emanueleprina commented 3 years ago

Environment

Describe the bug

Shelly Gas sensor entity concentration (gas level in ppm units) is not added to the device.

Steps to Reproduce

Expected behavior

Shelly for Hass should recognize and attach the concentration (ppm) sensor to the Shelly gas device entity

Screenshots

immagine

Traceback/Error logs

Maybe due to the Type "T:C" (guess stands for "concentration") instead of "T:S" (for sensor) in coap string as below:

{"I":3107,"T":"C","D":"concentration","U":"ppm","R":["U16","-1"],"L":1}

Full string:

CoAP Code: 69, Type SHGS-1, Id 50029178B643, Payload {"blk":[{"I":1,"D":"sensor_0"},{"I":2,"D":"valve_0"},{"I":3,"D":"device"}],"sen":[{"I":9103,"T":"EVC","D":"cfgChanged","R":"U16","L":3},{"I":3113,"T":"S","D":"sensorOp","R":["warmup/normal/fault","unknown"],"L":1},{"I":3114,"T":"S","D":"selfTest","R":"not_completed/completed/running/pending","L":1},{"I":6108,"T":"A","D":"gas","R":["none/mild/heavy/test","unknown"],"L":1},{"I":3107,"T":"C","D":"concentration","U":"ppm","R":["U16","-1"],"L":1},{"I":1105,"T":"S","D":"valve","R":["closed/opened/not_connected/failure/closing/opening/checking","unknown"],"L":2}]}

Additional context

mrandt commented 3 weeks ago

Same issue here.

I'd like to see the PPM in Home Assistant but the only sensor I have is the alarm state.

I found an option to display PPM in ShellyForHass settings (see screenshot), enabled it and deleted the Shelly Gas device. After recreating, still the PPM do not show.

image

ShellyForHASS release with the issue: 1.0.5 Last working ShellyForHASS release (if known): unknown Home Assistant Core release with the issue: core-2024.8.2 Operating environment (Home Assistant/Supervised/Docker/venv): Docker