NeuroTechX/eeg-101

Battery indicator

vasyl91 opened this issue · 6 comments

vasyl91 commented

Would you consider implementing battery indicator?

jdpigeon commented

That's a great idea, probably not too much work since the battery information is obtainable through the Muse SDK. We'll keep this addition in mind as we start developing the app again.

vasyl91 commented

Splendid ;) if I may ask, when are you planning to start developing app again?

jdpigeon commented

That's up in the air, but the plan is for development to start sometime before the end of the year. @tmcneely will be the main developer

vasyl91 commented

Ok, jobs done! After many trials and errors (I'm not professional programmer) I managed to add battery indicator to the project I'm working on ;) I also adjusted it to EEG101. That's my little contribution and way to say thank you for sharing your app.

EDIT: everything is few comments below.

  • 🎉1
jdpigeon commented

Amazing! Would you be comfortable making a pull request with your battery indicator? That way we can add it to the app 🎉

vasyl91 commented

Ofc, I'll check some things and will do that this evening ;)

EDIT: Just fixed an error (previously navbar didn't update battery status). Checked it, seems to work just fine.

Screenshots:
Screenshots.zip

EDIT2: Pulled request. You can close this issue ;)

EDIT3: Fixed another error that slowed down whole app. Now works fine (left app connected for the whole night).
Download:
BatteryIndicator.zip