adafruit/Adafruit_Python_DHT

Update Adafruit_Python_DHT library in Pypi

jesusiglesias opened this issue · 27 comments

jesusiglesias commented

Hi,

I have trouble with the execution of a code, which reads data from a DHT11 sensor, when the installation of the library is done with the pip command. My Raspberry Pi is a model 3 and I realized that this version is not included in the code when detecting the platform (common.py file -> get_platform() function; platform_detect.py file -> pi_version() function). For this reason, RPi v3 is not recognized and the library returns to the default value (Beaglebone). The transfer of parameters is then of type String.

Please, the solution is to keep Pypi updated with the latest version of GitHub.

Information:

  • Platform/operating system: Raspberry Pi with Raspbian
  • Python version: python2.7.13
  • Error message you are receiving, including any Python exception traces: TypeError: expected string or buffer
  • List the steps to reproduce the problem below (if possible attach code or commands
    to run): Have a Raspberry Pi v3. Install the library with the pip command. Execute a code where values from DHT11 sensor are read.
  • Author: @tdicola.
cerealcable commented

So the pypi version is from https://github.com/hexdump42/Adafruit_Python_DHT not adafruit, which is confusing at best. I've run into this exact issue and was quite annoyed myself. Not sure if there's something that can be done but it should would be awesome if this was in pypi and up to date.

jesusiglesias commented

This repository (https://github.com/hexdump42/Adafruit_Python_DHT) is a fork from original repository but it is outdated and therefore the version in Pypi also. This is the main problem. Besides, I think that this one was created to upload this library to Pypi and the author forgot it.

ladyada commented

hiya we opened a ticket https://sourceforge.net/p/pypi/support-requests/789/ hopefully they'll let us take it over! until then please install manually :)

  • 👍2
jesusiglesias commented

Thanks for the info. This is good news and I hope that it will be updated in Pypi soon. When it's ready, I'll close this issue.

cerealcable commented

@ladyada Could always package this as Adafruit_DHT and skip Python entirely, it is pypi after all so Python is a little redundant.

FireBelow commented

This really messed me up a couple of times when I got the two repositories confused. I really hope you can take control of the Pypi package

ladyada commented

we cannot, this is a down-side of pypi, there is nothing at all we can do until the owner appears and transfers it to us

cerealcable commented
ladyada commented

we'll take a look at that, but it wont stop people from using the old one. we don't even know where people are getting the idea of installing with pip? cause we dont recommend it :/

cerealcable commented
ladyada commented

ok that makes more sense! we will add this to our upcoming python library sweeps
if you can help location the author of the current pypi registry it would be great :)

FireBelow commented
Sispheor commented

Haha I got screwed by the pypi libs too.

Guys you should just create another package with a new name and reference it on the readme of this repo.

Anyway, the lib works well. Thanks !

tedder commented

Since hexdump42 doesn't have a posted email address and github issues are turned off, I created a pull request to ask them to turn over/share the pypi publishing keys.

PeteLawler commented

I think hexdump42, Mark Rees, might be the same Mark Rees as seen in this corporate bio https://www.corumgroup.com.au/about/management-team - the linkedin profile would seem to match up.

I'll try contacting him today, as he's in my timezone.

  • 👍1
hexdump42 commented

Just saw the email with regards this. The reason why I put it up in pypi a couple of years ago was that Adafruit hadn't and for a project I was working on needed to install via pip. If Adafruit want to keep pypi up-to-date I am happy to pass the pypi project across to them.

  • 👍1
tedder commented

That's great! So this is back in @ladyada's hands.

hexdump42 commented

@ladyada Please let me know the pypi user id you want me to assign owner rights to. There were a number of tweaks I made so a pip install worked correctly. These changes may need to be applied to the Adafruit master before building a new package to upload. Happy to provide help if you need it.

  • ❤️1
ladyada commented

AHHHHMAZING!!

ladyada commented

@hexdump42 we are "adafruit-python" if you move ownership, i'll figure out how to get this repo set up for pypi and 'replace' your code.
thank you :)

FireBelow commented
hexdump42 commented

@ladyada ownership of pypi project transferred

  • 👍4
  • 🎉1
ladyada commented

thanks a bunch @hexdump42 :)

  • 👍2
deanm1278 commented

ok this has been updated!
https://pypi.org/project/Adafruit_Python_DHT/

  • 🎉1
  • ❤️1
FireBelow commented
PeteLawler commented

Excellent community work folks, thanks @hexdump42 @ladyada @deanm1278 !

  • 👍1