Dev watch hybrid2 #1
Open
Conversation
This file contains hidden or 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
Use CGMManagerUI protocol for CGM setup, settings, and ingestion
Merge master into dev
* Shorcuts: Add Carbs * Update swiftcharts
* Refactor RC as a timeline To provide support for a safe, accurate IRC implementation, this patch provides retrospective correction discrepancies as a timeline, which can be extended for use in integration. * Bump LoopKit dev revision
* Put a glucose chart on the watch. This incorporates a ton of fine work by Eric Jensen, but I've squashed it down into a single commit for ease of management. The watch now has two discrete pages, one which is the standard actions based interface, the second is an information interface which shows the IOB, COB, basal rate and a glucose chart. Glucose data is read from HealthKit on the watch. Sometimes this lags behind so we also make requests to backfill glucose data as necessary. We're also minimizing the amount of data sent over BLE as much as possible since BLE is slow and less reliable. Fix filename mismatches in comments Prototype of a SpriteKit-based glucose chart Tweak the scene size for 42mm watches Add the rest of the chart components Set high watermark higher for mg/dl Clean up pause/unpause semantics and set the max hours to 4 The crown now adjusts the height of the graph Fix override range rendering to match the Loop app ui Small optimizations Clean up the rendering code to pause between data changes Remove debug code Highlight the upper BG when it changes Make updateGlucoseChart async to avoid hitting CPU limits More stability fixes & remove graph corner radius Force updateNode() to run on the main queue to avoid contention Improve watch size detection code Switch to the more efficient SpriteNode Only cache 4 hours of glucose history Remove pause/unpause code- it doesn't save significant CPU Try unpausing the WKInterfaceSKScene to get the animation to stop freezing Revert "Remove pause/unpause code- it doesn't save significant CPU" This reverts commit deeddb2. Move the IOB/COB below the graph Whitespace and comment cleanup Animate moving points on the chart Only animate the predicted path when adjusting the y-axis Clean up sprite animation code Adjust scene size on 38mm watch Only show overrides that haven't yet expired Fix node expiration code Break the more complex updateSprite into two more manageable pieces: getSprite() and SKSpriteKitNode.move() Clean up variable names and comments around how we expire inactive nodes for readability Add some comments to aid readability Change visible hours with force touch menu Revert changes in tools versions in storyboard Add icons for duration menu Rearrange menu actions Fix asset locations for graph menu icons Make mmol/L graph limits be integers Force minimum height for dated range rects Make target ranges more visible Only show future part of override ranges Revert "Only show future part of override ranges" This reverts commit 0b25d2e. Expire overrides once they move into past Set min range height via variable Rearrange image assets for graph menu Clarify minimum height on dated range Don't send expired overrides in watch context Rearrange testing of override end date Use Int16 instead of UInt16 to handle negative numbers in predicted glucose Change the appearance of the chart to be platter style to better fit with Watch aesthetics Remove duplicate files * chart tweaks adds auto-scaling, handles the SpriteKit lifecycle, and simplifies the sync story for target ranges
* Watch Feature Updates - Update IOB and COB immediately after carb/bolus send - Adjust complication push budget to spend evenly before midnight - Adjustments for Series 4, including rounded action buttons, correct text margin insets, and using system scroll haptics - Improve efficiency of the chart redraw * Disable crown haptics * Redraw immediately
glucose text roughly equivalent to the spacing between the cap line of the glucose text and the baseline of the title.
* Update dependencies * DEVELOPMENT_TEAM should be none
* Move data to a 3rd page * Clean up sizes and strings. Size is only verified on 40mm * Second pass at getting chart size correct * Use platter visual effect for data page * Fix chart sizes for all watch variants * Fix issues in the code that saves the start page * Remember and restore the graph visible duration * Switch back to 2 pages and put the data under the graph * First attempt at a hybrid mode. It's a little cluttered * Show Active Insulin at the bottom of the graph, get rid of custom scrolling code * Fix dangling outlet; adjust the size of graph on 42mm * Remove IOB & COB text from labels * Use screen bound ranges to identify watch models; Add more top margin to the graph * minor cleanups * Make the spacing from the top of the graph to the baseline of the glucose text roughly equivalent to the spacing between the cap line of the glucose text and the baseline of the title. * Layout and alignment tweaks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.