nightscout / AndroidAPS

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

BG Source "xDrip+ BG": settings in the wrong place / duplicated #3434

Open Gohtraw opened 1 month ago

Gohtraw commented 1 month ago

This is a minor thing I believe: In the config builder, section "BG Source" xDrip+ BG is the plugin that gets BG data from xDrip+. Screenshot 2024-09-09 121223 The only preferences this plugin has are: "BG upload settings"

There is another config builder section "Synchronization" NSClientV3 for data that's being sent from AAPS to NS. Screenshot 2024-09-09 121301

XXilay commented 6 days ago

I can confirm right now that this is not a purely cosmetic issue. The discussion of what exactly happened can be found on the discord, (https://discord.com/channels/629952586895851530/629954570394533889/1296464093993111602 ) but ill reiterate again here:

If you (apparently) use the search function to set these values, it ends up activating the same options twice and is not automatically turned off in 1 of the plugins like it would when edited directly under "plugin preferences"

This in turn causes aaps to send a sensor change entry and all data with it to nighthawk on every blood glucose update sent by the CGM. (from what i read off the logs, the data sent with that sensor change just has all fields set to null)