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

this is probably not an issue... #7

Closed
jackfruh opened this issue Jun 24, 2017 · 6 comments
Closed

this is probably not an issue... #7

jackfruh opened this issue Jun 24, 2017 · 6 comments

Comments

@jackfruh
Copy link

I'm using Arduino IDE 1.8.3 on osx.

The static values aren't working
I get an error

DHT11:26: error: 'SimpleDHT' has not been declared
   int err = SimpleDHT::ErrSuccess; 
             ^
DHT11:27: error: 'SimpleDHT' has not been declared

I'm assuming this is more an issue of my compiler than the code? If so any idea how to fix or what I'd search the net for? (I know I can hack it by substituting 0 for SimpleDHT::ErrSuccess, just wondering how to fix that the static class member's not being seen.)

@jackfruh
Copy link
Author

Digging in a little deeper, it looks like version 1.0.2 did not have the statics.
When I looked at the zip file on my machine, it was the 1.0.2 version

1.0.2 is the highest version shown in the Arduino IDE's Library Manager.

Doing a bit of research it seems the library.properties file may need to be updated for this to show up in Arduino IDE Library Manager. I put in a pull request for this.

(Note I am new to this, doing the best I can to find the cause/fix, but I may be way off)

@bgiesbrecht
Copy link

Hi @jackfruh, you can download the latest version from GitHub and manually install it. I just did this myself yesterday. If you run into issues or have any questions, shout. I like this newer version better due to the error codes.

Just in case you need the docs for this: https://www.arduino.cc/en/Guide/Libraries#toc5.

@jackfruhecolab
Copy link

Thanks bgiesbrecht! I did that and it got me a little closer! (getting some sensor checksum errors, ordered new sensors to see if that fixes the issue)

@winlinvip
Copy link
Owner

Maybe I shouldn't use the static field of class, maybe macro is ok.

@winlinvip
Copy link
Owner

I've fixed it, please test it.

@winlinvip
Copy link
Owner

For 1.0.4, please test:

  1. Whether it fixed the build failed bug?
  2. Whether it works for DHT22?

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

No branches or pull requests

4 participants