Theodoor

Results 150 comments of Theodoor

UI indicator for SMB on/off would be an important feature👍 My use case: using automation to switch on/off from Watch tile. But as is now there is no feedback. In...

https://github.com/nightscout/AndroidAPS/issues/3415

> In some usecases the SMBs (Loop) are not desired. Don't understand why you'd enable LGS. Loop is able to handle this, so if manually intervention by setting LGS is...

Personally, I have a lot of problems with this PR. I think there is no need for this change and changes may negatively affect wear watch performance. > Logs will...

**Note**: This _PR also changes logging behavior for the Wear App_. This negatively affects the Wear app possibly creating problems. Current settings of 7 days is **intentionally** to prevent the...

_Summarizing my review comments on PR changes above:_ **1) Wear logging**: By designs: do not chnage! Current values are optimized and tested for preventing storage problems on Wear Watches. **2)...

> A dedicated plugin for external loop analysis/emulator, with all required information managed in a structured and stable way would be a much more robust solution: Fully agree with that....

Yes. By design and this is correct. > Why did you do the "Close All" while bolus was executing... I did to show that _when you do_, there is no...

**Imho**: Any situation where _user loses control_ over what the pump is doing is "not be design" and "potentially dangerous" 🤔

@MilosKozak Merged PR4229 with current dev commits. @Angus-repo No issues (used your AAPS pr-CI btw 😄) NICE! Works as expected ❤️