lorenzo-deluca / espkyogate

Serial Bridge for Bentel Kyo Alarm System.
GNU Affero General Public License v3.0
40 stars 13 forks source link

Configuration trouble #3

Closed brusarp closed 2 years ago

brusarp commented 2 years ago

Hello Lorenzo, I am a recent and happy user of this integration. This isn't really an issue. I'm trying to configure all areas and zones in the most correct and total way but something is not right ... I noticed that if in the configuration I insert some areas or zones incorrectly in the home assistant "wrong" sensors are created (in a test yesterday, for example, "fuse fault" appeared to me as active but it was not true ...) it would not Is it possible to insert in the readme or in the espkyogate_configuraton.yaml file a configuration model where everything is commented? (currently for example there is only "Zone 1" configured) In this way people would have to do nothing but remove the comments they need to their taste ... I don't know if I made myself clear ...

lorenzo-deluca commented 2 years ago

Hello, it is a good tip. I think your false "fuse fault" report may be due to the order of the sensors put in the configuration... Possibly you can post your configuration, only the binary_sensor section, of course...?

brusarp commented 2 years ago

Sure, here you are:

`binary_sensor:

So if the problem is the order then I think it is all the more reason to "study" how to create a model where no line must be deleted otherwise it will skip everything ...

lorenzo-deluca commented 2 years ago

Hi, As I said, I'm not much of an expert on ESPHome. In any case, from what I understand, the component configuration must be consistent between the return and the sensors being passed. In your case it seems to me everything is ok until memoria_allarme_zona_1. After that, as you can see, after that, as you can see, there is no congruence... right? For example, memoria_sabotaggio_zona_1 it's not declared in return scope.

Can you check, fix and try again?

I agree with you that it's not exactly super simple, it can probably be done as you say but from the documentation on how to develop custom ESPHome components I couldn't find a smarter way.

brusarp commented 2 years ago

Ok, checked and you are right (wrong declaration). At this point I would say that to help you find a solution I will try to make a pull request with my idea...

lorenzo-deluca commented 2 years ago

Ok, checked and you are right (wrong declaration). At this point I would say that to help you find a solution I will try to make a pull request with my idea...

Great idea! Every pull request is welcome!

brusarp commented 2 years ago

Hello @lorenzo-deluca ! I sent you an email about this issue... I don't know if you've seen it... In short I can't get the Wemos D1 mini to work with all lines enabled...