Skip to content

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

Merged
merged 4 commits into from
Sep 28, 2021

Conversation

mlupo
Copy link

@mlupo mlupo commented Aug 29, 2021

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 any receive 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!

@mlupo mlupo marked this pull request as ready for review September 24, 2021 20:38
Copy link

@FoamyGuy FoamyGuy left a 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

@mlupo
Copy link
Author

mlupo commented Sep 27, 2021

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?

@FoamyGuy
Copy link

@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.

Copy link

@FoamyGuy FoamyGuy left a 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.

@FoamyGuy FoamyGuy merged commit cde6d01 into adafruit:main Sep 28, 2021
@mlupo
Copy link
Author

mlupo commented Sep 28, 2021

Woo! Thanks again for your help @FoamyGuy!

@FoamyGuy
Copy link

@mlupo you are welcome! Thanks again for the enhancement. Congrats if this was your first contribution to a CircuitPython Library🎉

adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Sep 28, 2021
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 "
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

Successfully merging this pull request may close these issues.

2 participants