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

fix: Handling retired or changed scanners #360

Merged
merged 1 commit into from
Nov 7, 2024
Merged

fix: Handling retired or changed scanners #360

merged 1 commit into from
Nov 7, 2024

Conversation

agittins
Copy link
Owner

@agittins agittins commented Nov 7, 2024

  • Refactor _refresh_scanners to be more efficient, and to handle the case where a scanner still exists but is no longer a scanner.
  • Refactor coordinator.init() to move callbacks out into class methods.
  • Tamed down lots of debug logging
  • Altered self.last_config_entry_update to use MONOTONIC_TIME(), and changed cooldown to a const of 10 seconds, and included check for parallel save tasks.
  • Removed assert that excluded scanners from area-by-mindist

- Refactor _refresh_scanners to be more efficient, and to handle the case where a scanner still exists but is no longer a scanner.
- Refactor coordinator.init() to move callbacks out into class methods.
- Tamed down lots of debug logging
- Altered self.last_config_entry_update to use MONOTONIC_TIME(), and changed cooldown to a const of 10 seconds, and included check for parallel save tasks.
- Removed assert that excluded scanners from area-by-mindist
@agittins agittins merged commit eee3e0e into main Nov 7, 2024
6 checks passed
@agittins agittins deleted the agittins-dev branch November 7, 2024 11:19
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

Successfully merging this pull request may close these issues.

1 participant