Page MenuHomeAleph Objects Inc

Add grounding for bed harness internal EL-HR0155
Closed, ResolvedPublic

Description

The internal bed harness needs to have another ground and a 6 pin connector to accept the changes made in T4903

Event Timeline

west created this task.Jan 2 2019, 4:24 PM
kent added a subscriber: kent.Jan 3 2019, 11:51 AM

Reviewed and merged.

kent closed this task as Resolved.Jan 3 2019, 11:51 AM
logan reopened this task as Open.Jan 24 2019, 4:18 PM
logan added a subscriber: logan.

This is incorrect on master branch, old rev of drawing is there

kent added a comment.Jan 25 2019, 8:35 AM

With the master checked out you can see this:

$ git log production_parts/electronics/internal_harness/EL-HR0155_CB_Bed_harness.odg
commit 5ddc8381ab7821462512acf1f0d042b014825cb4
Author: west <west@alephobjects.com>
Date:   Mon Jan 7 11:01:14 2019 -0700

    Add ferrites and heat shrink tubing to cb bed harness el-hr0155 and cb extruder sensor harness el-hr0153

commit 601173cf76447595cdfc3e23e7f8b55b08df3f73
Author: west <west@alephobjects.com>
Date:   Wed Jan 2 16:34:53 2019 -0700

    Add ground path to internal bed harness el-hr0155 T5059

and its listed as a merged change here rTAZQUIVER470ddff60d3a2daa0129c59fb8388972771d55b6 so it looks like when the changes to add the ferrites and heat shrink tubing were done, they didn't include the changes for this ticket and whoever merged in the changes for the ferrites and heatshink didn't catch it. This means we made two different rev A drawings.

Anyway, we should just fix it.

kent added a comment.Jan 25 2019, 8:52 AM

Ok I added the ground wire to the ferrite drawing with 53404d47905d4285e73430d456c8ed3988b2dfa2 this is ready to review @Steven

Steven closed this task as Resolved.Jan 25 2019, 9:39 AM

@kent this is reviewed and now merged in.