Skip to content

Commit 1ae59e9

Browse files
authored
Merge pull request #468 from LoopKit/hkunit-rename
renaming milligramsPerDeciliterUnit() to milligramsPerDeciliter()
2 parents e0705f4 + d0f0fa0 commit 1ae59e9

16 files changed

+33
-33
lines changed

Common/Extensions/HKUnit.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,17 +21,17 @@ extension HKUnit {
2121
}
2222
}
2323

24-
static func milligramsPerDeciliterUnit() -> HKUnit {
24+
static func milligramsPerDeciliter() -> HKUnit {
2525
return HKUnit.gramUnit(with: .milli).unitDivided(by: HKUnit.literUnit(with: .deci))
2626
}
2727

28-
static func millimolesPerLiterUnit() -> HKUnit {
28+
static func millimolesPerLiter() -> HKUnit {
2929
return HKUnit.moleUnit(with: .milli, molarMass: HKUnitMolarMassBloodGlucose).unitDivided(by: HKUnit.liter())
3030
}
3131

3232
// A glucose-centric presentation helper for the localized unit string
3333
var glucoseUnitDisplayString: String {
34-
if self == HKUnit.millimolesPerLiterUnit() {
34+
if self == HKUnit.millimolesPerLiter() {
3535
return NSLocalizedString("mmol/L", comment: "The unit display string for millimoles of glucose per liter")
3636
} else {
3737
return String(describing: self)

DoseMathTests/DoseMathTests.swift

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ class RecommendTempBasalTests: XCTestCase {
6161
return fixture.map {
6262
return GlucoseFixtureValue(
6363
startDate: dateFormatter.date(from: $0["date"] as! String)!,
64-
quantity: HKQuantity(unit: HKUnit.milligramsPerDeciliterUnit(), doubleValue: $0["amount"] as! Double)
64+
quantity: HKQuantity(unit: HKUnit.milligramsPerDeciliter(), doubleValue: $0["amount"] as! Double)
6565
)
6666
}
6767
}
@@ -81,15 +81,15 @@ class RecommendTempBasalTests: XCTestCase {
8181
}
8282

8383
var glucoseTargetRange: GlucoseRangeSchedule {
84-
return GlucoseRangeSchedule(unit: HKUnit.milligramsPerDeciliterUnit(), dailyItems: [RepeatingScheduleValue(startTime: TimeInterval(0), value: DoubleRange(minValue: 90, maxValue: 120))], workoutRange: nil)!
84+
return GlucoseRangeSchedule(unit: HKUnit.milligramsPerDeciliter(), dailyItems: [RepeatingScheduleValue(startTime: TimeInterval(0), value: DoubleRange(minValue: 90, maxValue: 120))], workoutRange: nil)!
8585
}
8686

8787
var insulinSensitivitySchedule: InsulinSensitivitySchedule {
88-
return InsulinSensitivitySchedule(unit: HKUnit.milligramsPerDeciliterUnit(), dailyItems: [RepeatingScheduleValue(startTime: 0.0, value: 60.0)])!
88+
return InsulinSensitivitySchedule(unit: HKUnit.milligramsPerDeciliter(), dailyItems: [RepeatingScheduleValue(startTime: 0.0, value: 60.0)])!
8989
}
9090

9191
var minimumBGGuard: GlucoseThreshold {
92-
return GlucoseThreshold(unit: HKUnit.milligramsPerDeciliterUnit(), value: 55)
92+
return GlucoseThreshold(unit: HKUnit.milligramsPerDeciliter(), value: 55)
9393
}
9494

9595
var insulinActionDuration: TimeInterval {
@@ -385,7 +385,7 @@ class RecommendTempBasalTests: XCTestCase {
385385
XCTAssertEqual(TimeInterval(minutes: 30), dose!.duration)
386386

387387
// Use mmol sensitivity value
388-
let insulinSensitivitySchedule = InsulinSensitivitySchedule(unit: HKUnit.millimolesPerLiterUnit(), dailyItems: [RepeatingScheduleValue(startTime: 0.0, value: 3.33)])!
388+
let insulinSensitivitySchedule = InsulinSensitivitySchedule(unit: HKUnit.millimolesPerLiter(), dailyItems: [RepeatingScheduleValue(startTime: 0.0, value: 3.33)])!
389389

390390
dose = DoseMath.recommendTempBasalFromPredictedGlucose(glucose,
391391
atDate: glucose.first!.startDate,
@@ -465,7 +465,7 @@ class RecommendBolusTests: XCTestCase {
465465
return fixture.map {
466466
return GlucoseFixtureValue(
467467
startDate: dateFormatter.date(from: $0["date"] as! String)!,
468-
quantity: HKQuantity(unit: HKUnit.milligramsPerDeciliterUnit(), doubleValue: $0["amount"] as! Double)
468+
quantity: HKQuantity(unit: HKUnit.milligramsPerDeciliter(), doubleValue: $0["amount"] as! Double)
469469
)
470470
}
471471
}
@@ -485,15 +485,15 @@ class RecommendBolusTests: XCTestCase {
485485
}
486486

487487
var glucoseTargetRange: GlucoseRangeSchedule {
488-
return GlucoseRangeSchedule(unit: HKUnit.milligramsPerDeciliterUnit(), dailyItems: [RepeatingScheduleValue(startTime: TimeInterval(0), value: DoubleRange(minValue: 90, maxValue: 120))], workoutRange: nil)!
488+
return GlucoseRangeSchedule(unit: HKUnit.milligramsPerDeciliter(), dailyItems: [RepeatingScheduleValue(startTime: TimeInterval(0), value: DoubleRange(minValue: 90, maxValue: 120))], workoutRange: nil)!
489489
}
490490

491491
var insulinSensitivitySchedule: InsulinSensitivitySchedule {
492-
return InsulinSensitivitySchedule(unit: HKUnit.milligramsPerDeciliterUnit(), dailyItems: [RepeatingScheduleValue(startTime: 0.0, value: 60.0)])!
492+
return InsulinSensitivitySchedule(unit: HKUnit.milligramsPerDeciliter(), dailyItems: [RepeatingScheduleValue(startTime: 0.0, value: 60.0)])!
493493
}
494494

495495
var minimumBGGuard: GlucoseThreshold {
496-
return GlucoseThreshold(unit: HKUnit.milligramsPerDeciliterUnit(), value: 55)
496+
return GlucoseThreshold(unit: HKUnit.milligramsPerDeciliter(), value: 55)
497497
}
498498

499499
var insulinActionDuration: TimeInterval {
@@ -589,13 +589,11 @@ class RecommendBolusTests: XCTestCase {
589589
XCTAssertEqual(1.325, dose.amount)
590590

591591
if case BolusRecommendationNotice.currentGlucoseBelowTarget(let glucose, let units) = dose.notice! {
592-
XCTAssertEqual(units, HKUnit.milligramsPerDeciliterUnit())
592+
XCTAssertEqual(units, HKUnit.milligramsPerDeciliter())
593593
XCTAssertEqual(glucose.quantity.doubleValue(for: units), 60)
594594
} else {
595595
XCTFail("Expected currentGlucoseBelowTarget, but got \(dose.notice!)")
596596
}
597-
598-
//XCTAssertEqual(BolusRecommendationNotice.currentGlucoseBelowTarget(glucose: GlucoseValue(56), unit: HKUnit.milligramsPerDeciliterUnit()), dose.notice!)
599597
}
600598

601599
func testDroppingBelowRangeThenRising() {
@@ -747,7 +745,7 @@ class RecommendBolusTests: XCTestCase {
747745
XCTAssertEqual(1.0, dose.amount)
748746

749747
// Use mmol sensitivity value
750-
let insulinSensitivitySchedule = InsulinSensitivitySchedule(unit: HKUnit.millimolesPerLiterUnit(), dailyItems: [RepeatingScheduleValue(startTime: 0.0, value: 10.0 / 3)])!
748+
let insulinSensitivitySchedule = InsulinSensitivitySchedule(unit: HKUnit.millimolesPerLiter(), dailyItems: [RepeatingScheduleValue(startTime: 0.0, value: 10.0 / 3)])!
751749

752750
dose = DoseMath.recommendBolusFromPredictedGlucose(glucose,
753751
atDate: glucose.first!.startDate,

Loop Status Extension/StatusViewController.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,8 @@ class StatusViewController: UIViewController, NCWidgetProviding {
5050
)
5151

5252
charts.glucoseDisplayRange = (
53-
min: HKQuantity(unit: HKUnit.milligramsPerDeciliterUnit(), doubleValue: 100),
54-
max: HKQuantity(unit: HKUnit.milligramsPerDeciliterUnit(), doubleValue: 175)
53+
min: HKQuantity(unit: HKUnit.milligramsPerDeciliter(), doubleValue: 100),
54+
max: HKQuantity(unit: HKUnit.milligramsPerDeciliter(), doubleValue: 175)
5555
)
5656

5757
return charts

Loop/Managers/CGM/EnliteCGMManager.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ final class EnliteCGMManager: CGMManager {
3939
self.sensorState = EnliteSensorDisplayable(latestSensorEvent)
4040
}
4141

42-
let unit = HKUnit.milligramsPerDeciliterUnit()
42+
let unit = HKUnit.milligramsPerDeciliter()
4343
let glucoseValues = events
4444
// TODO: Is the { $0.date > latestGlucoseDate } filter duplicative?
4545
.filter({ $0.glucoseEvent is SensorValueGlucoseEvent && $0.date > latestGlucoseDate })

Loop/Managers/DeviceDataManager.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ final class DeviceDataManager {
178178
// Enlite data is included
179179
if let date = glucoseDateComponents?.date {
180180
loopManager.addGlucose([(
181-
quantity: HKQuantity(unit: HKUnit.milligramsPerDeciliterUnit(), doubleValue: Double(glucose)),
181+
quantity: HKQuantity(unit: HKUnit.milligramsPerDeciliter(), doubleValue: Double(glucose)),
182182
date: date,
183183
isDisplayOnly: false
184184
)], from: nil)

Loop/Managers/DoseMath.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ struct DoseMath {
3131
- returns: The determined basal rate, in Units/hour
3232
*/
3333
private static func calculateTempBasalRateForGlucose(_ currentGlucose: HKQuantity, toTargetGlucose targetGlucose: HKQuantity, insulinSensitivity: HKQuantity, currentBasalRate: Double, maxBasalRate: Double, duration: TimeInterval) -> Double {
34-
let unit = HKUnit.milligramsPerDeciliterUnit()
34+
let unit = HKUnit.milligramsPerDeciliter()
3535
let doseUnits = (currentGlucose.doubleValue(for: unit) - targetGlucose.doubleValue(for: unit)) / insulinSensitivity.doubleValue(for: unit)
3636

3737
let rate = min(maxBasalRate, max(0, doseUnits / (duration / TimeInterval(hours: 1)) + currentBasalRate))

Loop/Managers/LoopDataManager.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -715,7 +715,7 @@ final class LoopDataManager {
715715
self.retrospectivePredictedGlucose = retrospectivePrediction
716716

717717
guard let lastGlucose = retrospectivePrediction.last else { return }
718-
let glucoseUnit = HKUnit.milligramsPerDeciliterUnit()
718+
let glucoseUnit = HKUnit.milligramsPerDeciliter()
719719
let velocityUnit = glucoseUnit.unitDivided(by: HKUnit.second())
720720

721721
let discrepancy = change.end.quantity.doubleValue(for: glucoseUnit) - lastGlucose.quantity.doubleValue(for: glucoseUnit) // mg/dL

Loop/Managers/StatusExtensionDataManager.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ final class StatusExtensionDataManager {
6262
)
6363
context.predictedGlucose = PredictedGlucoseContext(
6464
values: (1...36).map { 89.123 + Double($0 * 5) }, // 3 hours of linear data
65-
unit: HKUnit.milligramsPerDeciliterUnit(),
65+
unit: HKUnit.milligramsPerDeciliter(),
6666
startDate: Date(),
6767
interval: TimeInterval(minutes: 5))
6868

Loop/Managers/WatchDataManager.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ final class WatchDataManager: NSObject, WCSessionDelegate {
6262
private var lastComplicationContext: WatchContext?
6363

6464
private let minTrendDrift: Double = 20
65-
private lazy var minTrendUnit = HKUnit.milligramsPerDeciliterUnit()
65+
private lazy var minTrendUnit = HKUnit.milligramsPerDeciliter()
6666

6767
private func sendWatchContext(_ context: WatchContext) {
6868
if let session = watchSession, session.isPaired && session.isWatchAppInstalled {

Loop/Models/BolusRecommendation.swift

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import Foundation
1010
import LoopKit
1111
import HealthKit
1212

13+
1314
enum BolusRecommendationNotice: CustomStringConvertible, Equatable {
1415
case glucoseBelowMinimumGuard(minGlucose: GlucoseValue, unit: HKUnit)
1516
case currentGlucoseBelowTarget(glucose: GlucoseValue, unit: HKUnit)
@@ -74,6 +75,7 @@ struct BolusRecommendation {
7475
}
7576
}
7677

78+
7779
extension BolusRecommendation: Comparable {
7880
static func ==(lhs: BolusRecommendation, rhs: BolusRecommendation) -> Bool {
7981
return lhs.amount == rhs.amount

0 commit comments

Comments
 (0)