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

Periodic discovery #61

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Periodic discovery #61

wants to merge 2 commits into from

Conversation

azymohliad
Copy link
Owner

@azymohliad azymohliad commented Mar 2, 2024

A very basic periodic discovery is implemented. The current logic is the following:

  • Periodic discovery is used only when running in the background, otherwise (if Devices view is open) it scans continuously as before.
  • The intervals are hard-coded for now: 5 seconds scanning, 60 seconds pause.

TODO:

  • Review the logic: is it adequate to switch between periodic and continuous discovery based solely on the UI visibility? Are there cases where we want periodic discovery in the foreground, or continuous discovery in the background?
  • Review the default intervals. Should they be constant or ramp up over time? Should the first scan duration after startup or loosing the connection be longer than others?
  • Add interval settings (maybe without a UI, gsettings backend only).
  • Test thoroughly.
  • (maybe) Make a Flathub CI build and ask more people to test before releasing.

@azymohliad
Copy link
Owner Author

Waiting to get some answers and gather a bit more information before continuing:

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.

None yet

1 participant