Skip to content

Incorrect Diagram https://www.arduino.cc/en/Tutorial/ReadASCIIString #4260

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

Closed
vgatesr opened this issue Dec 5, 2015 · 7 comments
Closed
Assignees
Labels
Component: Documentation Related to Arduino's documentation content feature request A request to make an enhancement (not a bug fix)

Comments

@vgatesr
Copy link

vgatesr commented Dec 5, 2015

Your instruction at https://www.arduino.cc/en/Tutorial/ReadASCIIString show connecting the anode to +5 v and it does not work. Move the connection to GND and it does.
This one works (https://www.bing.com/images/search?q=rgb+led+arduino&view=detailv2&qpvt=rgb+led+arduino&id=72BAA7D024767F034FD516F082019FAC21C64EA6&selectedIndex=0&ccid=LN28wCQI&simid=608002808251027198&thid=OIP.M2cddbcc024082add7d2374cdadb8aeb6H0&ajaxhist=0)
Yours does not.

@matthijskooijman
Copy link
Collaborator

Uh, but the anode is supposed to get the positive voltage, and the cathode the negative one. From what you write, I suspect that you have a common cathode RGB led (one cathode, three anodes), which indeed needs to be connected differently. Since that's a different part, I think the instructions are not incorrect, but perhaps mentioning the wiring needed for a common cathode led would be a nice addition.

@matthijskooijman matthijskooijman added Component: Documentation Related to Arduino's documentation content feature request A request to make an enhancement (not a bug fix) labels Dec 5, 2015
@vgatesr
Copy link
Author

vgatesr commented Dec 5, 2015

You must be correct. I just used the RGB that was in the Stater Kit
assuming it would be consistent with the Arduino projects. Assumptions.
The starter book does not list the 4 pin RGB or any RGB for that matter. I
think I have noticed that the Ardunio community doesn't do a very good job
of listing component electrical characteristics ie data sheets.
Thanks for the response.
On Dec 5, 2015 2:27 PM, "Matthijs Kooijman" [email protected]
wrote:

Uh, but the anode is supposed to get the positive voltage, and the cathode
the negative one. From what you write, I suspect that you have a common
cathode RGB led (one cathode, three anodes), which indeed needs to be
connected differently. Since that's a different part, I think the
instructions are not incorrect, but perhaps mentioning the wiring needed
for a common cathode led would be a nice addition.


Reply to this email directly or view it on GitHub
#4260 (comment).

@per1234
Copy link
Collaborator

per1234 commented Dec 6, 2015

the Ardunio community doesn't do a very good job of listing component electrical characteristics ie data sheets.

The datasheet for the RGB LED is provided on the Arduino Starter Kit page:
https://www.arduino.cc/documents/datasheets/LED(RGB).pdf
As you can see it states that it's a common cathode.

@vgatesr
Copy link
Author

vgatesr commented Dec 6, 2015

Right again although it does not appear in the book that came with the kit. Thanks, I am learning a lot from this.

Thanks,

Vic

From: per1234 [mailto:[email protected]]
Sent: Saturday, December 05, 2015 4:54 PM
To: arduino/Arduino
Cc: vgatesr
Subject: Re: [Arduino] Incorrect Diagram https://www.arduino.cc/en/Tutorial/ReadASCIIString (#4260)

the Ardunio community doesn't do a very good job of listing component electrical characteristics ie data sheets.

The datasheet for the RGB LED is provided on the Arduino Starter Kit page https://www.arduino.cc/en/Main/ArduinoStarterKit :
https://www.arduino.cc/documents/datasheets/LED(RGB).pdf
As you can see it states that it's a common cathode.


Reply to this email directly or view it on GitHub #4260 (comment) .Image removed by sender.

@agdl
Copy link
Member

agdl commented Dec 14, 2015

Image fixed. Thank you!

@agdl agdl closed this as completed Dec 14, 2015
@leonardoemarq
Copy link

Hello, as Arturo (agdl) mentioned, the image was fixed to make it correspondent with a Common Cathode circuit (thank you Arturo). But the text stills indicating the connections as if you are using a Common Anode RGB LED. So, now the text and the image are inconsistent!

@agdl
Copy link
Member

agdl commented Mar 14, 2016

@leonardoemarq done thank you!

agdl added a commit that referenced this issue Mar 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Documentation Related to Arduino's documentation content feature request A request to make an enhancement (not a bug fix)
Projects
None yet
Development

No branches or pull requests

5 participants