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

Make Ultrasonic plugin running async. #4171

Open
4 tasks
TD-er opened this issue Aug 2, 2022 · 0 comments
Open
4 tasks

Make Ultrasonic plugin running async. #4171

TD-er opened this issue Aug 2, 2022 · 0 comments
Labels
Category: Plugin Related to supported sensors Type: Enhancement Improve something already present

Comments

@TD-er
Copy link
Member

TD-er commented Aug 2, 2022

A number of fixes were done in this PR: #4166

Right now a ping call is waiting for the result and thus blocking.

We need to make this event driven, so the task will continue when the ping reply has been received.
However this makes things a bit more complex when having several ultrasonic tasks as you don't want several pings to be "in transit" or else the sensor may have no idea if the reflected signal is a result from its sent out ping or an echo from another sensor.

Still to do:

  • Check if several ping tasks can send out pings while other is still active
  • Actually test on ESP32 (and ESP8266) to get some idea of the improvements using direct GPIO access
  • Document plugin
  • Test several Ultrasonic sensor(s)
@TD-er TD-er added Type: Enhancement Improve something already present Category: Plugin Related to supported sensors labels Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Plugin Related to supported sensors Type: Enhancement Improve something already present
Projects
None yet
Development

No branches or pull requests

1 participant