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

Rewrite bluetooth le #16592

Merged
merged 9 commits into from
Sep 14, 2018
Merged

Rewrite bluetooth le #16592

merged 9 commits into from
Sep 14, 2018

Conversation

pvizeli
Copy link
Member

@pvizeli pvizeli commented Sep 13, 2018

Description:

The old python gattlib work not anymore with newer python version and make a lot of problems on RPi with performance. I see this code: https://community.home-assistant.io/t/ble-custom-component/48355

I rewrite some parts. I hope this platform will be work suitable after this update.

home-assistant/home-assistant.io#6273

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • New or updated dependencies have been added to requirements_all.txt by running

result = mac in devs
if not result:
# Could not lookup device name
for mac in devs_to_track:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trailing whitespace

@pvizeli pvizeli changed the title WIP: Rewrite bluetooth le Rewrite bluetooth le Sep 13, 2018
@ghost ghost assigned balloob Sep 14, 2018
@balloob balloob merged commit 7705666 into dev Sep 14, 2018
@ghost ghost removed the in progress label Sep 14, 2018
@balloob balloob deleted the fix-bluetooth-le branch September 14, 2018 11:49
@balloob balloob added this to the 0.78.0 milestone Sep 15, 2018
balloob pushed a commit that referenced this pull request Sep 15, 2018
* Rewrite bluetooth le

* Update requirements_all.txt

* Update gen_requirements_all.py

* Update bluetooth_le_tracker.py

* Update bluetooth_le_tracker.py

* Update bluetooth_le_tracker.py

* Update bluetooth_le_tracker.py

* Update bluetooth_le_tracker.py

* Update bluetooth_le_tracker.py
@balloob balloob mentioned this pull request Sep 17, 2018
@home-assistant home-assistant locked and limited conversation to collaborators Feb 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants