We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Should check return value of .wait_for_active(timeout=<input timeout>) calls to see if they timed out or not, and raise an exception if so.
.wait_for_active(timeout=<input timeout>)
The .wait_for_active() methods of gpiozero devices call the .wait() method of an internal threading.Event (see https://gpiozero.readthedocs.io/en/stable/_modules/gpiozero/mixins.html) and that methods returns True if the Event was set or False if it wasn't (i.e. it timed out): https://docs.python.org/3/library/threading.html#threading.Event
.wait_for_active()
gpiozero
.wait()
threading.Event
True
Event
False
Will need to address #42 prior to this otherwise it will break the simulator (which currently relies on timeouts).
This will be needed as part of #39.
The text was updated successfully, but these errors were encountered:
Oh, this is effectively a duplicate of #32
Sorry, something went wrong.
You capture some good information here so would be good to incorporate into #32.
fergusL
No branches or pull requests
Should check return value of
.wait_for_active(timeout=<input timeout>)
calls to see if they timed out or not, and raise an exception if so.The
.wait_for_active()
methods ofgpiozero
devices call the.wait()
method of an internalthreading.Event
(see https://gpiozero.readthedocs.io/en/stable/_modules/gpiozero/mixins.html) and that methods returnsTrue
if theEvent
was set orFalse
if it wasn't (i.e. it timed out): https://docs.python.org/3/library/threading.html#threading.EventWill need to address #42 prior to this otherwise it will break the simulator (which currently relies on timeouts).
This will be needed as part of #39.
The text was updated successfully, but these errors were encountered: