-
Notifications
You must be signed in to change notification settings - Fork 49
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
Continuous enabled #30
base: master
Are you sure you want to change the base?
Conversation
(allows for local builds..)
(to better test actual library behavior)
also should fix main part of #27 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it looks good for me but don't have chip to test, can buy locally @ladyada
thanks Jan for the review! i have not looked into the failing Travis CI logs - i have spliced the whitespace-only things into a separate (earlier) pull-request (#28) so that it is easier to see what is changed... i also have another pull-request in the making for some interruppt/threshold things: extended interrupt tests & small api extension |
thanks @hoffmannjan for the travis fix. |
i have cleaned the library.properties conflict.. |
let user decide if sensor should run continuously or only on interaction.
(default: maintains old behavior = enable only on interaction)
so for the new 'continuously on mode' you have to enable it after the begin call:
should be fixing most things requested in #12
and all basics of #27