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

adds TMP36/LM35 library with an example #35

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

agwatic
Copy link

@agwatic agwatic commented Mar 18, 2014

No description provided.

@alexanderhiam
Copy link
Member

Hey @agwatic, any interest in testing and cleaning up your pull requests? I'd be happy to pull them if they get to a working state.

@agwatic
Copy link
Author

agwatic commented Apr 17, 2015

unfortunately, I don't have the components to test it. Any ways to test it
except buying the components ?
On 17 Apr 2015 03:13, "Alexander Hiam" [email protected] wrote:

Hey @agwatic https://github.com/agwatic, any interest in testing and
cleaning up your pull requests? I'd be happy to pull them if they get to a
working state.


Reply to this email directly or view it on GitHub
#35 (comment).

temp = sensor.readTempC()
if (not temp):
# The sensor reported an error.
print "Error reading the sensor value"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indent lines 23+24 by one more level

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.

3 participants