-
Notifications
You must be signed in to change notification settings - Fork 7.3k
Add support for DanaKit and MedtrumKit to Loop #402
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
base: dev
Are you sure you want to change the base?
Conversation
Update medtrum with dev updates
chore: update MedtrumKit for IOB fix
* prevent action while bolus in progress, * show No Patch when deactivated * crowdin updates
* add reservoir age for insulin age, * update translations, * 299331d...92d2cd7
Dana StatusPlaceholder for summarizing Dana. |
Medtrum StatusPlaceholder for summarizing Medtrum. |
Medtrum testFeature requests
DetailsDescriptionI tested a Medtrum patch for myself. Everything was nominal. I did not notice much difference from when I use an Omnipod.
I let it go past the 3 day plus 8 hour mark so I could see what happened when the pump ran out of insulin. I think the warning provided by Loop was inadequate. There should be a more urgent message that the pump is out of insulin. The Medtrum does have an on-patch alert, but it is incredibly quiet and very high pitched. SHAThe Medtrum SHA for the test where I ran the pump out of insulin was Although this commit was available before the end of the patch, I did not build it to my phone (sorry): I did update my build after applying the new patch. The Loop Report is from before the update. FilesThe LoopReport and daily Medtrum logs are provided. |
|
Thank you for testing @marionbarker ! Medtrum Feature request:
|
Purpose
Draft PR while we work through updates required before landing the support for DanaKit and MedtrumKit into LoopWorkspace dev branch.
The mixing of these two pump managers is for my convenience. Both are at about the same stage of development and are being developed by the same person, @bastiaanv.
Method
Testers will continue to build the feat/dev-dana-medtrum branch for testing. This branch can be updated as frequently as needed without a formal review process - although every update includes local testing prior to pushing the update to GitHub.
When a tester has an issue or concern, they should add a comment to this PR.
When we are ready to get formal approval for the merge, the target branch for this PR will be moved to a new branch, update_dev_to_M.m.x, with the appropriate version number.
Documentation
I will configure 2 comments, one for Dana and one for Medtrum, that can be edited as testing proceeds. These comments will summarize status and indicate must-have and nice-to-have features for the Pump Managers before landing this PR.
Testing
Testers should create a new comment each time they wish to document a bug or feature request for the pump.
For your comment - please Include:
Share Dana/Medtrum logs