Sensor-Data-Logger icon indicating copy to clipboard operation
Sensor-Data-Logger copied to clipboard

Implemented a basic json logger

Open jliebers opened this issue 4 years ago • 4 comments

Hi @Steppschuh,

first of all, thank you for your work with this fantastic app. I took a quick look at your thesis and I have to tell that we share the same goal. In my PhD I focus on getting rid of passwords as well. And for that reason I intend to log some spatial movement through an Android Wear Device (Moto 360 in my case), but I need that data logged to file for a post-hoc analysis. Whilst your app already does an excellent job of visualization, connection etc., I am glad to add this missing feature (#8). 🙂 🚀

Please find a pull request for some basic logging functionality that adds a second floating action button to start and stop the logging of the selected sensors to a JSON file. The file then is stored in Android's external storage in a directory resembling the package name. I also added some example data. It is only a very basic implementation, so please do not judge me (I did some Android in the past, but that's quite some years ago), but I think it does the job and it might be better than nothing. Also I tried to comment/document the most important things in the code.

If you have any comments I am open for feedback. I would be very happy if you could review the changes and in case of approval, please push an update to Google Play.

On a side note, I also had some troubles getting Gradle and the wear-build to work. Reason lies within some wear-library on version 1.4.0 being removed from the mirrors. So I had to fetch the files from an unofficial mirror and place them at this location: %appdata%\..\Local\Android\Sdk\extras\m2repository\com\google\android\support\wearable\1.4.0 (Win10). After that I built the app.

Everything was tested on a Sony Xperia XZ2 Compact and a Moto 360 smart watch.

Cheers!

jliebers avatar Feb 03 '21 20:02 jliebers

Hey Jonathan, thanks for your contribution! I'll look into this on the weekend and get back to you.

Regarding the deprecated wearable dependency: Are there breaking changes when updating to a newer version? If not, it might be easier to just migrate to a more recent version (same applies to other dependencies).

Steppschuh avatar Feb 04 '21 09:02 Steppschuh

Hey Stephan,

Regarding the deprecated wearable dependency: Are there breaking changes when updating to a newer version? If not, it might be easier to just migrate to a more recent version (same applies to other dependencies).

I tried to update the libraries but to be honest I am not an expert on Gradle or the Android Build ecosystem. So I naturally ran into lots of problems and gave up after ca. 2 hours of trying. Instead I opted to run the old build as described. But most likely someone with more experience can sort it out easily ... or not (as I cannot know). 😉

Cheers,

Jonathan

jliebers avatar Feb 04 '21 18:02 jliebers

Hey,

thanks for the feedback, I find it valuable and I will incorporate it once I find the time (probably around the weekend). Thanks!

Cheers,

Jonathan

jliebers avatar Feb 10 '21 09:02 jliebers

Hi, thanks for the feedback. I have included all of it and pushed a new commit to my branch. 🙂 If you like, take a look. Thanks in advance.

PS: I have tested the new implementation on a LG LM-K510 with a Moto 360.

jliebers avatar Feb 19 '21 19:02 jliebers