-
Notifications
You must be signed in to change notification settings - Fork 194
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
Dev python bluez #295
base: master
Are you sure you want to change the base?
Dev python bluez #295
Conversation
I'll have to think on this. I understand the purpose, obviously, but one of the major advantages of this project is its small list of dependencies and its single-language source. |
I understand your hesitation and I also not 100% happy about involving another project/language. However, as I read and referenced, hcitool is depreciated and no longer maintained. Sooner or later this tool needs to be replaced. I have thought about using Whatever you decide, I have been playing about with this python script and I found a version which is ported to python3 (Prime consideration was that python2 will be EOL this year). It works even better and I am testing this. If you decide to implement this solution let me know. I upload this new solution working with python3 |
Thanks for the comment. I think the best solution is a complete re-write of |
One thing I can do is to write an bash script, that automates the installation of monitor and the python stuff. Maybe that is a comprise? |
@gadric I’d help you do that in python, I like the direction you’re going in, maybe a fork of this repo would be a next step so @andrewjfreyer can keep this one as is and a parallel version in python to compliment his version |
Update for python 3 and updated version of rssi.py
Update for python 3 version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have updated the python script to version 3.
And more, the new rssi.py script does not calculate the average, but the lowest rssi value. Was more precise. |
|
||
#install bluetooth-proximity | ||
cd bluetooth-proximity | ||
sudo python3 setup.py install | ||
|
||
#enter `monitor` directory | ||
cd monitor/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cd monitor/ | |
cd ../monitor/ |
I've installed this, yesterday I had good results with it but today I still face the same I'd also like to contribute but don't have a whole lot of experience. Output of
|
I found that the logic in use for rssi calculation was making an assumption about 0 values, which was the primary source of inaccuracy for me. I've actually opened #362 as a fix for that, which is a much lighter-weight change than this script - keeping an eye out for instability however. |
readme.MD: instructions to install bluetooth, python-bluez package and python-setuptools. clone ewenchou/bluetooth-proximity repo and install the script.
A new file rssi.py added to get rssi of mobile devices
monitor.sh script adapted to python-bluez and the rssi.py script.