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.

Sign up for GitHub

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

Jump to bottom

bluetooth/aioble/examples/temp_sensor.py: Properly notifies on update. #835

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

bluetooth/aioble/examples/temp_sensor.py: Properly notifies on update. #835

wants to merge 2 commits into from

Conversation

mores
Copy link

@mores mores commented Mar 25, 2024

No description provided.

@mores
bluetooth/aioble/examples/temp_sensor.py: Properly notifies on update.
1331613 
Signed-off-by: Stephen More <stephen.more@gmail.com>
@mores
bluetooth/aioble/examples/temp_sensor.py: Sets timeout to none.
cb89e33 
Signed-off-by: Stephen More <stephen.more@gmail.com>
@konistehrad
Copy link

I think this should be merged. The timeout_ms=None call in particular keeps something deep in the internals from blocking service queries from the central, and without this the central doesn't get timely updates.

@aetelani
Copy link

Thanks for example update. I was doing this myself because the current example in master was not working.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@mores @konistehrad @aetelani