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

New plugin: SR04 range finder #49

Closed
enesbcs opened this issue Mar 15, 2019 · 4 comments
Closed

New plugin: SR04 range finder #49

enesbcs opened this issue Mar 15, 2019 · 4 comments
Labels
awaiting for feedback More details or tests needed enhancement New feature or request

Comments

@enesbcs
Copy link
Owner

enesbcs commented Mar 15, 2019

No description provided.

@enesbcs enesbcs self-assigned this Mar 15, 2019
@enesbcs enesbcs added enhancement New feature or request work in progress It's currently in focus labels Mar 15, 2019
@TD-er
Copy link

TD-er commented Mar 15, 2019

Please note that some of these need some extra electronics like a level converter.
Or else you may damage GPIO pins.
And for the Arduino environment we use the "newping" library, which appeared to be way more stable compared to the previous implementation.

@enesbcs
Copy link
Owner Author

enesbcs commented Mar 15, 2019

I know, already used an SR04 with ESPEasy for presence detection in restroom, with a voltage divider. (2 resistor)
For Linux/Python the newping library is not an option, and i think a multitasking Linux OS will not provide as accurate timings as an ESP8266/Arduino hardware does. Despite that my first test results seems fine in pure python. Now i will test it with multithreading code.

@enesbcs
Copy link
Owner Author

enesbcs commented Mar 15, 2019

Support added in commit 3d697ce. Seems to works ok in multithreaded environment. I have experienced 1-4mm differences between measures with physically same distances. There were some big values in multiple measures, but with 5-8 samples at once the plugin can filter them - mostly.

@enesbcs enesbcs added awaiting for feedback More details or tests needed and removed work in progress It's currently in focus labels Mar 15, 2019
@enesbcs enesbcs removed their assignment Mar 15, 2019
@enesbcs
Copy link
Owner Author

enesbcs commented Mar 30, 2019

Implementation issue is closed now.

@enesbcs enesbcs closed this as completed Mar 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting for feedback More details or tests needed enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants