Step counter history added #2120
Conversation
|
Build size and comparison to main:
|
|
Nice! You should be able to test this on your sealed watch without problems. As long as you don't verify the firmware, a reboot will be enough to get it back to working. And if you do verify it, a force reboot into the backup firmware still enables you to install another firmware. |
|
Sorry for the seemingly endless review! It's looking really good now, almost there |
|
It's alright, I'm actually enjoying this opportunity to "nerd out" with people on a project 😄 |
Hunman
force-pushed
the
step-counter-history
branch
from
20168fa to
7f7189e
Compare
Hunman
force-pushed
the
step-counter-history
branch
from
7f7189e to
85c1f4e
Compare
Since most if not all calls expect today, I set it as default value
Hunman
force-pushed
the
step-counter-history
branch
from
85c1f4e to
e973125
Compare
|
Very nice! Do you have any InfiniSim screenshots or pictures of it on a real watch? |
|
Yes, I have some pictures both in InfiniSim and on the real watch Nothing fancy, so far. If there's demand, I'm thinking about storing the past 7 or so days and displaying it in a table (like the tasks in the Settings > About menu's 4th screen). Eventually I'd also like to tell companion applications the step history count, but I'll need to look more into how BLE communication works for that.
|
I changed the MotionController to remember the past N (currently 2) day's step counter history after the midnight reset. And I also changed the Steps screen to display (and update) yesterday's step count in addition to today's.
I didn't feel comfortable changing the return type of
MotionService::NbSteps()yet without feedback, instead I wrote a new method to return the whole history.I've tested this only using InfiniSim, testing with devkits would be welcome!
In the future I would like to increase the history size from 2 to 7 and transfer the whole history to the companion applications, but I haven't really looked into how bluetooth communication or "API"s work, so feedback would be most appreciated!