jziolkowski / tdm

GUI application to discover and monitor devices flashed with https://github.com/arendst/tasmota
GNU General Public License v3.0
667 stars 84 forks source link

tdmgr - Generates lots of {"Command":"Unknown"} #288

Closed Noschvie closed 1 month ago

Noschvie commented 2 months ago

Hi @jziolkowski the latest version generate the logs below, version tdmgr_0.2.13 doesn't.

09:16:30.512 MQT: stat/tasmota_F982B0/RESULT = {"GPIOs2":{"10016":"ME007 Tri","10048":"ME007 Rx","10080":"TuyaMCUBr Tx","10112":"TuyaMCUBr Rx","10144":"BioPDU PZEM0XX Tx","10176":"BioPDU PZEM016 Rx","10208":"BioPDU Bit","10240":"MCP23xxx Int","10272":"MCP23Sxx CS","10304":"PCF8574 Int","10336":"LoxO2 RX","10368":"GM861 Tx","10400":"GM861 Rx","10432":"Dingtian OE","10464":"HDMI CEC","10496":"HC8 Rx","10528":"I2S DAC","10560":"MagicSwitch","10592":"Pipsolar TX","10624":"Pipsolar RX","10656":"LoRa CS","10688":"LoRa Rst","10720":"LoRa Busy","10752":"LoRa DIO0","10784":"LoRa DIO1","10816":"LoRa DIO2","10848":"LoRa DIO3","10880":"LoRa DIO4","10912":"LoRa DIO5","10944":"TS SPI CS","10976":"TS RST","11008":"TS IRQ","11040":"RN2XX3 Tx","11072":"RN2XX3 Rx","11104":"RN2XX3 Rst","11136":"TCP Tx En","11168":"ASR650x Tx","11200":"ASR650x Rx","11232":"Wooliis Rx","11264":"ADC Voltage","11296":"ADC Current"}}
09:16:30.531 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:30.731 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:30.931 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:31.136 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:31.336 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:31.537 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:31.739 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:31.990 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:32.236 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:32.436 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:32.638 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:32.839 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:33.043 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:33.243 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:33.443 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
09:16:33.644 MQT: stat/tasmota_F982B0/RESULT = {"Command":"Unknown"}
jziolkowski commented 2 months ago

This is not a bug per se.

Tasmota has somewhat limited capabilities in reporting which features are enabled and what is their state. For example shutters.

For that reason, TDM queries devices with certain commands at the beginning. If a device doesn't have these features then you see what you see.

As time goes on, in newer firmwares when possible Theo adds certain data to the status payloads so that I don't have to do it, but there are always devices in the wild that are not up to date.

As for why the previous release didn't have as much: newer release queries for more features.

So thanks for being vigilant, but this is a feature 😉

Noschvie commented 2 months ago

I don't like this feature.

Maybe you can disable this feature for "newer firmwares".

jziolkowski commented 2 months ago

I'd argue it doesn't harm anyone/anything, just some noise.

However I'll have a look about less lazy approach. No promises though.

jziolkowski commented 1 month ago

Go ahead and test: https://github.com/jziolkowski/tdm/releases/tag/2024.10.1.1

Noschvie commented 1 month ago

Tested with v2024.10.1.1 and the {"Command":"Unknown"} is gone. Well done, thanks again!

jziolkowski commented 1 month ago

Thanks for the feedback (and coffee ;) )