pimpimmi / LibreAlarm

https://plus.google.com/communities/118306966582165178782
GNU General Public License v3.0
53 stars 48 forks source link

xDrip-plus support and more #18

Closed jamorham closed 8 years ago

jamorham commented 8 years ago

Here is the current version I am using with xDrip+ It works really well in my tests so far.

What this does:

xDrip+ uses the raw value from the sensor and so I have extended the protocol to include this data as well as battery level and root status.

I consider the original LibreAlarm code to be pretty elegant. My new code for handling the root features is much less elegant.

One reason for this is that, for example, the NFC subsystem is inconsistent in processing requests and its timings and the implemented code flow is the most reliable method I found interacting with it. The root power saving features were added with a rapid prototyping phase where I had to experiment to find what would actually work.

Another example is that when the synaptics touchscreen driver is disabled at the kernel level this upsets once service which we then have to restart to avoid runaway cpu usage.

Please let me know what you think of these changes?

Thanks. Jon

pimpimmi commented 8 years ago

Had a few comments. Nothing major. Could be a bit cleaned up in WearActivity.

Can't wait to try it =)