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

[Clima.HackKit] Fritzing diagram LM35 error #44

Open
patridge opened this issue Sep 27, 2023 · 4 comments
Open

[Clima.HackKit] Fritzing diagram LM35 error #44

patridge opened this issue Sep 27, 2023 · 4 comments
Assignees
Labels
documentation Improvements or additions to documentation Fixed in upcoming release Fixed in upcoming release

Comments

@patridge
Copy link

https://github.com/WildernessLabs/Clima/blob/main/Image_Assets/Clima_Fritzing_Diagram.png

The LM35 is off-by-one and appears to be flipped front-to-back.

image

If I am reading this TI datasheet correctly, the front (flat) side has the left pin as GND.

image

That would mean the component should be flipped over and moved one pin down.

@NevynUK
Copy link

NevynUK commented Oct 2, 2023

I don't think is flipped, off by one, yes; flipped, no.

@doingnz
Copy link
Contributor

doingnz commented Nov 27, 2023

@jorgedevs, if anyone looks at the above and is wondering about flipped/not-flipped, "BOTTOM VIEW" note on the image is key to understanding the orientation. i.e. bottom view is when you look at the package with the legs of the device facing towards you. The Fritz diagram is the view with the legs facing away from you which can create the incorrect impression it needs to be flipped. It helps to add pin numbers. :-)

image

@jorgedevs jorgedevs self-assigned this Nov 27, 2023
@jorgedevs jorgedevs added the documentation Improvements or additions to documentation label Nov 27, 2023
@jorgedevs
Copy link
Contributor

Good catch and diagram corrected, available on develop branch.

@jorgedevs jorgedevs reopened this Nov 27, 2023
@jorgedevs jorgedevs added the Fixed in upcoming release Fixed in upcoming release label Nov 27, 2023
@jorgedevs
Copy link
Contributor

Re-opened, and will close when its available on main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation Fixed in upcoming release Fixed in upcoming release
Projects
None yet
Development

No branches or pull requests

4 participants