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

Change the network of extruder to match 1pc (#203) #204

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

qqqwqk
Copy link

@qqqwqk qqqwqk commented May 22, 2023

#203
before
MCU's S1A is connected to stepper‘s S2A & MCU's S1B to stepper‘s S1A …… by this mistake
3KW4}Z%4ESKPHWN 3 872(B

Change the network of extruder to match 1pc (#203)
after
32~J$FTHNCI% 9QX7G102MQ

Change the network of extruder to match 1pc
@timmit99
Copy link
Contributor

timmit99 commented Aug 8, 2023

this would be a @hartk1213 issues. Does it make sense to make this change?

@hartk1213
Copy link
Contributor

this would be a @hartk1213 issues. Does it make sense to make this change?

no the older way is the way that was giving issues i dont think i updated the 1pc board but i might have just not updated the silkscreen the 2pc one is the correct one even if the silkscreen is incorrect , i could update the silkscreen and push a new version

@qqqwqk
Copy link
Author

qqqwqk commented Aug 8, 2023

this would be a @hartk1213 issues. Does it make sense to make this change?

no the older way is the way that was giving issues i dont think i updated the 1pc board but i might have just not updated the silkscreen the 2pc one is the correct one even if the silkscreen is incorrect , i could update the silkscreen and push a new version

I produced both(1PC and 2PC)
when I replacing it
I had to swap the pins of extruded motor cable for it to work
the better way is use same sch file to design and check to got best compatibility

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.

3 participants