Skip to content
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

Battery life improvements #8

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

Katharine
Copy link

Hi there!

We have noticed that Futura Weather can exhibit some behaviour that causes moderate or severe reductions in watch battery life. This PR aims to fix those.

In particular, this PR:

  • Does not reload the weather image from flash every second
  • Only updates the displayed weather information when it actually changes, instead of every clock tick
  • Does not get stuck in a retry loop that kills the battery in under a day when disconnected
  • Reduces the tick interval from once per second to once per minute
  • Makes use of the bluetooth service for retries and icon display
  • Simplifies some logic

Please take a look and merge.

Thanks! :D

@Katharine Katharine changed the title Pebble fixes Battery life improvements Jun 13, 2014
@m-p-3
Copy link

m-p-3 commented Jun 24, 2014

Can I assume the version in the Pebble Store has those fixes implemented?

Thanks Katharine!

@lepinsk
Copy link
Contributor

lepinsk commented Jun 24, 2014

@Katharine – I'm confused; did you guys just replace @Niknam's watchface with your own fork of it on the Pebble watchapp store?

@Katharine
Copy link
Author

Yes. We asked first.

@lepinsk
Copy link
Contributor

lepinsk commented Jun 24, 2014

Ah, that makes sense. :-)

On Tue, Jun 24, 2014 at 6:10 PM, Katharine Berry [email protected]
wrote:

Yes. We asked first.


Reply to this email directly or view it on GitHub
#8 (comment)
.

@Katharine
Copy link
Author

Do note that due to an appstore bug, updating an app actually requires that you unload it from your watch, remove it from your locker, and add it back from the appstore page.

You may find this hard to do due to another appstore bug that sometimes breaks removing apps from your locker (sigh…).

We are working on a way of fixing this both for the special case of Futura Weather, as well as globally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants