timofurrer / w1thermsensor

A Python package and CLI tool to work with w1 temperature sensors like DS1822, DS18S20 & DS18B20 on the Raspberry Pi, Beagle Bone and other devices.
MIT License
493 stars 113 forks source link

Wiring Diagram suggestion #65

Closed jesse1d closed 4 years ago

jesse1d commented 4 years ago

The wiring diagram appears to show a pull down resistor, connecting the signal line to ground. This should be, IMHO, a pull-up resistor connected to the supply line. ...but thanks for publishing a brilliant library that makes a very complex interface really easy to implement. Dave

t7ko commented 4 years ago

Yes, now that I looked at it, all other resources show it as pull up... And I have already soldered and packaged it the other way following the diagram :(

apocalipsys commented 4 years ago

so, we have to connect like this?

Raspi VCC (3V3) Pin 1 ----------------------------- VCC DS18B20
                                                             R1 = 4k7 ...10k 
                                                             |
                                                             |

Raspi GPIO 4 Pin 7 ----------------------------- Data DS18B20

Raspi GND Pin 6 ----------------------------- GND DS18B20

timofurrer commented 4 years ago

The README should be fixed now. Thanks for reporting and sorry for the troubles it may has caused you. :tada: