Skip to content
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

Display of mmol/L units #159

Closed
beached opened this issue Sep 5, 2016 · 8 comments
Closed

Display of mmol/L units #159

beached opened this issue Sep 5, 2016 · 8 comments
Milestone

Comments

@beached
Copy link
Contributor

beached commented Sep 5, 2016

It looks like Loop is trying to display too many digits for mmol/L. Only 3 are ever needed at most. Actually , when the pump reports a bg <=99mg/dl only 2 are needed( so <=5.5mmol/L) but for consistency 3 is usually fine.

As one can see, this causes formatting bugs
bg display

Seeing as the data from the pump expresses the info in mg/dl and generally has 3 significant digits this could be accomplished with a formatting flag of %.1f

@beached
Copy link
Contributor Author

beached commented Sep 5, 2016

I am testing the use of

numberFormatter.usesSignificantDigits = true
numberFormatter.minimumSignificantDigits = 2
numberFormatter.maximumSignificantDigits = 3

To format the glucose hud display.

@loudnate
Copy link
Collaborator

loudnate commented Sep 5, 2016

Thanks – the NSNumberFormatter significant digits setting is exactly how I planned to fix these cases.

The screenshots are really helpful. Please add any more where things aren't correct to this ticket!

@beached
Copy link
Contributor Author

beached commented Sep 5, 2016

I was thinking that maybe, excuse my swift, but a member of the HKUnit that formats blood glocose or a subtype of sorts that does it. then any string for a glucose uses that.

@loudnate
Copy link
Collaborator

loudnate commented Sep 5, 2016

@beached
Copy link
Contributor Author

beached commented Sep 6, 2016

So that fixed the issue in issue in the glucosehud. Not the graphs or other areas.
fixed

@beached
Copy link
Contributor Author

beached commented Sep 16, 2016

Looks like the retrospective correction screen is hardwired to mg/dl
image

@beached
Copy link
Contributor Author

beached commented Sep 16, 2016

Actually, I am wrong, it does eventually but the scale is off
image

@beached beached closed this as completed Sep 16, 2016
@beached
Copy link
Contributor Author

beached commented Sep 16, 2016

The units updated after checking/unchecking

@beached beached reopened this Sep 16, 2016
@loudnate loudnate added this to the 1.0 milestone Sep 17, 2016
loudnate added a commit that referenced this issue Sep 29, 2016
 - New labels that highlight eventual glucose, current IOB, total delivery, and COB
 - Fixes #159
 - Dynamic selection of graph history length based on available display with. Rotate landscape to see more data.
 - New treatment for the recommended temp cell
loudnate added a commit that referenced this issue Sep 29, 2016
* Graph improvements

 - New labels that highlight eventual glucose, current IOB, total delivery, and COB
 - Fixes #159
 - Dynamic selection of graph history length based on available display with. Rotate landscape to see more data.
 - New treatment for the recommended temp cell

* Displaying boluses
loudnate added a commit that referenced this issue Sep 30, 2016
Fixes #130

Better use of space on iPhone SE and iPhone 7 Plus
loudnate added a commit that referenced this issue Sep 30, 2016
Fixes #130

Better use of space on iPhone SE and iPhone 7 Plus
ps2 pushed a commit that referenced this issue Aug 4, 2020
* Add basal rate schedule

* Move some Guardrail stuff into LoopKit

* Support for basal rates and bolus volumes

* PR Feedback
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

No branches or pull requests

2 participants