Skip to content

Dev watch #4

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

Merged
merged 10 commits into from
Aug 23, 2018
Merged

Dev watch #4

merged 10 commits into from
Aug 23, 2018

Conversation

iMaces
Copy link
Owner

@iMaces iMaces commented Aug 23, 2018

No description provided.

ps2 and others added 10 commits August 3, 2018 08:44
* Improve error message display

* update revision of rlkit used
* Localize loop error messages

* Refactor error strings into enums to separate out presentation concerns and avoid duplication
LoopKit#778)

* From DM61's work on IRC, fixes a case where RC velocity can have an artificially large magnitude from short glucose change intervals.

* Typo
* Language commits

* adding folders

* touchup on translations

* norwegian touchup

* Localize loop error messages

* touchups

* spanish

* Loop Errors for french, chinese, russian

* Language commits

* adding folders

* touchup on translations

* norwegian touchup

* french

* updating translations after rebase

* fix errant "head"

* italian and chinese

* added strings for loop errors...all but Norwegian done

* touchups

* norwegian

* DONE!

* ALLL DONE!  norwegian touchups

* Polish addition

* more norwegian cleanup

* one polish word
Added success haptic to WatchApp carb, bolus sends
iMaces pushed a commit that referenced this pull request Aug 23, 2018
Fix override range display, mmol/L limits
@iMaces iMaces merged commit e7f2ccd into dev-watchM Aug 23, 2018
@iMaces iMaces deleted the dev-watch branch August 23, 2018 22:26
iMaces pushed a commit that referenced this pull request Mar 29, 2019
* Move mock health data deletion to Loop

* Fix mock data deletion section logic
iMaces pushed a commit that referenced this pull request Jun 30, 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants