Skip to content

Schematic drawing instantiate a common cathode RGB #226

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.

Sign up for GitHub

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

Jump to bottom
Closed
mattinew opened this issue May 11, 2022 · 2 comments
Closed

Schematic drawing instantiate a common cathode RGB #226

mattinew opened this issue May 11, 2022 · 2 comments
Assignees
@jhansson-ard
Labels
community Bugs and fixes suggested by the community fix/update A small fix or update Tutorial

Comments

@mattinew
Copy link

In built-in-examples/communication/ReadASCIIString schematic drawing a common cathode RGB LED is instantiated, while both in Hardware Required and in Code sections a common anode is intended.
@marqdevx
Copy link
Member

Thanks @mattinew for spotting the issue!
We are on it ✏️

@marqdevx marqdevx added fix/update A small fix or update Tutorial community Bugs and fixes suggested by the community labels May 18, 2022
@marqdevx marqdevx mentioned this issue May 20, 2022
Merged
@marqdevx
Copy link
Member

Fix merged
Thanks for the report! @mattinew 😄

Serjlee pushed a commit that referenced this issue Dec 6, 2022
@marqdevx @aliphys
@Fr4ng @BenjaminDannegard @jcarolinares @canchebagur
PLC IDE Tutorials [PC-1041, PC-1003] (#219)
b9f4d72 
* PLC IDE setup guide [PC-974] (#172)

Co-authored-by: Ali Jahangiri <[email protected]>
Co-authored-by: Fr4ng <[email protected]>
Co-authored-by: BenjaminDannegård <[email protected]>

* Fix naming

* PLC IDE: Programming with IEC standard languages introdution [PC-975] (#174)

Co-authored-by: BenjaminDannegard <[email protected]>
Co-authored-by: Fr4ng <[email protected]>

* Move setup tutorial to correct folder

* Fix and align metadata

* Create Pin Mapping for PMC on the Arduino PLC IDE [PC-1002] (#189)

Co-authored-by: Julián Caro Linares <[email protected]>

* Empty-Commit

* PLC IDE + PMC, Pin Map tutorial: Small tweaking [PC-1050] (#225)

* Small tweaking

* Minor rewording

* Pin Map bold

* Tweak explanation

* map table bold

* Update content/hardware/05.pro-solutions/solutions-and-kits/portenta-machine-control/tutorials/plc-ide-pin-mapping/content.md

Co-authored-by: Julián Caro Linares <[email protected]>

* Re-order, rename headings

* Title case

* Update content/hardware/05.pro-solutions/solutions-and-kits/portenta-machine-control/tutorials/plc-ide-pin-mapping/content.md

Co-authored-by: Julián Caro Linares <[email protected]>

* Requirements rewording

Co-authored-by: Julián Caro Linares <[email protected]>

* Add symlinks to the software page

* Update license and PLC IDE links

* Add 2 executables info

* PLC-IDE CAN set up [PC-1000] (#227)

* First commit

* Lint and remove tags

* Fix typo

* Apply suggestions from code review

Co-authored-by: Julián Caro Linares <[email protected]>

* Fix typo, title case and next-step paragraph

Co-authored-by: marqdevx <[email protected]>

* Add paragraph from feedback

* Hide settings list

Co-authored-by: Julián Caro Linares <[email protected]>

* PLC-IDE ModBus set up [PC-1001] (#215)

* First commit

* Add screenshot

* MD fixes

* Lint fix and added RTU naming

* Lint fix

* PLC IDE + PMC ModBus: tutorial fix (#226)

* Added ModBus TCP

* Add modbus node configration

* MD fixes

* assets fix

* Empty-Commit

* Fix overview

* Add TCP and move Parametrization section

* Add ethernet note

* ModBus capitalization

* Add software metadata

Co-authored-by: canchebagur <[email protected]>

* Add symlinks

Co-authored-by: Ali Jahangiri <[email protected]>
Co-authored-by: Fr4ng <[email protected]>
Co-authored-by: BenjaminDannegård <[email protected]>
Co-authored-by: Julián Caro Linares <[email protected]>
Co-authored-by: canchebagur <[email protected]>
marqdevx added a commit that referenced this issue Dec 6, 2022
@ArduinoBot @marqdevx
@sebromero @aliphys @Fr4ng @BenjaminDannegard @jcarolinares @canchebagur
Arduino PLC IDE - Launch (Tutorials + Software Page) (#713)
89d971c 
* Software: Add PLC-IDE page [PC-1038] (#209)

* Update paragraph (#228)

Co-authored-by: sebromero <[email protected]>

* PLC IDE Tutorials [PC-1041, PC-1003] (#219)

* PLC IDE setup guide [PC-974] (#172)

Co-authored-by: Ali Jahangiri <[email protected]>
Co-authored-by: Fr4ng <[email protected]>
Co-authored-by: BenjaminDannegård <[email protected]>

* Fix naming

* PLC IDE: Programming with IEC standard languages introdution [PC-975] (#174)

Co-authored-by: BenjaminDannegard <[email protected]>
Co-authored-by: Fr4ng <[email protected]>

* Move setup tutorial to correct folder

* Fix and align metadata

* Create Pin Mapping for PMC on the Arduino PLC IDE [PC-1002] (#189)

Co-authored-by: Julián Caro Linares <[email protected]>

* Empty-Commit

* PLC IDE + PMC, Pin Map tutorial: Small tweaking [PC-1050] (#225)

* Small tweaking

* Minor rewording

* Pin Map bold

* Tweak explanation

* map table bold

* Update content/hardware/05.pro-solutions/solutions-and-kits/portenta-machine-control/tutorials/plc-ide-pin-mapping/content.md

Co-authored-by: Julián Caro Linares <[email protected]>

* Re-order, rename headings

* Title case

* Update content/hardware/05.pro-solutions/solutions-and-kits/portenta-machine-control/tutorials/plc-ide-pin-mapping/content.md

Co-authored-by: Julián Caro Linares <[email protected]>

* Requirements rewording

Co-authored-by: Julián Caro Linares <[email protected]>

* Add symlinks to the software page

* Update license and PLC IDE links

* Add 2 executables info

* PLC-IDE CAN set up [PC-1000] (#227)

* First commit

* Lint and remove tags

* Fix typo

* Apply suggestions from code review

Co-authored-by: Julián Caro Linares <[email protected]>

* Fix typo, title case and next-step paragraph

Co-authored-by: marqdevx <[email protected]>

* Add paragraph from feedback

* Hide settings list

Co-authored-by: Julián Caro Linares <[email protected]>

* PLC-IDE ModBus set up [PC-1001] (#215)

* First commit

* Add screenshot

* MD fixes

* Lint fix and added RTU naming

* Lint fix

* PLC IDE + PMC ModBus: tutorial fix (#226)

* Added ModBus TCP

* Add modbus node configration

* MD fixes

* assets fix

* Empty-Commit

* Fix overview

* Add TCP and move Parametrization section

* Add ethernet note

* ModBus capitalization

* Add software metadata

Co-authored-by: canchebagur <[email protected]>

* Add symlinks

Co-authored-by: Ali Jahangiri <[email protected]>
Co-authored-by: Fr4ng <[email protected]>
Co-authored-by: BenjaminDannegård <[email protected]>
Co-authored-by: Julián Caro Linares <[email protected]>
Co-authored-by: canchebagur <[email protected]>

* PLC IDE launch, fixes (#229)

* Fix software metadata

* Update CAN tutorial

* Fix ModBus tutorial

Co-authored-by: Pablo Marquínez Ferrándiz <[email protected]>
Co-authored-by: Sebastian Romero <[email protected]>
Co-authored-by: Ali Jahangiri <[email protected]>
Co-authored-by: Fr4ng <[email protected]>
Co-authored-by: BenjaminDannegård <[email protected]>
Co-authored-by: Julián Caro Linares <[email protected]>
Co-authored-by: canchebagur <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jhansson-ard jhansson-ard

Labels
community Bugs and fixes suggested by the community fix/update A small fix or update Tutorial
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@marqdevx @mattinew @jhansson-ard