Skip to content

Battery Status Empty #141

Closed
Closed

Description

@ashrind

Any thoughts as to why my battery status might not be filling in? Running on a 722, changed the battery when I started with Loop last night
img_7367

Activity

  1. loudnate commented on Aug 31, 2016

    @loudnate
    Collaborator

    This is a know issue with the x22 pumps. @jlucasvt has started documenting the time-voltage curve for a lithium battery. I'm hoping someone can do the same for an alkaline as well.

  2. ashrind commented on Aug 31, 2016

    @ashrind
    Author

    @loudnate Thanks for the answer!

  3. jlucasvt commented on Sep 6, 2016

    @jlucasvt
    Contributor

    I'm finally committed to gathering Alkaline data. Should have it mapped out in about a week.

  4. tomasboudr commented on Sep 20, 2016

    @tomasboudr

    Awesome that this is getting some love! Always wondered why the battery was blank since you could see the voltage in pump status on a x22.

  5. jlucasvt commented on Sep 24, 2016

    @jlucasvt
    Contributor
  6. loudnate commented on Sep 24, 2016

    @loudnate
    Collaborator

    Very awesome!

  7. loudnate commented on Sep 24, 2016

    @loudnate
    Collaborator

    @jlucasvt my personal recommendation would be to expose a voltage-to-percentage remaining function on the PumpModel enum in MinimedKit, in the rileylink_ios repo.

  8. jlucasvt commented on Sep 25, 2016

    @jlucasvt
    Contributor

    Yes if we use the linear eq's it should be pretty simple. I I did do a quick run with excel to generate a 4th Ord Polynomial Eq for both but they may be a bit overboard..

    screen shot 2016-09-25 at 10 45 21 am

  9. loudnate commented on Sep 25, 2016

    @loudnate
    Collaborator

    Ideally there is no user setting for choosing their battery type.

  10. tomasboudr commented on Sep 27, 2016

    @tomasboudr

    I agree with you nate, ideally - but if its tricky to detect in software, you have to realize its a switch that will be used very rarely. I would rather time be spent on other things and there be a switch.

  11. jlucasvt commented on Sep 27, 2016

    @jlucasvt
    Contributor

    Yeah , My one touch ping made you choose on battery swap (ALK or LITH). It will be ideal to not worry about the battery type.

    In the past I wired this up in the DeviceDataManager.assertCurrentPumpData just after battery state is retrieved and before NSUpload.

    if let sentrySupported = self.pumpState?.pumpModel?.larger , !sentrySupported {
         self.setBatteryStatusforNonMySentryPumps(voltage: status.batteryVolts)
    }
    

    I am going to start here and and then will need some guidance on the next steps

  12. scottleibrand commented on Sep 27, 2016

    @scottleibrand

    We could certainly make is a 3-way selection: unspecified, alkaline, or lithium. If unspecified (default) we use an average decay curve that does something reasonable for both. If specified, we can be more accurate...

  13. jlucasvt commented on Oct 23, 2016

    @jlucasvt
    Contributor

    Updated to latest v1.1 Loop Code Testing this week and will do a PR.

    https://github.com/jlucasvt/Loop/tree/master-1.1-x22BatteryStatus

  14. added a commit that references this issue on Oct 24, 2016
  15. jlucasvt commented on Oct 24, 2016

    @jlucasvt
    Contributor

    Doing some testing now with battery and alerts. It's working pretty good.

    x22 Battery Inidication

    img_6151

    Lithium Enable Switch

    img_6152

  16. tomasboudr commented on Oct 24, 2016

    @tomasboudr

    That looks good! Nice work.

  17. ashrind commented on Oct 24, 2016

    @ashrind
    Author

    This is awesome!

  18. jlucasvt commented on Nov 15, 2016

    @jlucasvt
    Contributor

    I have rebuilt the UI to allow selection of Alkaline or Lithium and a description explaining the difference for the user. I've made all the requested changes from my last PR and hopefully it will be accepted this time. Here is the current source it's based on dev 1.1.1

    https://github.com/jlucasvt/Loop/tree/dev-1.1.1-x22BatteryStatus

    img_6206

  19. ps2 commented on Nov 22, 2016

    @ps2
    Collaborator

    More lithium battery history:
    devicestatus

  20. tomasboudr commented on Jan 16, 2017

    @tomasboudr

    This is working... One minor note though is that in practice with normal Duracell (alkalines) my battery has lasted 30-40% (ballpark) longer than loop's current algorithm predicts. I have had it warm me for 0% and the pump still had 2/4 bars on the battery meter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions