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

Conceptual problems #1

Open
DavidLP opened this issue Oct 23, 2016 · 0 comments
Open

Conceptual problems #1

DavidLP opened this issue Oct 23, 2016 · 0 comments

Comments

@DavidLP
Copy link

DavidLP commented Oct 23, 2016

I really like the reverse engineering effort and the effort to make the results available to an open source community. Thank you for that!

What I do not like are the dependencies of this project, which makes the package very difficult to use. It uses the (to my mind) unnecessary pygattlib library. Pygattlib is not a wrapper around gatttlib (a tool from the bluez package) but more a fork of it, since it ships and compiles its own (outdated!) version.

To ease the installation I would recommend to make this a pure python package (including dependencies) by just calling gatttlib in a subprocess. This would also make testing much simpler. Check e.g. this library that does the same with a different Ble thermostat device.

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

1 participant