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

Dev #7225

Merged
merged 65 commits into from
Dec 10, 2021
Merged

Dev #7225

merged 65 commits into from
Dec 10, 2021

Conversation

tlwitt
Copy link

@tlwitt tlwitt commented Dec 8, 2021

No description provided.

sulkaharo and others added 29 commits November 12, 2021 02:14
Forcing a newer version of share2nightscout-bridge in order to keep up
with Dexcom changes.
Include enhancements on the minimed-connect-to-nightscout plugin to include
fetching data from a Careportal account.
Using a ratio of 18 mg/ml to 1 mmol/l causes NightScout to report different values compared Dexcom app/receivers.

A more accurate value is based on 0.0555 mmol/l to 1 mg/dl. The inverse is 18.018018018 mg/dl to 1 mmol/l

This puts the value closer to the Dexcom app/receiver. Affected values:

|mg/dl|  18|18.018018018|
|----:|---:|-----------:|
|  109| 6.1|         6.0|
|  118| 6.6|         6.5|
|  127| 7.1|         7.0|
|  136| 7.6|         7.5|
|  145| 8.1|         8.0|
|  154| 8.6|         8.5|
|  163| 9.1|         9.0|
|  172| 9.6|         9.5|
|  181|10.1|        10.0|
|  190|10.6|        10.5|
|  199|11.1|        11.0|
|  208|11.6|        11.5|
|  217|12.1|        12.0|
|  226|12.6|        12.5|
|  235|13.1|        13.0|
|  244|13.6|        13.5|
|  253|14.1|        14.0|
|  262|14.6|        14.5|
|  271|15.1|        15.0|
|  280|15.6|        15.5|
|  289|16.1|        16.0|
|  298|16.6|        16.5|
|  307|17.1|        17.0|
|  309|17.2|        17.1|
|  316|17.6|        17.5|
|  318|17.7|        17.6|
|  325|18.1|        18.0|
|  327|18.2|        18.1|
|  334|18.6|        18.5|
|  336|18.7|        18.6|
|  343|19.1|        19.0|
|  345|19.2|        19.1|
|  352|19.6|        19.5|
|  354|19.7|        19.6|
|  361|20.1|        20.0|
|  363|20.2|        20.1|
|  370|20.6|        20.5|
|  372|20.7|        20.6|
|  379|21.1|        21.0|
|  381|21.2|        21.1|
|  388|21.6|        21.5|
|  390|21.7|        21.6|
|  397|22.1|        22.0|
|  399|22.2|        22.1|
Using share2nightscout-bridge, the device property is defined as 'share2'

displayBg should honour the Dexcom HIGH and LOW values when the device is share2

This ensures that NightScout returns "LOW" for 39mg/dl and "HIGH" for 401mg/dl as is reported in the Dexcom app/receiver.
This patches uses a newer versin of minimed-connect-to-nightscout
that is designed to work under the same version of node as
Nightscout.  It should result in Nightscout gaining a capability
to use Medtronic Careportal accounts for cloud to cloud
synchronization.
The `npm audit fix` in the `Dockerfile` is currently
deleting vulnerable packages. Although we do want to have package
security, this should be done in a development process, not as part
of the Docker build.

This commit also reduces the Docker image size and improves security.
This should force the coverage job to return with a success error code.
Codacy changed support for legacy projects.  This change should not cause
builds to fail and should not influence ability to merge pull requests.  Until
further work with coverage with codacy, this change emits a warning of "NO
COVERAGE" while allowing the build to pass, rather than failing the build.
@bewest bewest merged commit 4750f13 into nightscout:master Dec 10, 2021
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.

6 participants