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

ISF calculation and results in autosense.json #120

Open
mountrcg opened this issue Nov 17, 2021 · 0 comments
Open

ISF calculation and results in autosense.json #120

mountrcg opened this issue Nov 17, 2021 · 0 comments

Comments

@mountrcg
Copy link
Contributor

mountrcg commented Nov 17, 2021

Facts:

  • Autosense calculated Sensitivity of 1.28
  • I had autotune activated. Its calculation resulted in 44mg/dl/U
    E38B69D9-E94C-4596-8466-1A2DD1430F7C
    (... it did run at 1am in the morning, when I analysed I deactivated autotune and deleted results, and re-ran it again for the screenshot)
  • scheduled ISF from pump at 6 am was 40 mg/dl/U
  • scheduled ISF from pump at 4 am was 44 mg/dl/U

Oref calculated an adjusted ISF:
A43E65A9-385E-4172-89F5-3299E26C8E06_1_105_c
... using the scheduled ISF as basis. Also the suggestion included that calculation:
EDBBEBC8-DE5A-4DF2-8BE1-7382A70353C7_1_105_c
So that was used to adjust the therapy.

However autosens.json showed a calculation based on the scheduled ISF from 04:43 am that was also on 44 (same value as the autotune result), resulting in ISF of 34:
0E2B519A-4B5F-474C-AD9B-89C8E4715531_1_105_c

Two issues come to mind:

  1. shouldn't oref use the autotuned ISF instead of the scheduled one
  2. shouldn't autosens.json show the current parameters instead of old ones

As I am not sure which value to look at in FAX to know the actual ISF beeing applied I have included the ISF from oref suggestion (2nd picture, as this is the calculations truth) into suggestion.json and showing it on HomeScreen as I noticed above sketched discrepancies.

ivalkou pushed a commit that referenced this issue Nov 17, 2021
* include ISF in suggestion.json

* make ISF available via $state.isf
bjornoleh pushed a commit to bjornoleh/freeaps that referenced this issue Dec 18, 2021
mountrcg added a commit to mountrcg/freeaps that referenced this issue Feb 9, 2022
adjust
ISF on Header using glucose view

fix spacing
Partly address ivalkou#120 and include ISF in suggestion.json  (ivalkou#121)

* include ISF in suggestion.json

* make ISF available via $state.isf
adjustments for GlucoseView & ISF
ISF text color


HomeView
fix HomeView
alignment & size header
fix decimal ISF and default app group

fix ISF view


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

No branches or pull requests

1 participant