search

openaps / AndroidAPSdocs

Documentation for AndroidAPS
GNU Affero General Public License v3.0
131 stars 321 forks source link

check documentation and if necessary update for Omnipod Dash to get a good base for the rework-project #1617

Open peterleimbach opened 1 year ago

peterleimbach commented 1 year ago

this page please - https://androidaps.readthedocs.io/en/latest/Configuration/OmnipodDASH.html

Please PR on the branch rework-project.

FlyingToto commented 1 year ago

REMOVE " 30 minutes BASAL rate not supported " unless I am missing something, you can't set basal rate starting at 11:30am in AAPS (at least with dash I can't) so can we replace this whole paragraph:

30 min Basal Rate Profiles are NOT supported in AAPS. The AAPS Profile does not support a 30 minute basal rate time frame If you are new to AAPS and are setting up your basal rate profile for the first time, please be aware that basal rates starting on a half-hour basis are not supported, and you will need to adjust your basal rate profile to start on the hour. For example, if you have a basal rate of 1.1 units which starts at 09:30 and has a duration of 2 hours ending at 11:30, this will not work. You will need to change this 1.1 unit basal rate to a time range of either 9:00-11:00 or 10:00-12:00. Even though the Omnipod Dash hardware itself supports the 30 min basal rate profile increments, AAPS is not able to take them into account with its algorithms currently.

with:

AAPS basal rates Profiles only support full hour increments. IE: you can setup a basal of 1.1 between 11:00 and 15:00 but not between 11:30 and 15:30... image

peterleimbach commented 1 year ago

@vanelsberg Theodor can you please help here?

FlyingToto commented 1 year ago

"Activate Pod" section 2:

"The Fill Pod screen is displayed. Fill a new pod with at least 80 units of insulin and listen for two beeps indicating that the pod is ready to be primed. When calculating the total amount of insulin you need for 3 days, please take into account that priming the pod will use about 3-10 units."

  1. Does the 80 units pertain to the minimum pressure to trigger to pod to activate regardless of the insulin concentration? (10u/ml vs100u/ml vs 200u/ml) ? if yes would it make sense to include "regardless of the insulin concentration" ?
  2. In the same way should we replace "please take into account that priming the pod will use about 3-10 units." with "please take into account that priming the pod will use about 1ml"?
  3. about the retry... replace: "NOTE: Just in case you get the below error message (this can happen), do not panic. Click on the Retry button. In most situations activation will continue successfully." with: "NOTE: Just in case you get the below error message (this can happen), do not panic. Click on the Retry button. In most situations activation will continue successfully; you might want to relocate and move the pod/phone to another room or moving yourself a few feet away from the phone/pods which should be a few inches apart at most ."
FlyingToto commented 1 year ago

Troubleshooting TBR vs suspend

according to the doc:
"There is no suspend button anymore. If you want to “suspend” the pod, you can set a zero TBR for x minutes." except that I seem to see the opposite, it that because I have SMB enabled? I do see the suspend button in aaps.. . but I don't really see a way to setup a Temporary Basal Rate (I can do a temporary target or switch profile). I assume that's because I have SMB enabled so I am always having SMB adjusting into TBR. I do see the button to cancel the TBR. If that's the case we might want to rephrase the paragraph a bit... please confirm so I can submit a PR. Screenshot_20230623_145343_AAPS.jpg

FlyingToto commented 1 year ago

note: I can take care of updating the text once I get the feedback to my comments above.

vanelsberg commented 1 year ago

REMOVE " 30 minutes BASAL rate not supported " unless I am missing something, you can't set basal rate starting at 11:30am in AAPS (at least with dash I can't) so can we replace this whole paragraph:

30 min Basal Rate Profiles are NOT supported in AAPS. The AAPS Profile does not support a 30 minute basal rate time frame If you are new to AAPS and are setting up your basal rate profile for the first time, please be aware that basal rates starting on a half-hour basis are not supported, and you will need to adjust your basal rate profile to start on the hour. For example, if you have a basal rate of 1.1 units which starts at 09:30 and has a duration of 2 hours ending at 11:30, this will not work. You will need to change this 1.1 unit basal rate to a time range of either 9:00-11:00 or 10:00-12:00. Even though the Omnipod Dash hardware itself supports the 30 min basal rate profile increments, AAPS is not able to take them into account with its algorithms currently.

with:

AAPS basal rates Profiles only support full hour increments. IE: you can setup a basal of 1.1 between 11:00 and 15:00 but not between 11:30 and 15:30...

This paragraph is intended to inform users migrating from pump (or other APS) where the basal profile can be set to 30 minute time increments. They need to translate there profile to one using full hour time slots. I do not see the problem, nor a reason why the paragraph should be removed?

FlyingToto commented 1 year ago

@vanelsberg ,

the idea is to replace with a simple 1 liner without losing any of the sense to simplify the doc: "AAPS basal rates Profiles only support full hour increments. IE: you can setup a basal of 1.1 between 11:00 and 15:00 but not between 11:30 and 15:30"

vanelsberg commented 1 year ago

"_The Fill Pod screen is displayed. Fill a new pod with at least 80 units of insulin ...

Ad 1) Insulin DASH requires filling with a minimum of 80 units of insulin. Actually this is about a minimum filling volume. So yes: this could be considered as an error in the documentation. That said:

  1. In the same way should we replace "please take into account that priming the pod will use about 3-10 units." with "please take into account that priming the pod will use about 1ml"?

Ad 2) Why do you think that? That would be wrong: DASH does use approx 3-10 units (U100) for activating and priming.

  1. about the retry... replace: "NOTE: Just in case you get the below error message (this can happen), do not panic. Click on the Retry button. In most situations activation will continue successfully." with: "NOTE: Just in case you get the below error message (this can happen), do not panic. Click on the Retry button. In most situations activation will continue successfully; you might want to relocate and move the pod/phone to another room or moving yourself a few feet away from the phone/pods which should be a few inches apart at most ."

Ad 3) Nope. For DASH Documentation is exactly describing how to solve the problem. Do not change! (Your suggestion is to change for a solution that was need for the old Omnipod EROS witch used a Rileylink device)

FlyingToto commented 1 year ago

So I would argue there is no problem with the documentation and should be adapted once U200 is supported If anyone likes to update to 80 units (U100) I would not mind.

ok so

Ad 2) Why do you think that? That would be wrong: DASH does use approx 3-10 units for activating and priming.

well that's in our current documentation, so should I remove it then?
image

Nope. For DASH Documentation is exactly describing how to solve the problem. Do not change! (...old Rileylink device...)

kk no changes there then.

vanelsberg commented 1 year ago

@vanelsberg ,

the idea is to replace with a simple 1 liner without losing any of the sense to simplify the doc: "AAPS basal rates Profiles only support full hour increments. IE: you can setup a basal of 1.1 between 11:00 and 15:00 but not between 11:30 and 15:30"

I would not be against simplifying. I think below would even be more readable?

Like "The AAPS basal rate Profile only supports full hour timeslots. So you can not set a basal rate start or end time at 11:30 for example"

FlyingToto commented 1 year ago

"The AAPS basal rate Profile only supports full hour timeslots. So you can not use a basal rate start or end time 11:30 for example"

Perfect!

FlyingToto commented 1 year ago

BTW, regarding the 3-10units "wasted" during priming, does it even matter? especially with 100u/ml concentration, I don't think that people are going to calculate the exact amount of insulin to load in the pod down to 0.1ml. Don't you? Maybe we can delete it completely or move it as a footnote? what do you think?

vanelsberg commented 1 year ago

does it even matter? I would say it should not.

I think this is about the documentation trying to explain the discrepanties users are expected to see between the amount filled (referencing what is shown on the syringe when filling) and what is reported by DASH on the pod reservoir and insulin delivered (all in units U100).

(Btw: Yes, the documentation makes mention on this because users actually asked questions about this.)

vanelsberg commented 1 year ago

note: I can take care of updating the text once I get the feedback to my comments above.

Thanks! Let me know, happy to review your changes when done.

FlyingToto commented 1 year ago

@vanelsberg

note: I can take care of updating the text once I get the feedback to my comments above.

Thanks! Let me know, happy to review your changes when done.

Pull Request is here: https://github.com/openaps/AndroidAPSdocs/pull/1635 note: there is an unrelated update for the intro page as well..

jbr7rr commented 1 year ago

Please take the following PR's in mind which changes some screens: https://github.com/nightscout/AndroidAPS/pull/2700 : Adds some settings https://github.com/nightscout/AndroidAPS/pull/2549 : Adds things to the preference screens See also: https://github.com/openaps/AndroidAPSdocs/issues/1569

Also I think in general a lot of screenshots are outdated on the dash page