nightscout / AndroidAPS

Opensource automated insulin delivery system (closed loop)
https://wiki.aaps.app
GNU Affero General Public License v3.0
685 stars 1.67k forks source link

Error in dynamicISF algorithm #3173

Closed mordechi-david closed 7 months ago

mordechi-david commented 7 months ago

In version dev3.3.0 build on 14.01.24 the dynamicISF gives an abnormal result Attached is a picture I don't find the log something strange in this version I don't have recent logs Screenshot_20240116-033118

MilosKozak commented 7 months ago

what is your dynisf factor? what it one time?

parapenT1sta commented 7 months ago

@mordechi-david About the logs: https://github.com/nightscout/AndroidAPS/issues/3151#issuecomment-1873488139

parapenT1sta commented 7 months ago

Same issue? https://github.com/nightscout/AndroidAPS/issues/3153

mordechi-david commented 7 months ago

Thanks @MilosKozak indeed this is the problem the factor changes to 0 as reported in issue #3153 (I did not change)

MilosKozak commented 7 months ago

you had to set it back to previous value manually?

mordechi-david commented 7 months ago

Yes, I restored it manually and it worked

MilosKozak commented 7 months ago

I checked the code. The code itself only reads the value. So, to modify you have to:

olorinmaia commented 7 months ago

@MilosKozak It happend on my test-rig now after updating to latest dev. I haven't touched the setting ofc. attaching screenshots and logs. Can you check if there is something in the logs that can be helpful if this can be improved?

Probably around 14:00 - 14:10

AndroidAPS_LOG_1705497291262.log.zip

image image

mordechi-david commented 7 months ago

@MilosKozak I did change the preferences and it works fine The only question is how it changed to 0 without me touching it

MilosKozak commented 7 months ago

app was started at 14:08 and then restarted first run was already wrong last run during night was ok somthing happened between

idalioreis commented 7 months ago

i think the dynamicISF factor is by default 0%, so cause this error, if update to value that i define it works.

olorinmaia commented 7 months ago

i think the dynamicISF factor is by default 0%, so cause this error, if update to value that i define it works.

default value is 100

ApsDynIsfAdjustmentFactor(R.string.key_dynamic_isf_adjustment_factor, 100, 1, 300),

idalioreis commented 7 months ago

i think the dynamicISF factor is by default 0%, so cause this error, if update to value that i define it works.

default value is 100

ApsDynIsfAdjustmentFactor(R.string.key_dynamic_isf_adjustment_factor, 100, 1, 300),

i understand, but i never change this value, only choose dynamicISF and was 0%.

i put 100% after, and work fine.

szantos commented 7 months ago

I've entered preferences to check the value (did not edit it), and it was zero. I've set it back to the normal value, but 1 loop run was affected, and I had a similar high ISF. Unable to provide logs.

MilosKozak commented 7 months ago

what was the value before?

szantos commented 7 months ago

80

MilosKozak commented 7 months ago

https://github.com/nightscout/AndroidAPS/commit/bc0eac0eb4900c401753cd514f3d2ea683e6de69