Update Defaults (high, rulerMarks, maxCarbs, sensitivityRaisesTarget)

[MikePlante1]

high (for stats & main chart line): 145 -> 180
rulerMarks (the high/low lines on the main chart): false -> true
maxCarbs: 1000 -> 250
sensitivityRaisesTarget: true -> false

[bjornoleh]

Looks good. Should we also take a Quick Look at the defaults inPreferences (OpenAPS settings)? I believe there are some things that could be off. Specifically, “Sensitivity Raises Target” currently defaults to ON. I suggest change this to OFF.

Good to get this done now that setting are being reset by similar changes.

[MikePlante1]

Specifically, “Sensitivity Raises Target” currently defaults to ON. I suggest change this to OFF.

Looks like OpenAPS defaults that to ON as well, though: https://github.com/openaps/oref0/blob/88cf032aa74ff25f69464a7d9cd601ee3940c0b3/lib/profile/index.js#L24

[MikePlante1]

Looks like OpenAPS also defaults UAM to ON, though. 🤔

[bjornoleh]

Specifically, “Sensitivity Raises Target” currently defaults to ON. I suggest change this to OFF.

Looks like OpenAPS defaults that to ON as well, though: https://github.com/openaps/oref0/blob/88cf032aa74ff25f69464a7d9cd601ee3940c0b3/lib/profile/index.js#L24

Thanks for checking. What’s the verdict on that setting in our app? How does it fit if enabling dynamic ISF? My impression from looking at others discussing it is that it causes more problems than it solves, but I have always turned it off, so can’t really tell.

[bjornoleh]

Looks good! Will merge this now with two approvals.

[MikePlante1]

Adding for transparency that this was discussed more extensively on Discord.