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

Fixes readings error #6

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

Conversation

lthiery
Copy link

@lthiery lthiery commented May 3, 2013

I was having a similar error to issue 1 and issue 3

I noticed that the most significant byte was being lost. I changed the data type returned by shiftIn and that appears to resolve the issue (on my end at least).

Made additions to the library:

  •  decoupled requests, read in, and parsing
    
  • temperature coefficients will change with voltage (when instantiating SHT object)
  • stores temperature for humidity conversion rather than re-requesting it

The result runs much faster and lets you program tasks while waiting for SHT1x responses.

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