BasedHardware
Battery indicator by double tap
1/ added double tap with battery indicator on device
2/ added docs with info about led colors and it's meaning
https://github.com/user-attachments/assets/8b8700fd-a377-41b2-9b13-f8855d5ec8e3
cool man. please split this PR into 2 parts, one for bug fix and another onr for the new feature.
for the bug fix, please tell me more about the 4096 number. why is this and why we need this ?
@skywinder
do you mean 4074?
It's s the maximum voltage my Omi gets when it’s fully charged. That corresponds to about 96%, which is what I usually see in the app when it’s fully charged.
The better approach would be to gradually adjust the percentage rather than just cutting off the top value — but that would require more changes. For now, I want to keep it minimal. So the battery indicator in the app will show 100%, then drop to 97% sharply, and from there start decreasing as usual.
0/ ok, replied https://github.com/BasedHardware/omi/pull/2154#issuecomment-2782426832
1/ cool man ~ btw, what is the indicator system are you following for the battery-related indicator ? it looks complicated a bit so i want to understand more about how our users could adapt with this system.
@skywinder
hey man, it's cleaning time.
/ closed
3 days with no updates feel free to reopen it anytime
it looks complicated a bit so i want to understand more about how our users could adapt with this system.
So, I’ve tested five different options—different colors, blink counts, and frequencies—and I found this one to be the most straightforward. But I’m totally open to feedback—maybe you have a better idea for how to handle the indication.
The only thing I really want is to be able to check the battery on the OMI without needing my phone nearby. The simplest way is just to blink from 1 to 10 times, maybe with different colors—but counting up to 10 blinks is actually pretty hard in practice, so I decided to simplify it.
I went with a maximum of 3 blinks using different colors to show the battery level. I also wrote a detailed README so it’s clear and easy to understand.
What’s your take on this? Any ideas for how to improve the UX further? @thinhx
Man, I don’t want to disappoint you, but I don’t think we should add more LED logic to the LED system just to indicate battery level. Honestly, I’m not sure why an always-on device really needs this feature.
So next, please create an issue, reference your implementation in that issue, and seek feedback from the team and the community. I’d be happy to merge it if the team or our community finds the feature valuable.
@skywinder
Agree. It makes no sense. No one will remember the number of blinks and their relation to battery levels.
/ close
> > 3 days without updates. > please seek feedback from the community/team, then feel free to reopen the ticket.