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

Rethink shared state and how it triggers various threads. #175

Open
pritamghanghas opened this issue Mar 8, 2022 · 1 comment
Open

Rethink shared state and how it triggers various threads. #175

pritamghanghas opened this issue Mar 8, 2022 · 1 comment

Comments

@pritamghanghas
Copy link
Contributor

pritamghanghas commented Mar 8, 2022

At a minimum this will require some more documentation like why wifi_scan is triggered in bluetooth advertisement processor

  • Comments added to this class to clarify. Possibly changing the class name too.
  • Triggering advertisement from some of the wifi related characteristics
  • See if the threads are being trigger correctly and if all of them are even required.

Originally posted by @marvinmarnold in #174 (comment)

@shawaj
Copy link
Member

shawaj commented Mar 8, 2022

IIRC we originally did a scan at advertisement so that the results could be cached and ready for when connected.

That was a while ago though so maybe no longer relevant

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

No branches or pull requests

2 participants