Update tube_tipper.scad #1
Conversation
tube_tipper.scad
Outdated
| 0.25 * tube_spacing]) | ||
| rotate([0, 71, 0]) | ||
| cylinder(d = 0.5 * tube_diameter, h = 2 * wall_thickness); | ||
| cylinder(d = 0.5 * (tube_diameter + tube_z_stretch), h = 2 * wall_thickness); |
There was a problem hiding this comment.
I don't think this works, it will not introduce the right kind of stretch if you add it to the diameter. Better to add a resize([0, 0, tube_diameter + tube_z_stretch]) above the rotate instead.
There was a problem hiding this comment.
Made this change, in the next commit
tube_tipper.scad
Outdated
| translate([-wall_thickness, | ||
| (y - (number_tubes+1)/2) * (tube_diameter + tube_spacing), | ||
| -0.5 * (tube_spacing + tube_diameter) + (0.5 + x) * (tube_diameter + tube_z_stretch + tube_spacing)]) | ||
| -0.5 * (tube_spacing + 0.75 * tube_diameter) + (0.5 + x) * (tube_diameter + tube_z_stretch + tube_spacing)]) |
There was a problem hiding this comment.
it seems this translate does not yet move the bottom row down more, wasn't this already part of the last printing run?
sidenote: I think for the 20degree angle to keep working you should only move the bottom holes lower, the top ones must stay where they are, right?
There was a problem hiding this comment.
Moving the bottom row down was part of the last test, I think I need to move it a little back up so that the bottom of the hole is sitting right at the base.
On the sidenote: yes, that is how angles work, fixing now.
Fixed several of the comments from the pull request (mostly z stretch related). Still need to fix the height of the top row.
I think I got the top row back into the right spot, will do a small test with the 5 tube
@jen-reeve
easiest to trouble shoot in a pull request since additional commits you make to your branch will just automatically become part of this pull request
i'll leave in line comments below