pazaan / 600SeriesAndroidUploader

Your Medtronic 600-series pump data, direct to Nightscout
http://pazaan.github.io/600SeriesAndroidUploader/
MIT License
189 stars 312 forks source link

Showing closed-loop-67-G micro-boluses on nightscout #167

Closed kretch closed 6 years ago

kretch commented 6 years ago

I would like to see the micro boluses the 670G does ( in closed-loop mode) on nightscout.

Your Environment

Latest version of uploader S8+, OTG to CNL24, uploading to NS site http://yaronbginfo.herokuapp.com/ Just FYI A Dexcom G5 also uploads to the same site

Brief Explanation of Issue

NA

Uploader Error Code

NA

Steps to Reproduce (for bugs)

NA

Severity Score

1

pazaan commented 6 years ago

That work is already in progress.

kretch commented 6 years ago

Thanks. Is that part of the develop-indra branch?

pazaan commented 6 years ago

No, it's in a local branch on my machine at the moment. I'll push it to a separate branch so that we don't interrupt getting an Indra release out ASAP.

kretch commented 6 years ago

Great, thanks! I was able to build Indra and use it on my s8, so looking forward to trying out the local branch

On Fri, Jan 5, 2018, 15:36 Lennart Goedhart notifications@github.com wrote:

No, it's in a local branch on my machine at the moment. I'll push it to a separate branch so that we don't interrupt getting an Indra release out ASAP.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-355694284, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMEcUul1NktFP3OtnFhp5fRiNn_gnks5tHrHkgaJpZM4RTR1H .

kretch commented 6 years ago

Hi Lenhart Would it be possible to get access to the branch with the micro-bolus upload changes? TIA

Yaron

pazaan commented 6 years ago

It's being worked on at https://github.com/pazaan/600SeriesAndroidUploader/tree/pazaan/develop-indra-670G

kretch commented 6 years ago

Thanks much!

On Wed, Jan 10, 2018 at 3:55 PM, Lennart Goedhart notifications@github.com wrote:

It's being worked on at https://github.com/pazaan/ 600SeriesAndroidUploader/tree/pazaan/develop-indra-670G

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-356777841, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMLe2CfbeEtJyRIP91o4S4divsAmeks5tJU3-gaJpZM4RTR1H .

kretch commented 6 years ago

The develo-indra-670G worked great, and I can see the micro boluses on my NS site - thank you so much!

Pogman commented 6 years ago

@kretch there are some issues with the 670G that are being worked on but we are limited when it comes the devs that have a 670G to work with - if your interested in trying out the debug version and your ok with us viewing your NS site please build and run the branch here: https://github.com/pazaan/600SeriesAndroidUploader/tree/develop-indra-670G-debug

kretch commented 6 years ago

I built develop-indra-670G , installed on Samsung S7, and you can see the NS results in http://yaronbginfo.herokuapp.com/

Thanks again!

On Tue, Jan 16, 2018 at 10:00 AM, Pogman notifications@github.com wrote:

@kretch https://github.com/kretch there are some issues with the 670G that are being worked on but we are limited when it comes the devs that have a 670G to work with - if your interested in trying out the debug version and your ok with us viewing your NS site please build and run the branch here: https://github.com/pazaan/600SeriesAndroidUploader/tree/ develop-indra-670G-debug

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358050139, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMHDtKxEK0hwSKthsPCkJ20wlh6hVks5tLOOmgaJpZM4RTR1H .

Pogman commented 6 years ago

It needs to be this branch "develop-indra-670G-debug" to be of use for us.

kretch commented 6 years ago

Building that now, should have it uploading data into NS within the hour.

On Tue, Jan 16, 2018 at 10:32 AM, Pogman notifications@github.com wrote:

It needs to be this branch "develop-indra-670G-debug" to be of use for us.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358060441, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMD6raisGpp9HFEyTmX_FBvg6WDx5ks5tLOtbgaJpZM4RTR1H .

Pogman commented 6 years ago

cool - thanks!

You can just let it run as we need to see the debug data as you use it over time.

kretch commented 6 years ago

Ok,debug version installed and running

On Tue, Jan 16, 2018, 10:40 Pogman notifications@github.com wrote:

cool - thanks!

You can just let it run as we need to see the debug data as you use it over time.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358062814, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMAsi__YdJTjUq5rbciL0rDoATo39ks5tLO0wgaJpZM4RTR1H .

kretch commented 6 years ago

Just FYI- The CNL keeps disconnecting with the debug version (vs. in the non-debug indra 670G, where the CNL doesn't disconnect)

On Tue, Jan 16, 2018 at 11:41 AM, Yaron Kretchmer yaronkretchmer@gmail.com wrote:

Ok,debug version installed and running

On Tue, Jan 16, 2018, 10:40 Pogman notifications@github.com wrote:

cool - thanks!

You can just let it run as we need to see the debug data as you use it over time.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358062814, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMAsi__YdJTjUq5rbciL0rDoATo39ks5tLO0wgaJpZM4RTR1H .

Pogman commented 6 years ago

ok shouldn't be doing that at all - what error is it showing in the log on your uploader? also you can go into the uploader settings under debug and turn on "extended errors"

try restarting your phone and if it still does it try clearing the app data

kretch commented 6 years ago

ok, changed phones and uploader running again.

On Tue, Jan 16, 2018 at 1:32 PM, Pogman notifications@github.com wrote:

ok shouldn't be doing that at all - what error is it showing in the log on your uploader? also you can go into the uploader settings under debug and turn on "extended errors"

try restarting your phone and if it still does it try clearing the app data

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358113576, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMGshv_C6L3nFXYk4YC7ZeDtvq28Pks5tLRVpgaJpZM4RTR1H .

Pogman commented 6 years ago

great - I see some debug data coming through too - thanks for the help

kretch commented 6 years ago

App keeps crashing (Nougat SDK 24, I'm trying to rebuild the APK on 24 just to be sure)

On Tue, Jan 16, 2018 at 2:05 PM, Pogman notifications@github.com wrote:

great - I see some debug data coming through too - thanks for the help

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358123104, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMB97n-jgbcf9ad_0IhQRJ2sglp-Xks5tLR0ggaJpZM4RTR1H .

aboncher commented 6 years ago

Yeah, keeps crashing at the end of attempting to get history. Actually, I see the full SG history on the uploader graph, but it never makes it to nightscout.

info.nightscout.android_issue_1_crash_5A5E7A1802BE00017C5554A0765DEB63_7c47ed64fb0b11e7b2a556847afe9799_0_v2.txt

Pogman commented 6 years ago

ok I'll have a look

Pogman commented 6 years ago

Updated debug branch - should be working ok now

aboncher commented 6 years ago

Running it now, looks to be working well!

Other note: I see lots of Fingerstick BG entries (that I didn't take). They show the same BG value that I used for my most recent calibration..

The first red BG value is an actual test, the other 5 are duplicate values but not from a test.

image

kretch commented 6 years ago

Seems to be running well for me as well. Thanks!

On Tue, Jan 16, 2018 at 5:15 PM, aboncher notifications@github.com wrote:

Running it now, looks to be working well!

Other note: I see lots of Fingerstick BG entries (that I didn't take). They show the same BG value that I used for my most recent calibration..

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358162165, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMOmQre_rEX8rL2B8vaEXp08W9w2Zks5tLUmYgaJpZM4RTR1H .

kretch commented 6 years ago

App is crashing again now, only returns to working after "clear data"

On Tue, Jan 16, 2018 at 5:18 PM, Yaron Kretchmer yaronkretchmer@gmail.com wrote:

Seems to be running well for me as well. Thanks!

On Tue, Jan 16, 2018 at 5:15 PM, aboncher notifications@github.com wrote:

Running it now, looks to be working well!

Other note: I see lots of Fingerstick BG entries (that I didn't take). They show the same BG value that I used for my most recent calibration..

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358162165, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMOmQre_rEX8rL2B8vaEXp08W9w2Zks5tLUmYgaJpZM4RTR1H .

Pogman commented 6 years ago

the pump records a lot of bg data on the 670 and the debug ver will show me this in detail for now what is the crash error? would it be ok to see your NS site? it would be really useful for this - you can message me on Gitter

kretch commented 6 years ago

Sure, http://yaronbginfo.herokuapp.com/ The crash error is just the generic "600 Series Uploader has stopped".

On Tue, Jan 16, 2018 at 5:32 PM, Pogman notifications@github.com wrote:

the pump records a lot of bg data on the 670 and the debug ver will show me this in detail for now what is the crash error? would it be ok to see your NS site? it would be really useful for this - you can message me on Gitter

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358165239, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMJFz_dplJQjMWME-x9vQvTAF0d0Fks5tLU20gaJpZM4RTR1H .

Pogman commented 6 years ago

@kretch sorry thought it was @aboncher :) I was talking too - it's late here

kretch commented 6 years ago

np :) If you need the site info, let me know.

On Tue, Jan 16, 2018 at 5:34 PM, Pogman notifications@github.com wrote:

@kretch https://github.com/kretch sorry thought it was @aboncher https://github.com/aboncher :) I was talking too - it's late here

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358165641, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMLsFLEpVzznHgwFgUtzv5LaEGfkjks5tLU5AgaJpZM4RTR1H .

kretch commented 6 years ago

BTW, the fabric crash URL is https://fabric.io/myself11111111111516153267/android/apps/info.nightscout.android/issues/5a5eae258cb3c2fa63faffcb?time=last-seven-days

On Tue, Jan 16, 2018 at 5:35 PM, Yaron Kretchmer yaronkretchmer@gmail.com wrote:

np :) If you need the site info, let me know.

On Tue, Jan 16, 2018 at 5:34 PM, Pogman notifications@github.com wrote:

@kretch https://github.com/kretch sorry thought it was @aboncher https://github.com/aboncher :) I was talking too - it's late here

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358165641, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMLsFLEpVzznHgwFgUtzv5LaEGfkjks5tLU5AgaJpZM4RTR1H .

aboncher commented 6 years ago

@Pogman https://abns.herokuapp.com

Pogman commented 6 years ago

@aboncher thanks for that

Pogman commented 6 years ago

@aboncher those extra bgs you see are not part of this versions code - strange. The code that generates that type of BG entry (top and bottom of graph) doesn't even exist is this version, it does in the v5.1 release version.

aboncher commented 6 years ago

I also ran the release version today. So if that's a known bug in the release it makes sense.

Pogman commented 6 years ago

ok must be to do with that - do you use the meal wizard and enter carbs etc or just bolus a unit amount for food? this is an area I'm looking to target over the next few days with the feedback from this version.

aboncher commented 6 years ago

In automode with 670g you have to use the bolus wizard. You can only adjust BG and carbs input, not the total units.

Pogman commented 6 years ago

You always have to enter carbs?

aboncher commented 6 years ago

You can only adjust the 2 inputs, BG and carbs. You can leave carbs at 0 and it will calculate a correction if needed.

Pogman commented 6 years ago

Great thanks

Pogman commented 6 years ago

new version uploaded

kretch commented 6 years ago

Thanks much! Still in the indra-670G-debug branch?

On Wed, Jan 17, 2018 at 11:18 AM, Pogman notifications@github.com wrote:

new version uploaded

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358412546, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMJGRZbju8FoSs7NB6flo9BYpKGY6ks5tLkeQgaJpZM4RTR1H .

aboncher commented 6 years ago

Running it now :)

kretch commented 6 years ago

It's been uploading steadily for the last 30min- looking good!

On Wed, Jan 17, 2018 at 12:03 PM, aboncher notifications@github.com wrote:

Running it now :)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358426724, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMGmQIowlLTs9R6DaNeY6HvxbFIQIks5tLlH1gaJpZM4RTR1H .

Pogman commented 6 years ago

@aboncher data coming through from new update

@kretch yours still looks like it's on an older version

kretch commented 6 years ago

@pogman- thanks, I changed to the latest code and restarted. Is the version correct now?

On Wed, Jan 17, 2018 at 12:18 PM, Pogman notifications@github.com wrote:

@aboncher https://github.com/aboncher data coming through from new update

@kretch https://github.com/kretch yours still looks like it's on an older version

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358431424, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMF2M7hijIBxTr4YPoR60fCYOFA95ks5tLlWegaJpZM4RTR1H .

Pogman commented 6 years ago

looks like it's still the old one - it the note it drops to NS every 30mins you should see a value like "AI: 3.45" and it's missing.

aboncher commented 6 years ago

@kretch make sure you're pulling correctly, I had 1 merge conflict I had to resolve.

kretch commented 6 years ago

Sorry about that- apparently I'm building it incorrectly somehow.

On Wed, Jan 17, 2018 at 4:30 PM, aboncher notifications@github.com wrote:

@kretch https://github.com/kretch make sure you're pulling correctly, I had 1 merge conflict I had to resolve.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pazaan/600SeriesAndroidUploader/issues/167#issuecomment-358496602, or mute the thread https://github.com/notifications/unsubscribe-auth/AHXFMLV0Wm6ThptB8-EtmnCcrjefCH4_ks5tLpCXgaJpZM4RTR1H .

Pogman commented 6 years ago

Another update today - some nice stuff in this one!

aboncher commented 6 years ago

Pulling carbs from bolus events now 👍 👍

image