-
Notifications
You must be signed in to change notification settings - Fork 8
Added a 'wait' argument to control how long the radio waits to receive a message #16
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
Conversation
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.
Thanks for making this enhancement. It looks like there was an issue with the docs building, and I have a few other suggestions
Thanks again @FoamyGuy for your feedback! I made the changes on my branch, but I'm not sure if I need to close this request and open a new one, or is it possible to share my newest commits within this request? |
@mlupo thanks for working on that. They are updated here automatically once pushed into your branch so there is no need to close / make a new PR. This one contains all of the latest changes now. I will circle back and look this over again this evening. |
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.
Thanks for working on this @mlupo! These changes look good to me. I tested them successfully with a modified version of the simpletest script on a CLUE and CPB devices.
Woo! Thanks again for your help @FoamyGuy! |
@mlupo you are welcome! Thanks again for the enhancement. Congrats if this was your first contribution to a CircuitPython Library🎉 |
Updating https://github.com/adafruit/Adafruit_CircuitPython_BLE_Radio to 0.4.0 from 0.3.5: > Merge pull request adafruit/Adafruit_CircuitPython_BLE_Radio#16 from mlupo/main > Globally disabled consider-using-f-string pylint check > Moved default branch to main > Moved CI to Python 3.7 > Added help text and problem matcher > Added pull request template Updating https://github.com/adafruit/Adafruit_CircuitPython_IterTools to 1.1.6 from 1.1.5: > Merge pull request adafruit/Adafruit_CircuitPython_IterTools#14 from earthprelude/add-init-file > Globally disabled consider-using-f-string pylint check > Moved default branch to main > Moved CI to Python 3.7 > Added help text and problem matcher > Added pull request template > "Increase duplicate code check threshold "
In a project using a few Circuit Playground Bluefruit boards, I needed a way to control how long each board waited for a new message. With the edits in this request, it is possible to pass a
wait
keyword argument to anyreceive
call.Usage, in my case:
radio.receive(wait=0.2)
I am not a programmer by trade, so my apologies if this is absolutely not the right way to do a pull request!