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

Move MP3 to prevent Inc3B from shorting against USB connector of Arduino Mega #5

Conversation

blurfl
Copy link
Collaborator

@blurfl blurfl commented Sep 9, 2018

Without this change, one pin of MP3 will short out, making the encoder for the Left motor inoperable. This picture shows a board that was modified by trimming the pin short and insulating it to prevent the short. The changed .brd file makes this handwork unnecessary.
img_1308

img_1309

img_1310

@blurfl
Copy link
Collaborator Author

blurfl commented Sep 9, 2018

Note that this change is only needed on the SMD version, the thru-hole version already has MP3 moved to avoid the issue.

@MaslowCommunityGardenRobot
Copy link
Contributor

Congratulations on the pull request @blurfl

Now we need to decide as a community if we want to integrate these changes. Vote by giving this comment a thumbs up or a thumbs down. Votes are counted in 48 hours. Ties will not be merged.

I'm just a robot, but I love to see people contributing so I'm going vote thumbs up!

@MaslowCommunityGardenRobot
Copy link
Contributor

Time is up and we're ready to merge this pull request. Great work!

@MaslowCommunityGardenRobot MaslowCommunityGardenRobot merged commit 0d1852c into MaslowCommunityGarden:master Sep 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants