caspervk/AndroidAPS
1
0
Fork 0
Code Releases Activity

Combo: retry twice to establish connection before giving up.

Browse source 
(cherry picked from commit a16560a)
This commit is contained in:
Johannes Mockenhaupt 2018-02-01 00:06:29 +01:00
parent 26e7834093
commit 1f06d801a2
No known key found for this signature in database
GPG key ID: 9E1EA6AF7BBBB0D1
1 changed files with 5 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
app/src/main/java/info/nightscout/androidaps/plugins/PumpCombo/ComboPlugin.java
View file

@ -824,6 +824,9 @@ public class ComboPlugin implements PluginBase, PumpInterface, ConstraintsInterf
private CommandResult runOnConnectChecks() { private CommandResult runOnConnectChecks() {
// connect, get status and check if an alarm is active // connect, get status and check if an alarm is active
CommandResult preCheckResult = ruffyScripter.readPumpState(); CommandResult preCheckResult = ruffyScripter.readPumpState();
for (int retries = 2; !preCheckResult.success && retries > 0; retries--) {
preCheckResult = ruffyScripter.readPumpState();
}
if (!preCheckResult.success) { if (!preCheckResult.success) {
return preCheckResult; return preCheckResult;
} }
@ -1116,6 +1119,8 @@ public class ComboPlugin implements PluginBase, PumpInterface, ConstraintsInterf
return null; return null;
} }
// TODO still regularly continues even though no changes on pump exist ...
// OPTIMIZE this reads the entire history on start, so this could be optimized by persisting // OPTIMIZE this reads the entire history on start, so this could be optimized by persisting
// `timestampOfLastKnownPumpBolusRecord`, though this should be thought through, to make sure // `timestampOfLastKnownPumpBolusRecord`, though this should be thought through, to make sure
// all scenarios are covered // all scenarios are covered