Fixing pumpState observation #7
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kenstack
pushed a commit
to kenstack/Loop
that referenced
this pull request
Feb 22, 2018
Ns loop pill
jasoncalabrese
pushed a commit
to jasoncalabrese/Loop
that referenced
this pull request
May 9, 2019
integral retrospective correction
ddaniels1
pushed a commit
to ddaniels1/Loop
that referenced
this pull request
May 25, 2020
Dev merge with ab otp
Closed
jeremybarnum
pushed a commit
to jeremybarnum/Loop
that referenced
this pull request
May 12, 2022
Update Localizable.strings
ps2
pushed a commit
that referenced
this pull request
Feb 20, 2023
* Add unannounced meal func to CarbStoreProtocol * V1 of UAM algo * Add debug logs * Handle unannounced meal notification & open up controller * Add mealstart to notification userinfo * Improve notification description * Add direct entry to carb flow from notification on watch * Add auto-setting of carb entry on watch * carbThreshold -> mealCarbThreshold * Retract UAM notifications after the carbs have expired * Improve function naming * Move notification logic from LoopKit to Loop * Make current date configurable * Make current date configurable during unit testing * Update status enum naming for point-of-use clarity * Make 'now' time configurable * Extract loop data manager testing logic into base class * Extract loop data manager dosing tests into their own file * Add unannounced meal tests * Fix concurrency issue * Pull UAM constants into separate struct * Fix notifications not being retracted after their expiration date * Expire using start of meal instead of current time * Add unannounced meal notifications permission * Add AlertPermissionsViewModel * Remove unneeded `AnyView`s * Removed unused completion block * Use the counteraction effects passed into the function * Schedule missed meal notifications to avoid notification during an microbolus (#3) * Delay missed meal notification if possible to avoid delivering it during an autobolus * Add tests for notification delay * Update `estimatedDuration` function headers in response to PR feedback * Add UAM banner to carb entry screen (#5) * Add warning banner to remind the user to edit the meal estimate * Update warning text * UAM algo updates: only use directly observed carb absorption (#6) * Add ability to calculate the number of carbs in a missed meal (#4) * Plumb a customizable carb amount through UAM notification architecture * Fix merge conflict * Update tests for changes from the merge * Make UAM notifications unit testable * Add carb autofill clamping based on the user's carb threshold & max bolus * Update target order to group extensions together * Improve issue report description * Create `MealDetectionManager` from old UAM functions in `CarbStore` * Move notification logic into `MealDetectionManager` * Updates based on feedback for UAM PR (#7) * Create `MealDetectionManager` from old UAM functions in `CarbStore` * Move notification logic into `MealDetectionManager` * Move UAM test fixtures from LoopKit to Loop * Remove old TODO * Revert change to Loop signing team * Fix merge issues * Make tests runnable * Lower meal carb threshold to 15 g to reduce false-negatives * Fix carb entry controller merge issues * Unannounced meal / UAM -> missed meal * UAM test fixture -> missed meal test fixture * Variable naming improvements * Remove `AlertPermissionsViewModel` * Move call to check for missed meals to `loop()`-level * Remove `CarbStore` dependency from `MealDetectionManager` These changes also remove the requirement that there be no carbs entered after the missed meal was detected - since we're now observing direct absorption, I didn't think this requirement made sense anymore * Reduce `minRecency` required to detect a meal * Update counteraction effect math in `MealDetectionManager` to skew towards an earlier meal time instead of a later one * Fix conflict + move missed meal toggle 1 level higher
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.