Update CGM.md #969
Merged
Update CGM.md #969
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
nits
scottleibrand
requested changes
Jul 23, 2017
| * Medtronic (MiniMed Paradigm REAL-Time Revel or Enlite) | ||
|
|
||
|
|
||
| The openaps tool set currently primarily supports three different CGM systems: the Dexcom G4 Platinum system (with or without the [Share](http://www.dexcom.com/dexcom-g4-platinum-share) functionality), the newer Dexcom G5 Mobile system and the [Medtronic system](https://www.medtronicdiabetes.com/treatment-and-products/enlite-sensor). Other CGM or CGM-like devices (Libre) can also be used if the data is uploaded to Nightscout and the OpenAPS rig has Internet connectivity. |
There was a problem hiding this comment.
This is talking about the underling openaps toolkit, so should not be capitalized.
|
Ah okay
I need to understand that. I'll catch up with u on Gitter soon
11 mile hike today
Rig did great
I need to read docs about offline display. Obviously Nightscout wasn't available
Need to get something to work on hikes
Dirk
On Jul 23, 2017, at 10:16 AM, Scott Leibrand <[email protected]<mailto:[email protected]>> wrote:
@scottleibrand requested changes on this pull request.
________________________________
In docs/docs/Gear Up/CGM.md<#969 (comment)>:
@@ -5,7 +5,7 @@
* Medtronic (MiniMed Paradigm REAL-Time Revel or Enlite)
…-The openaps tool set currently primarily supports three different CGM systems: the Dexcom G4 Platinum system (with or without the [Share](http://www.dexcom.com/dexcom-g4-platinum-share) functionality), the newer Dexcom G5 Mobile system and the [Medtronic system](https://www.medtronicdiabetes.com/treatment-and-products/enlite-sensor). Other CGM or CGM-like devices (Libre) can also be used if the data is uploaded to Nightscout and the OpenAPS rig has Internet connectivity.
This is talking about the underling openaps toolkit, so should not be capitalized.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#969 (review)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AbwwkZsINwibqtQ-GSdb24Bhc4jquE7oks5sQ3_RgaJpZM4OgXDI>.
|
scottleibrand
approved these changes
Jul 31, 2017
danamlewis
pushed a commit
that referenced
this pull request
Jul 31, 2017
* Update loops-in-progress.md * Update loops-in-progress.md Adding my name to the list of people who have loops in progress. * Update ifttt-integration.md Searching for "maker" no longer yields the correct action service. Must now search for "webhooks," icon is now blue instead of green. With apologies, didn't know how to update screen shots. * Fix some typos * Update loops-in-progress.md * Update useful-mobile-apps.md "uname -a" no longer reports the version of jubilinux you're running but rather the version of the Linux kernel you're running. * Note that kernel version is useful for determining jubilinux version * Update how-openaps-works-overview.md nits * Update index.rst (#963) Conforming in same paragraph "internet" to "Internet" (capital "I"). * Update overview-of-build-process.md (#965) missing word added "usually" to code snippet because they're not always code snippets, sometimes buttons, and other screen references * Update hardware.md (#967) Conform to manufacturers' name styles/conventions (OmniPod, t:slim) Remove ' because not possessive but rather plural Lower rather than lesser * Update nightscout-setup.md Provided information for users who use mmol instead of mgdl on how to set BG targets in Heroku so don't end up with "urgent high" alerts when they are in range. * Update loops-in-progress.md (#970) * Update understanding-your-Explorer-Board-rig.md (#972) Just a few edits to clarify this step. It took me a while to realize I needed two sets of screws! * Adding my name to Loops in Progress * Add meetup group URLs * adding emphasis to max-iob and maxSafeBasal (#977) * adding emphasis to max-iob and maxSafeBasal * screenshot of maxSafeBasal * typo fix * word smithing * Add a linkable header * combine people from both copies of loops-in-progress.md so we can delete one * fix broken images for token based authentications after doc update (#981) * fix broken images after doc update * add line to explain API_SECRET is also in crontab * Update collect-data-and-prepare.md (#978) Nits Typos Name styles Conform periods in lists * Update understanding-your-Explorer-Board-rig.md (#982) Conform to "LiPo" Conform to "Explorer" "and hold" nits simplified language, deleted excess words embedded link for video replaced ellipsis with comma * Update loops-in-progress.md (#984) Added my name. Trying to build the loop for my son. * Update loops-in-progress.md (#985) * Specify version 0.1.1 (or earlier) of Jubilinux (#986) * Specify version 0.1.1 (or earlier) of Jubilinux Current oref0 master doesn't work with 0.2.0. * Remove "or earlier" There are some fixes in 0.1.1 that are useful for easier setup and increased reliability of OpenAPS rigs * Adding my name to the list (#987) Adding myself to the list . * image link fixes (#983) * Delete old loopers file that's now in a different place * Update usability-considerations.md (#959) * Update edison.md (#974) "Jubilinux" is styled with initial lower case "jubilinux" Remove double-preposition (on and to) Replace "ones" with "Edisons" Replace "more" with "additional" Removed unnecessary / Spelled out LiPo Replace and/or with or Change LIPO to LiPo Inserted link for battery URL Added missing .) at end of Amazon link. Inserted link for other battery URL Corrected type easilly internet to Internet (style) Style preference on extended sentence with a hyphen followed by "however" Replace exacto with X-ACTO Initial cap "Explorer" Added . to end sentence Replace "screw" with "attach" Typo harware Added . to end sentence Conform mah to mAh with a space after the # Added (as well as some 2500 mAh options) * Update edison.md (#954) * Update edison.md The original text said to use M2 as spacers, which means you have to screws these spacers between the boards onto the M2 screw, which is kind of difficult and impossible to tighten. If you use slightly larger nuts as spacers, you can just push the screws through the board then through the spacers then the other board and then put a nut on. This is way more secure and you won't have to tighten them every day! * Clarify * Update CGM.md (#969) * Update CGM.md nits * Lowercase openaps toolkit * Update offline-looping-and-monitoring.md (#973) Did the best I could to say that offline looping is possible with iPhone via modified Loop app, but don't really know enough to explain it well. Hope this is helpful. * Update nightscout-setup.md (#980) * Update nightscout-setup.md nits formatted links typos clarification These graphics are not showing on this page: AddRole, AddSubject Dana/Scott/others: please see notes in brackets above. * Remove NOTEs and clarify using token auth roles * Fix image links * Redirected urls (#990) * add quotes * update links to reflect https://readthedocs.org/dashboard/openaps/redirects/ * Update update-your-rig.md * Update edison.md
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.
nits