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

Android light sensor #292

Merged
merged 6 commits into from
Mar 16, 2017
Merged

Android light sensor #292

merged 6 commits into from
Mar 16, 2017

Conversation

sumitmadhwani
Copy link
Contributor

Android light sensor api to measure the ambient light level(illumination) in lx.

Can be merged.

screenshot_2017-02-24-00-49-28-434_org test plyerlight
screenshot_2017-02-24-00-49-45-054_org test plyerlight

Tested on:
Device - Xiaomi Mi 4i
Toolchain - old and new both
Android sdk - 24
Android api - 19

@dessant dessant merged commit f147d71 into kivy:master Mar 16, 2017
@dessant
Copy link
Contributor

dessant commented Mar 16, 2017

Hopefully ios will have an api for it in the future, i see UWP already has:
https://docs.microsoft.com/en-us/windows/uwp/devices-sensors/use-the-light-sensor

@sumitmadhwani
Copy link
Contributor Author

Thanks @dessant .

@sumitmadhwani sumitmadhwani deleted the android_light branch March 16, 2017 20:29
@dessant
Copy link
Contributor

dessant commented Mar 17, 2017

@Malverick, the README table was messed up. 😛

@sumitmadhwani
Copy link
Contributor Author

Sorry @dessant. It was messed up in my editor too so I edited it but after pushing the changes it again got messed up which I forgot to notice. Sorry. I will create a new pr after fixing it.

@sumitmadhwani
Copy link
Contributor Author

I think this is happening when I directly commiting from github. Right now also when I am directly trying to edit the README file, there is nothing to fix. It's absolutely fine.
I will fix it in my system and then push the changes within an hour. I don't have access to my laptop right now.

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.

2 participants