AllSpice-Demos/Altium-Demo
AllSpice-Demos/Altium-Demo
12
Fork 1
Files Issues 7 Design Reviews 1 Actions NEW Releases 2 Activity

Add ESD Protection to Endstops #2

Open
AllSpiceAlice wants to merge 5 commits from develop into main
pull from: develop
merge into: AllSpice-Demos:main
Conversation 1 Commits 5 Files Changed 4 +2 -2
4 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

Archimajor.PcbDoc
View File

LOADING
brendan marked this conversation as resolved
RevaReviewa commented 2024-07-28 23:27:03 +00:00
Review
Copy Link

@daniel, why did we remove the debug header?

!thumbnail[](Archimajor.PcbDoc){ diff="AllSpice-Demos/Altium-Demo:c285374952fcea88c33ee99a6badbfa58c18cc50...7eaa84ee12e13afc3adee15ebf0e4f30f55007c4" pr="2" layers="82,81,74,72,71,69,68,66,65,64,63,62,60,59,58,57,33,35,37,1,41,40,2,39,32,38,34" diff-visibility="full" variant="default" view-coords="17.9,3.2,33.8,21.9" aspect-ratio="1.347" } @daniel, why did we remove the debug header?
daniel-allspice commented 2024-07-28 23:28:05 +00:00
Review
Copy Link

It's the same connector as one of the fan drivers, so technicians and end-users were accidentally pluging in debug equipment into the fan drivers and vice-versa. This was to make it accident-proof.

It's the same connector as one of the fan drivers, so technicians and end-users were accidentally pluging in debug equipment into the fan drivers and vice-versa. This was to make it accident-proof.
RevaReviewa commented 2024-07-28 23:28:39 +00:00
Review
Copy Link

That makes sense, thank you.

That makes sense, thank you.
brendan marked this conversation as resolved
daniel-allspice commented 2024-07-28 23:30:24 +00:00
Review
Copy Link

@AllSpiceAlice , can you have someone on your team review the new 5V power trace thickness?

!thumbnail[](Archimajor.PcbDoc){ diff="AllSpice-Demos/Altium-Demo:c285374952fcea88c33ee99a6badbfa58c18cc50...7eaa84ee12e13afc3adee15ebf0e4f30f55007c4" pr="2" layers="82,81,74,72,71,69,68,66,65,64,63,62,60,59,58,57,33,35,37,1,41,40,2,39,32,38,34" diff-visibility="full" variant="default" view-coords="62.1,11.7,82.9,34.8" aspect-ratio="1.347" } @AllSpiceAlice , can you have someone on your team review the new 5V power trace thickness?
AllSpiceAlice commented 2024-07-28 23:31:52 +00:00
Review
Copy Link

Thank you @daniel, this looks great. All the changes were so quick. Thank you for the expedited layout changes! 🚀

Thank you @daniel, this looks great. All the changes were so quick. Thank you for the expedited layout changes! 🚀
brendan marked this conversation as resolved
MikaChanical commented 2024-07-28 23:42:06 +00:00
Review
Copy Link

@AllSpiceAlice, new mounting holes are perfect. 🥇

!thumbnail[](Archimajor.PcbDoc){ diff="AllSpice-Demos/Altium-Demo:c285374952fcea88c33ee99a6badbfa58c18cc50...7eaa84ee12e13afc3adee15ebf0e4f30f55007c4" pr="2" layers="82,81,74,72,71,69,68,66,65,64,63,62,60,59,58,57,33,35,37,1,41,40,2,39,32,38,34" diff-visibility="full" variant="default" view-coords="53.0,0.4,64.8,15.4" aspect-ratio="1.347" } @AllSpiceAlice, new mounting holes are perfect. 🥇
allspice-hermes commented 2024-10-04 17:42:15 +00:00
Review
Copy Link

Can we make sure that they have the right measures? I thought they were smaller @AllSpiceAlice @MikaChanical

Can we make sure that they have the right measures? I thought they were smaller @AllSpiceAlice @MikaChanical
AllSpiceAlice commented 2024-11-26 18:42:21 +00:00
Review
Copy Link

@daniel why was this debug header removed

!thumbnail[](Archimajor.PcbDoc){ diff="AllSpice-Demos/Altium-Demo:c285374952fcea88c33ee99a6badbfa58c18cc50...7eaa84ee12e13afc3adee15ebf0e4f30f55007c4" pr="2" layers="82,81,74,57,33,35,37,1,41,40,2,39,32,38,34" diff-visibility="full" variant="default" view-coords="18.0,3.1,32.0,13.0" aspect-ratio="1.347" } @daniel why was this debug header removed

4
Archimajor.csv
View File

@ -1,8 +1,8 @@
Description,Designator,Footprint,Quantity,MANUFACTURER,MANUFACTURER #
"CAP CER 0603 100nF 50V 10% X7R","C1, C15, C18, C23A, C23B, C23C, C23D, C23E, C36, C47A, C47B, C47C, C47D, C47E, C47F, C47G, C47H, C95, C121A, C121B, C121C, C121D, C121E, C121F, C121G, C121H, C122A, C122B, C122C, C122D, C122E, C122F, C122G, C122H, C199, C202, C203","C0603","37","Kemet","C0603C104K5RACTU"
"CAP CER 0.1UF 6.3V 10% X5R 0201","C2, C60A, C60B, C60C, C60D, C62A, C62B, C62C, C62D, C67, C78, C91, C100, C105, C106, C149A, C149B, C149C, C149D, C149E, C149F, C149G, C149H, C204, C205, C206, C207, C208, C215, C217","C0201 Large Pads","30","Samsung","CL03A104KQ3NNNC"
"CAP CER 0402 TBD, CAP CER 0402 22pF 50V 5% C0G","C3, C151","C0402","2","TBD, AVX","TBD, 04025A220JAT2A"
"CAP CER 0.1UF 6.3V 10% X5R 0201","C2, C60A, C60B, C60C, C60D, C60E, C62A, C62B, C62C, C62D, C62E, C67, C78, C91, C100, C105, C106, C149A, C149B, C149C, C149D, C149E, C149F, C149G, C149H, C204, C205, C206, C207, C208, C215, C217","C0201 Large Pads","32","Samsung","CL03A104KQ3NNNC"
"CAP CER 0402 TBD","C3","C0402","1","TBD","TBD"
"CAP CER 47UF 10V X5R 1210","C4, C5","C1210","2","Murata","GRM32ER61A476KE20L"
"CAP CER 4.7UF 50V X7R 1210","C6, C7, C440","C1210","3","Murata","GRM32ER71H475KA88L"
"Capacitor Non Polarized","C8, C9, C10, C11, C27, C50, C153, C154, C209, C211","C0603","10","","DNI-C0603TBD"

1 Description Designator Footprint Quantity MANUFACTURER MANUFACTURER #
2 CAP CER 0603 100nF 50V 10% X7R C1, C15, C18, C23A, C23B, C23C, C23D, C23E, C36, C47A, C47B, C47C, C47D, C47E, C47F, C47G, C47H, C95, C121A, C121B, C121C, C121D, C121E, C121F, C121G, C121H, C122A, C122B, C122C, C122D, C122E, C122F, C122G, C122H, C199, C202, C203 C0603 37 Kemet C0603C104K5RACTU
3 CAP CER 0.1UF 6.3V 10% X5R 0201 C2, C60A, C60B, C60C, C60D, C62A, C62B, C62C, C62D, C67, C78, C91, C100, C105, C106, C149A, C149B, C149C, C149D, C149E, C149F, C149G, C149H, C204, C205, C206, C207, C208, C215, C217 C2, C60A, C60B, C60C, C60D, C60E, C62A, C62B, C62C, C62D, C62E, C67, C78, C91, C100, C105, C106, C149A, C149B, C149C, C149D, C149E, C149F, C149G, C149H, C204, C205, C206, C207, C208, C215, C217 C0201 Large Pads 30 32 Samsung CL03A104KQ3NNNC
4 CAP CER 0402 TBD, CAP CER 0402 22pF 50V 5% C0G CAP CER 0402 TBD C3, C151 C3 C0402 2 1 TBD, AVX TBD TBD, 04025A220JAT2A TBD
5 CAP CER 47UF 10V X5R 1210 C4, C5 C1210 2 Murata GRM32ER61A476KE20L
6 CAP CER 4.7UF 50V X7R 1210 C6, C7, C440 C1210 3 Murata GRM32ER71H475KA88L
7 Capacitor Non Polarized C8, C9, C10, C11, C27, C50, C153, C154, C209, C211 C0603 10 DNI-C0603TBD
8 CAP CER 10nF 50V X7R 0402 C12A, C12B, C12C, C12D, C12E, C30A, C30B, C30C, C30D, C30E, C40, C43, C71, C77, C84, C86, C157, C158, C159, C160, C171, C214, C216 C0402 23 Murata GRM155R71H103JA88D

EndStops.SchDoc
View File

LOADING
brendan marked this conversation as resolved
daniel-allspice commented 2024-07-28 23:20:05 +00:00
Review
Copy Link


@RevaReviewa, the extra endstop channel has been removed. Thank you!

!thumbnail[](EndStops.SchDoc){ diff="AllSpice-Demos/Altium-Demo:c285374952fcea88c33ee99a6badbfa58c18cc50...7eaa84ee12e13afc3adee15ebf0e4f30f55007c4" pr="2" diff-visibility="full" variant="default" view-coords="4.8,40.4,49.6,60.8" aspect-ratio="1.286" } @RevaReviewa, the extra endstop channel has been removed. Thank you!
❤️ 1
gautam commented 2024-09-05 15:39:46 +00:00
Review
Copy Link

@daniel What's up with this?

!thumbnail[](EndStops.SchDoc){ diff="AllSpice-Demos/Altium-Demo:c285374952fcea88c33ee99a6badbfa58c18cc50...7eaa84ee12e13afc3adee15ebf0e4f30f55007c4" pr="2" diff-visibility="full" variant="default" view-coords="51.6,15.4,93.9,37.3" aspect-ratio="1.286" } @daniel What's up with this?
AllSpiceAlice commented 2024-09-05 21:19:11 +00:00
Review
Copy Link

We wanted to add ESD protection. The users kept frying the boards when connecting cables.

We wanted to add ESD protection. The users kept frying the boards when connecting cables.
AllSpiceAlice commented 2024-12-05 20:07:17 +00:00
Review
Copy Link

@daniel-allspice , why did we remove this channel?

!thumbnail[](EndStops.SchDoc){ diff="AllSpice-Demos/Altium-Demo:c285374952fcea88c33ee99a6badbfa58c18cc50...7eaa84ee12e13afc3adee15ebf0e4f30f55007c4" pr="2" diff-visibility="full" variant="default" view-coords="4.4,37.9,50.7,62.0" aspect-ratio="1.286" } @daniel-allspice , why did we remove this channel?

Fans.SchDoc
View File

LOADING
brendan marked this conversation as resolved
PavelInPurchasing commented 2024-07-28 23:23:42 +00:00
Review
Copy Link

@daniel, can you replace the LED with something from the approved parts list? We're about to remove this LED because it is end-of-life EOL.

!thumbnail[](Fans.SchDoc){ diff="AllSpice-Demos/Altium-Demo:c285374952fcea88c33ee99a6badbfa58c18cc50...7eaa84ee12e13afc3adee15ebf0e4f30f55007c4" pr="2" diff-visibility="full" variant="default" view-coords="21.3,8.6,37.4,29.5" aspect-ratio="1.286" } @daniel, can you replace the LED with something from the approved parts list? We're about to remove this LED because it is end-of-life EOL.
👍 1
daniel-allspice commented 2024-07-28 23:25:04 +00:00
Review
Copy Link

@PavelInPurchasing, I've switched to a standard APL LED. Thank you! 🙏

@PavelInPurchasing, I've switched to a standard APL LED. Thank you! 🙏
AllSpiceAlice commented 2024-09-11 17:30:34 +00:00
Review
Copy Link

@MikaChanical , is this correct

!thumbnail[](Fans.SchDoc){ diff="AllSpice-Demos/Altium-Demo:c285374952fcea88c33ee99a6badbfa58c18cc50...7eaa84ee12e13afc3adee15ebf0e4f30f55007c4" pr="2" diff-visibility="full" variant="default" view-coords="63.0,27.8,77.1,37.7" aspect-ratio="1.286" } @MikaChanical , is this correct