search

gSpotx2f / luci-app-temp-status

Temperature sensors data for the LuCI status page (OpenWrt webUI)
MIT License
115 stars 35 forks source link

Remove duplicates #5

Closed remittor closed 9 months ago

remittor commented 10 months ago

First variart: https://github.com/openwrt-xiaomi/luci-app-temp-status/commit/ac0150cbd0bd43172beab437f766fa1d641e9342 Second variant (best): https://github.com/openwrt-xiaomi/luci-app-temp-status/commit/456bbee1aeaa3bc423a9ef8db93ca501fcbb53a7

gSpotx2f commented 10 months ago

Hi! Unfortunately, I don't have a device with duplicate tsensors. If you don't mind, please test this version of /usr/libexec/rpcd/luci.temp-status.

/usr/libexec/rpcd/luci.temp-status call getTempStatus
ubus call luci.temp-status getTempStatus
remittor commented 10 months ago

Да. Ваш вариант тоже отлично работает. Придётся мне заново в свой репо коммитить.

Так же посмотрите на коммит Grouping all sensors with the name "sensor*_thermal" into one line : https://github.com/openwrt-xiaomi/luci-app-temp-status/commits/master/ Он делает вот такое: image

(мин значение - медианное - макс значение) А то надоело 10 сенсоров глазами смотреть.

remittor commented 10 months ago

Всё же решил сделать немного иначе: image

Вот сам код: https://github.com/openwrt-xiaomi/luci-app-temp-status/commit/a549cf4ca1d8d57526a673aa366103d18a7911bd