Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ble_gateway: Document discovery mode #16

Open
tbaumann opened this issue Aug 11, 2022 · 3 comments
Open

ble_gateway: Document discovery mode #16

tbaumann opened this issue Aug 11, 2022 · 3 comments

Comments

@tbaumann
Copy link

I just came across the option discovery: true

I think this is pretty much the default behavior anyone would expect of a BLE gateway. And it's a lot more easy to understand than the currently documented method with the text_sensor.

I think this option should be mentioned in the documentation and examples.

My minimal config is this.

ble_gateway:
  id: $device_name
  discovery: true
  on_ble_advertise:
    then:
    - homeassistant.service:
        service: ble_monitor.parse_data
        data:
          packet: !lambda return packet;
          gateway_id: $device_name

Add this to any esphome device and receive BLE advertisements. Simple.

@myhomeiot
Copy link
Owner

myhomeiot commented Aug 11, 2022

I will think how to add this into documentation to make things clear.

Main disadvantage of automatic discovery it's if you are living in environment with many BLE devices around it's can overload Passive BLE Monitor. Earlier versions of Passive BLE Monitor in my case takes 50-60% CPU (instead 5-7%) for packet processing when ESP sends all BLE advertisement packets.

@tbaumann
Copy link
Author

Oh the load issues are surprising. Passive BLE Monitor, when using a local HCI interface also default to listening to everything AFAIK.

Hard to say what's the better default I guess.

@myhomeiot
Copy link
Owner

Yes, just found that now Passive BLE Monitor listening and parsing all packets because auto discovery enabled by default, probably the problem with heavy CPU load was solved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants