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

Modified Resistors to 0603 #2

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

mcmadhatter
Copy link

Hi

I have modified the resistors to be 0603, and moved them all to the same side of the board, mostly to facilitate making the assembly slightly easier. PCBs are on order for jlcpcb.

😃

Modified Resistors to 0603, and moved them to the same side, so as to make assembly slightly easier.
@chilipeppr
Copy link
Owner

You had to do a decent amount of work now that I'm looking at this

image

image

It looks like you put the Step LED and Power LED signal on the other side of the board
image

But left the Fan LED indicator where it was. I gotcha. You may find that you don't like not seeing those indicators when the board is inside the 3D printed shell. I put them on the side where they were facing up during debugging. However, once the touch sensor lid is on you can't see them directly anyway, but you can see the red glow through the 3D printed shell, so it's probably fine.

Looks like you lost some of the labels during the process.
image

@mcmadhatter
Copy link
Author

I’ll recheck the labels and get them added back . I am a bit worried about one of those vias now i see the renders, i might have to manual rework my ordered pcbs if it is shorting to ground. I tried to get the final led on the other side as well but it was just a bit tight in the end.

@chilipeppr
Copy link
Owner

Yeah, that via on the 1MOhm resistor does appear a bit close in the render. It seems ok in Eagle, but that could end up with bridging during soldering.
image

Merging to Fork from Master
Moved the vias further away from the top side resistors. Also covered vias with solder mask to minimise risk of shorts whilst soldering
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.

2 participants