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

Rapid Reconnect #3829

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Navid200
Copy link
Collaborator

@Navid200 Navid200 commented Dec 25, 2024

This PR makes it much easier to establish connectivity to a G7.

A G7 that loses connection goes into Rapid Reconnect mode after 15 minutes. In this mode, it transmits each reading 5 times at 1-minute intervals. If xDrip is attempting to connect for the very first time to this G7, it could pair on any of those 5 transmissions. But, only one of those 5 is on the once-every-5-minute time grid. The other 4 are not.

After there is a pairing, G7 stops Rapid Reconnect immediately and only transmits once every 5 minutes.

If xDrip pairs on any of the 4 transmissions that are not on the once-every-5-minute grid, the xDrip anticipate routine will set it to wake up in 5 minutes when there will be no transmission from G7.
If the user waits patiently for several read cycles for xDrip to enable always scan mode, it will eventually discover the correct time grid assuming it does it before G7 goes back into Rapid Reconnect again.

The problem is that many users will intervene thinking that something is wrong. So, they restart the phone or restart collector etc. Sometimes, these will succeed. Sometimes, they won't.

I wrote this hoping to help: #3813

The problem is that you cannot expect everyone to read all the guides.

This PR ensures connectivity taking into account everything that could possibly go wrong.


Testing details to be added soon.

@Navid200 Navid200 marked this pull request as draft December 26, 2024 18:18
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