nightscout / Trio

Trio - an automated insulin delivery system for iOS based on the OpenAPS algorithm with adaptations.
https://docs.diy-trio.org/en/latest/
MIT License
95 stars 485 forks source link

Add TidePool service in OiAPS #30

Closed avouspierre closed 6 months ago

avouspierre commented 7 months ago

This PR adds the TidePool service in OiAPS as described in https://github.com/nightscout/Open-iAPS/discussions/16

https://github.com/nightscout/Open-iAPS/assets/4339604/5d287a93-e4d3-4181-8018-5b375bddf0cb

The service is available in settings :

CleanShot 2024-03-27 at 21 37 44@2x

The implementation did not include:

This implementation was tested with Frestyle and DexCom G6 and Dash pump on iphone 15.

marionbarker commented 7 months ago

Normally a right caret is used to indicate a row will open a new screen. I think that should be added to the Tidepool row.

marionbarker commented 7 months ago

I don't know if it was operator error or not, but I had add TidepoolService submodule to the workspace index to get Browser build to work. See marionbarker:mdb_testing

% git diff 0eee7965 7ec257e2
diff --git a/TidepoolService b/TidepoolService
new file mode 160000
index 00000000..a2ccad72
--- /dev/null
+++ b/TidepoolService
@@ -0,0 +1 @@
+Subproject commit a2ccad72a55600c28549ab86ab1964c0d6558868
marionbarker commented 7 months ago
  1. I logged out from the test Tidepool account I've been using with a Loop test phone.
  2. I logged in using Open-iAPS 0.1.0 (my mdb_testing branch).
  3. First time I looked at Tidepool in browser, I got error message shown below.
  4. I logged out and opened again and Open-iAPS data is showing now with Loop data showing for previous days.

Error message (first time only) from Tidepool:

MikePlante1 commented 7 months ago

Looks like all boluses are labeled as "automated". This highlighted bolus was manual.

(This test was done via the alpha branch, not specifically this PR, btw)

Tidepool Manual Bolus
bjornoleh commented 6 months ago

Tidepool support was already added to Alpha in #67. Closing this PR