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

Characteristic value updates faster than asyncio schedule #240

Closed
superfashi opened this issue Jul 2, 2020 · 1 comment
Closed

Characteristic value updates faster than asyncio schedule #240

superfashi opened this issue Jul 2, 2020 · 1 comment
Assignees
Labels
Backend: Core Bluetooth Issues and PRs relating to the Core Bluetooth backend bug Something isn't working
Milestone

Comments

@superfashi
Copy link
Contributor

superfashi commented Jul 2, 2020

  • bleak version: 0.7.0
  • Python version: 3.7.0
  • Operating System: MacOS

Description

did_update_value_for_characteristic will potentially be called after the characteristic value has changed again due to asyncio scheduling, causing the callback not be able to receive the old data.

What I Did

This can be fixed by the pull request #241.

@hbldh hbldh added this to the Version 0.7.X milestone Jul 2, 2020
@hbldh hbldh self-assigned this Jul 2, 2020
@hbldh hbldh added bug Something isn't working Backend: Core Bluetooth Issues and PRs relating to the Core Bluetooth backend labels Jul 2, 2020
@hbldh
Copy link
Owner

hbldh commented Jul 2, 2020

This is an error I also encountered when using a MetaWear device with bleak. It needs to be fixed. Thank you for finding this!

@hbldh hbldh modified the milestones: Version 0.7.X, Version 0.7.1 Jul 2, 2020
@hbldh hbldh mentioned this issue Jul 2, 2020
@hbldh hbldh closed this as completed in a845d80 Jul 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend: Core Bluetooth Issues and PRs relating to the Core Bluetooth backend bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants