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

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

Open
peterleimbach opened this issue Jun 23, 2023 · 16 comments

Comments

@peterleimbach
Copy link
Collaborator

peterleimbach commented Jun 23, 2023

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

Please PR on the branch rework-project.

@FlyingToto
Copy link
Contributor

FlyingToto commented Jun 23, 2023

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
Copy link
Collaborator Author

@vanelsberg Theodor can you please help here?

@FlyingToto
Copy link
Contributor

FlyingToto commented Jun 23, 2023

"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
Copy link
Contributor

FlyingToto commented Jun 23, 2023

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
Copy link
Contributor

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

@vanelsberg
Copy link
Contributor

vanelsberg commented Jun 23, 2023

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
Copy link
Contributor

FlyingToto commented Jun 23, 2023

@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
Copy link
Contributor

vanelsberg commented Jun 23, 2023

"_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:

  • Using U200 insulin with DASH by definition is unsupported use.
  • Using U200 insulin with AAPS is not unsupported
    User can use U200, but on there own risk, users required to know what they are doing.
    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.
  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
Copy link
Contributor

FlyingToto commented Jun 23, 2023

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

  • I can update the doc mentioning the (U100) in paranthesis.
  • I don't recall any warnings about the insulin concentration in the rest of the doc, so we can add it in the rework doc (probably in the intro and in each of the pump pages)

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
Copy link
Contributor

vanelsberg commented Jun 23, 2023

@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
Copy link
Contributor

"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
Copy link
Contributor

FlyingToto commented Jun 23, 2023

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
Copy link
Contributor

vanelsberg commented Jun 23, 2023

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
Copy link
Contributor

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
Copy link
Contributor

@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: #1635
note: there is an unrelated update for the intro page as well..

@jbr7rr
Copy link

jbr7rr commented Oct 12, 2023

Please take the following PR's in mind which changes some screens:
nightscout/AndroidAPS#2700 : Adds some settings
nightscout/AndroidAPS#2549 : Adds things to the preference screens
See also: #1569

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants