-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
AutoISF plugin prototype #3327
base: dev
Are you sure you want to change the base?
AutoISF plugin prototype #3327
Conversation
…oISFPlugin # Conflicts: # app/src/main/kotlin/app/aaps/activities/MyPreferenceFragment.kt # core/keys/src/main/kotlin/app/aaps/core/keys/BooleanKey.kt # core/keys/src/main/kotlin/app/aaps/core/keys/IntKey.kt # core/keys/src/main/kotlin/app/aaps/core/keys/UnitDoubleKey.kt # core/keys/src/main/res/values/keys.xml # plugins/aps/src/main/kotlin/app/aaps/plugins/aps/openAPSSMB/OpenAPSSMBPlugin.kt # plugins/aps/src/main/res/values/strings.xml
…oISFPlugin # Conflicts: # core/keys/src/main/kotlin/app/aaps/core/keys/BooleanKey.kt
…nto Philoul/2-in-1-AutoISFPluginDraft
…nto Philoul/2-in-1-AutoISFPluginDraft
This reverts commit 1897694.
…nto Philoul/2-in-1-AutoISFPluginKTtestDraft
@@ -47,5 +47,29 @@ data class OapsProfile( | |||
//DynISF only | |||
var variable_sens: Double, | |||
var insulinDivisor: Int, | |||
var TDD: Double | |||
var TDD: Double, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Create data class OapsProfileAutoIsf : OapsProfile
this makes the profile value confusing for other algs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added the dedicated OapsProfileAutoIsf file (without modifying database).
😉
make it available in engineering mode only for now (see Autotune plugin) |
You are right, I forgot to included it in the new proposal (I made a PR to ga-zelle branch, with engineering mode + dev branch like autotune). |
Dev + Engineering mode only for AutoISF
Create OapsProfileIsf
|
According to the hint given in the earlier submission (#3206) this version has AutoISF completely contained in its own plugin.
It represents AutoSF version 3.0 as described in https://github.com/ga-zelle/autoISF/blob/A3.2.0.2_ai3.0/autoISF3.0_Quick_Guide.pdf. However, some functionality was not yet included, namely
AutoISF version 3.0.1 is in its final testing. Once verfied it will be added and offers a reduced set of preferences and flatter preference menus.
The code was tested
A big thank you goes to @Philoul . Not only did he create the plugin structure but he also put in considerable work to align the architecture as much as possible with DynamicISF. This resulted in some code duplication but AutoISF is independent of the remaining code and can be tested and developed independantly.