majuss / lupupy

A python3 library to control the Lupusec alarm control panel.
MIT License
9 stars 11 forks source link

Adds temperature and humidity sensor #17

Open FL550 opened 1 month ago

FL550 commented 1 month ago

This adds a new sensor type, which provides its status as '24.8°C 48%' or with separate attributes temperature and humidity.

FL550 commented 1 month ago

I'll also have a working update for the Homeassistant plugin. I'll make a PR when you have released a new version of this package with my changes.

image

suaveolent commented 1 week ago

Thanks, for the PR.

Can you maybe also post a sample response from the device? I'm wondering why there is a need for a regular expression.

From what I understand from https://github.com/majuss/lupupy/issues/16, the values should be readable as json variables temp and humidity directly.

FL550 commented 1 week ago

Very strange, I swear it wasn't there before 😅

You are right, I'll adjust my PR to use these values directly.