yo-han / Home-Assistant-Carelink

Unofficial Home Assistant Carelink Component
MIT License
19 stars 8 forks source link

No readings #58

Closed meluvalli closed 2 months ago

meluvalli commented 4 months ago

Hello. I'm in the US and tried using this HA component, but when I setup the "device" in HA, it pulls the patient correctly, shows me info about the pump, but there are no CGM readings (Last glucose level mg/dl = unknown).

I tried logging in with follower account and patient account and same exact results.

Any thoughts?

Screenshot 2024-01-14 025501

meluvalli commented 4 months ago

Ok. Found out that my wife's pump isn't connected the phone currently. It got disconnected and is stuck "reconnecting to pump".. This is why there is no "current" data.

So, now I have a feature request... Would be nice if it would pull the last "known" numbers... This is in the data file if I use the python client to download the data... So, would be nice if it pulled the latest "known" number and time stamp when it was. Once my wife's pump reconnects, I am hopeful this plugin will work and it will get the current numbers. Meanwhile, I can create a new sensor to auto update with the latest number manually!

yo-han commented 4 months ago

Hi @meluvalli, yes this is exactly how it works now. The disconnection of the pump, when batteries are dead or by bad connection, is a well known issue. I like the idea of keeping the latest know value of the most important sensors I't also very important to show the timestamp of this last measurement to prevent people from miscalculation with very old data.

meluvalli commented 4 months ago

Correct. I would envision it having something like “current reading” and then another value named “last known reading” or something like that so they are very different. That’s what I would recommend anyways. Then both with time stamps.

On Sun, Jan 14, 2024 at 7:50 AM Yo-han @.***> wrote:

Hi @meluvalli https://github.com/meluvalli, yes this is exactly how it works now. The disconnection of the pump, when batteries are dead or by bad connection, is a well known issue. I like the idea of keeping the latest know value of the most important sensors I't also very important to show the timestamp of this last measurement to prevent people from miscalculation with very old data.

— Reply to this email directly, view it on GitHub https://github.com/yo-han/Home-Assistant-Carelink/issues/58#issuecomment-1890973397, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTHBDKL47FAKLNF6WQWPDYOPWCTAVCNFSM6AAAAABBZ7IJ72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQHE3TGMZZG4 . You are receiving this because you were mentioned.Message ID: @.***>

greenhawk89 commented 3 months ago

Have you guys been able to figure out how to get a "last known reading" going? That would be great. I often don't know if it's the pump failing to connect, or if there's some kind of problem with the carelink integration.

sedy89 commented 3 months ago

grafik

I am testing some changes at the moment, including a new sensor that shows the timestamp of the last valid glucose reading. As you can see, the reading is already 25 min old but is still shown. The "Last update" is the old sensor that basically just shows the 5min update periode from the pump.

That's what you were looking for, right?

meluvalli commented 3 months ago

Yep! That sounds right!

Thank you!

On Fri, Feb 2, 2024 at 7:54 AM sedy @.***> wrote:

grafik.png (view on web) https://github.com/yo-han/Home-Assistant-Carelink/assets/65983953/591090f5-9b2d-49c6-9f13-d0e7b9620da7

I am testing some changes at the moment, including a new sensor that shows the timestamp of the last valid glucose reading. As you can see, the reading is already 25 min old but is still shown. The "Last update" is the old sensor that basically just shows the 5min update periode from the pump.

That's what you were looking for, right?

— Reply to this email directly, view it on GitHub https://github.com/yo-han/Home-Assistant-Carelink/issues/58#issuecomment-1924044181, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTHBAE3FJTZPEX7NV3HPTYRT43ZAVCNFSM6AAAAABBZ7IJ72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRUGA2DIMJYGE . You are receiving this because you were mentioned.Message ID: @.***>

yo-han commented 3 months ago

The new PR by @sedy89 is merged and released as a pre-release. If you like to test the release check the beta releases in Home Assistant for this add-on to try it out.

meluvalli commented 3 months ago

Thanks! I have installed it, and will let you know how it goes once the pump hasn't updated... (Probably when my wife replaces sensor with any luck)

meluvalli commented 3 months ago

Works perfectly! THANK YOU!!!

github-actions[bot] commented 2 months ago

This issue is stale because it has been open for 30 days with no activity.