nightscout / AndroidAPS

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

Enable the nsclient local broadcast even when uploads are disabled #1997

Closed 4RK4N closed 2 years ago

4RK4N commented 2 years ago

Time: always Build: 3.1.0.2 Steps to reproduce:

  1. setup nsclient to only upload on wifi when charging or disable all nsclient uploads manually via the sync settings
  2. set xdrip+ to recieve treatments from local broadcast, disable treatments download from ns
  3. enable aaps nsclient local broadcast to xdrip+
  4. enter any treatment (bolus, profile change, sensor change, etc)
  5. see the treatment in aaps treatments logs
  6. see no treatment in xdrip+
  7. re-enable nsclient uploads
  8. wait for upload queue to be cleared
  9. see treatment in xdrip+ now

Apparently the local broadcast of nsclient seems to be tied to the upload queue, specifically getting send after the queue is cleared. if no uploads happen, so if the queue doesnt get cleared, no local broadcast is sent either. that kinda contradicts the term "local" f it only gets sent when im "online".

proposed solution: separate the queue uploads and the local broadcast. local broadcasts shouldnt be tied to a "be online" condition.

Reporting bugs

MilosKozak commented 2 years ago

I already explained it elsewhere This function is not xDrip - AAPS synchronization but broadcast of data coming from NS to avoid multiple downloads. So it's working like expected

nlof commented 2 years ago

@MilosKozak With the looming shutdown of Heroku's Nightscout service, there should be a way for Xdrip to receive treatments from AAPS directly, without involving NS at all...

Even before that, I don't think It makes sense to force people to use NS and the internet in order for the local Xdrip on the same phone to receive AAPS treatments... or am I missing something?