Skip to content

LOOP-4665 Functional Algorithm#594

Merged
ps2 merged 27 commits into
devfrom
ps2/LOOP-4665/functional-algorithm
Sep 7, 2023
Merged

LOOP-4665 Functional Algorithm#594
ps2 merged 27 commits into
devfrom
ps2/LOOP-4665/functional-algorithm

Conversation

@ps2
Copy link
Copy Markdown

@ps2 ps2 commented Sep 6, 2023

https://tidepool.atlassian.net/browse/LOOP-4665

Contains tests that assert that the new functional algorithm added in tidepool-org/LoopKit#542
(see testLiveCaptureWithFunctionalAlgorithm) produces the same output as LoopDataManager (see testForecastFromLiveCaptureInputData())

Other changes included:

hannkm and others added 27 commits April 12, 2023 21:22
Fix predicted BG graph only showing sometimes
…pectiveCorrectionSelectionView

✏️  Fix a typo in IntegralRetrospectiveCorrectionSelectionView
Updated translations from Lokalise on Sat Aug 19 14:25:48 CDT 2023
Fixed numeral localization in insulin delivery table view
…nore-calibration

Disable missed meal detection if there are user-entered or calibration BGs
…reamline app startup when running app just for tests
@ps2 ps2 requested review from ArwainK, Camji55 and nhamming September 6, 2023 22:06
Copy link
Copy Markdown

@nhamming nhamming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

let descriptionWithFont = String(format:"<style>body{font-family: '-apple-system', '\(font.fontName)'; font-size: \(font.pointSize);}</style>%@", description)

guard let attributedDescription = try? NSMutableAttributedString(data: Data(descriptionWithFont.utf8), options: [.documentType: NSAttributedString.DocumentType.html], documentAttributes: nil) else {
guard let attributedDescription = try? NSMutableAttributedString(data: descriptionWithFont.data(using: .utf16)!, options: [.documentType: NSAttributedString.DocumentType.html], documentAttributes: nil) else {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is force unwrap safe here?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe we are creating all the strings that go in there; it's not user data, so I think yes.

@ps2 ps2 merged commit 1c88b00 into dev Sep 7, 2023
@ps2 ps2 deleted the ps2/LOOP-4665/functional-algorithm branch September 7, 2023 20:00
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.

7 participants