Skip to content

Eliminate user retries on pod setup resumes (Loop issue #2117 follow up)#114

Merged
marionbarker merged 3 commits into
LoopKit:devfrom
itsmojo:pod-setup-resume-retry-prevention
Apr 2, 2024
Merged

Eliminate user retries on pod setup resumes (Loop issue #2117 follow up)#114
marionbarker merged 3 commits into
LoopKit:devfrom
itsmojo:pod-setup-resume-retry-prevention

Conversation

@itsmojo
Copy link
Copy Markdown

@itsmojo itsmojo commented Mar 8, 2024

  • Have PairAndPrime ViewModel do an automatic retry on error
  • Have InsertCannula ViewModel do an automatic retry on error
  • Add resumingPodSetup func to attempt a getStatus and sleep on errors
  • Added some improved and updated pumpManager comments
  • Have pumpManager detect pod setup resumes to invoke resumingPodSetup()
  • Include additional isConnected handling to OmniBLE resumingPodSetup()

itsmojo added 3 commits March 7, 2024 23:57
+ Have PairAndPrime ViewModel do an automatic retry on error
+ Have InsertCannula ViewModel do an automatic retry on error
+ Add resumingPodSetup func to attempt a getStatus and sleep on errors
+ Added some improved and updated pumpManager comments
+ Have pumpManager detect pod setup resumes to invoke resumingPodSetup()
+ Include additional isConnected handling to OmniBLE resumingPodSetup()
@marionbarker
Copy link
Copy Markdown
Collaborator

I did a code review and tested this PR.
It is successful.

Background:
With the previous PR, there were several times when a retry was required to get the setup process to continue.
See: #113 (comment)

Since the testing for PR 113, the rPi code was updated (when using PR 11 for that code), so the rPi simulator waits for priming and insertion to complete before progressing to the new pod state.

Therefore, this PR can be tested using just the rPI DASH simulator.

Test:
This code behaved nominally when resuming following

  • Quit and restart once priming had begun
    • Repeated the process during the priming and after priming completed
  • Quit and restart once the insert cannula bolus had begun
    • Repeated the process during the insertion bolus and after bolus completed

I repeated the pairing and inserting several times, since I can use the rPi DASH. Every time I returned to the pod menu, the process continued without needing to retry.

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.

2 participants