Monitor-App icon indicating copy to clipboard operation
Monitor-App copied to clipboard

Suggestion: combine some of the sensors to reduce clutter :)

Open skynet01 opened this issue 4 years ago • 4 comments

I have 4 people and 3 sensors so my set up is not very crazy. I also want to add guests so I will probably have 10 total people.. which you guessed it, results in A LOT of sensors being created in HA. I suggest reducing the number of sensors to just binary sensors and a monitor state sensor. We can then move all the confidence information inside each person's sensor so it's still accessible if someone needs it.

Soo then we will only have the following:

sensor.monitor_state

  • primary display: show current monitor status
  • attribute: online nodes
  • attribute: offline nodes

binary_sensor.alias

  • primary display: show the current state
  • attribute: mac address
  • attribute: nearest node
  • attribute: list of nodes, their confidence, and timestamp of the update.

Again these are just minor suggestions, love this app!

skynet01 avatar Aug 10 '20 21:08 skynet01

This kind of makes sense, I understand, and not sure why I didn't think of it before really. If the binary sensor has attributes that states their confidence levels, that will actually be neater. Could be a serious breaking change for some folks. So what I could do, is to make it optional. It will be verbose as is for now as default, and allow for a setting. I personally will prefer this, since I don't use the confidence levels for anything (like most ppl).

Won't be easy though, thanks

Odianosen25 avatar Aug 11 '20 11:08 Odianosen25

Ah a serious issue to doing it this way, is that the confidence sensors are required before the binary sensors themselves are created. This might wait a while, really more complicated thinking of it as I am looking via the code.

Odianosen25 avatar Aug 11 '20 11:08 Odianosen25

Maybe they can live inside Appdaemon kind of like monitor.xxx sensors live?

skynet01 avatar Aug 11 '20 19:08 skynet01

They already living in AD. So if that’s the case, will move to using the AD internal ones for the confidence reading (which I had the idea to do before anyway, but got lazy). Then I will make the verbose on HA disables by default. That works thanks

Odianosen25 avatar Aug 12 '20 04:08 Odianosen25