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 #176

Closed
wants to merge 4 commits into from
Closed

Dev #176

wants to merge 4 commits into from

Conversation

logichammer
Copy link
Contributor

Dana / Scott --

I have the first pass at AMA documentation and a few questions for you in the doc in bold.

Hope this looks decent. Given the number of reports and aliases, this was the best way I could think to do it and based on the MA documentation it looked like it was OK to do so.

Thanks as always!
Matt

@scottleibrand
Copy link
Contributor

This is good stuff. I'd like to get it merged soonish, but don't really want to overwrite the existing autosens and meal-assist documentation for people who don't want to go all the way to AMA just yet. Perhaps we should make this a new page, called "Configuring Advanced Meal Assist" or similar, to make clear that it is a separate and optional thing for now?

I'd also like to get away from telling people to paste in .ini contents. (That's better than nothing, but after merging would want to update that part to use openaps commands to add each section, if you don't beat me to it.)

@danamlewis
Copy link
Contributor

Looking good @logichammer! Ditto the opinion of @scottleibrand ..I'm thinking 3 separate pages: Configuring Meal Assist; Configuring Advanced Meal Assist; and Configuring Automatic Sensitivity Mode (all under phase 6 section).

A few minor edits in here that I'd add, but I'll do that after we get this to the right page structure and merged :) Thanks!


2) Next in order to properly execute the new auto-sensitivity module, you need to have at least 24 hour worth of pump history data and enough bg readings (past 24 hours).
In your openaps.ini apply the following changes:
`nano openaps.ini` cut and paste:
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we need to have anyone cut and paste here. There are only a couple of commands needed to add the necessary reports. http://openaps.readthedocs.io/en/dev/docs/walkthrough/phase-6/Configure-Automatic-Sensitivity-Mode.html already has most of them: looks like the missing ones are:
openaps device add meal process --require "pumphistory profile clock glucose basal carbs" oref0 meal and
openaps report add monitor/meal.json text meal shell monitor/pumphistory-zoned.json settings/profile.json monitor/clock-zoned.json monitor/glucose.json settings/basal_profile.json monitor/carbhistory.json

Update Configure-Automatic-Sensitivity-Mode-and-Advanced-Meal-Assist.md
@scottleibrand
Copy link
Contributor

Closing this in favor of wip/ama-docs

@logichammer
Copy link
Contributor Author

Very glad to see it over there!!!!

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.

None yet

4 participants