Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ancora_sensors: opensource libsensors #8

Open
wants to merge 3 commits into
base: cm-12.0
Choose a base branch
from
Open

ancora_sensors: opensource libsensors #8

wants to merge 3 commits into from

Conversation

doadin
Copy link

@doadin doadin commented Feb 11, 2015

No description provided.

this libakm doesnt spam logcat when sensor is active. In my opinion this
solves the last issue with switching to opensource sensor code for the
other parts.
@doadin
Copy link
Author

doadin commented Jun 4, 2015

The stock libakm was spaming logcat when active so this new lib eliminates that to avoid perfomance issues. In my testing there was a 100% repeatable performance difference with and without the akm sensor active in the same app. And with that in my testing with this new libakm the performance in the same app is equal to if not really close to the perfomance of the test with the stock libakm disabled. With this new libakm brings imo the opensourceing to a close. It would be nice to opensource libakm too but we seem to be at a stall everyones got there own thing and not much has changed on this front but this gives us a lot better performing sensors with no additional cost so close enough to done imo.

This reverts commit fe939d8.
@doadin
Copy link
Author

doadin commented Jun 11, 2015

i coulda swore it worked on cm11 but apparently it doesnt on cm12. Maybe im just crazy. oh well.

Apparently akm just doesnt work on cm12.1 idk.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant