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

High temp recommended after carb entry and bolus #120

Closed
dm61 opened this issue Aug 16, 2016 · 1 comment
Closed

High temp recommended after carb entry and bolus #120

dm61 opened this issue Aug 16, 2016 · 1 comment
Milestone

Comments

@dm61
Copy link
Contributor

dm61 commented Aug 16, 2016

When a user enters carbs and delivers a bolus, a high temp immediately shows up as recommended. I believe this is because the meal is immediately taken into account by the Loop calculations, while the bolus is not, because the Loop relies solely on actual pump data to calculate IOB (as it should). This would not normally be an issue, as the Loop would become aware of the bolus delivered on the next pump readout. However, the high temp recommendation could become problematic if the user accepted the recommended high temp and if the Loop, for whatever reason, failed to communicate with the pump just after that. The user could be left with the bolus and the high temp for 30 min (effectively a double bolus) - a potentially hazardous situation.

@loudnate
Copy link
Collaborator

As @jlucasvt mentioned, this is an issue of displaying a "recommended temp" based on carb delivery but before the bolus has been confirmed as delivered. No temporary basal is enacted at that stage.

I do think it's important to fix this by not displaying the recommended temp to avoid confusion. Thank you for filing.

@loudnate loudnate added this to the 1.0 milestone Sep 17, 2016
ps2 pushed a commit that referenced this issue Aug 4, 2020
…120)

* LOOP-1395: For Tidepool Loop, exclude HealthKit data from other apps

Adds a FeatureFlag `excludeHealthKitDataFromOtherApps` that also pushes down into LoopKit

* checkpoint PR feedback

* Get rid of unneeded VStack

* PR Feedback: use observeHealthKitForCurrentAppOnly and use consistently

* Fix DIY build
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

No branches or pull requests

2 participants