X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=blobdiff_plain;f=powerbank-bike-clamp.scad;h=18b3adbd2f3869056f8f6993e490242df6463304;hb=HEAD;hp=0759bd2de645dea4b1ca728ab4daa0addf522c56;hpb=70024ed624b285f7ad02693f6744f00e77960bb4;p=reprap-play.git

diff --git a/powerbank-bike-clamp.scad b/powerbank-bike-clamp.scad
index 0759bd2..2a755a8 100644
--- a/powerbank-bike-clamp.scad
+++ b/powerbank-bike-clamp.scad
@@ -15,6 +15,7 @@ minor_wall_min = 1;
 
 screw = 5.0 + 0.75;
 screw_head = 8.7 + 1.2;
+screw_head_space_above = 10;
 screw_nut_across = 7.9 + 0.75;
 screw_nut_th = 3.9 + 0.75;
 
@@ -26,6 +27,8 @@ clamp_min_width = 20;
 
 clamp_gap = 2;
 
+corner_rounding_r = 10;
+
 lower_th = 1;
 
 overlap_l = 0.1;
@@ -50,6 +53,9 @@ strap_th = 3;
 strap_w = 5;
 strap_above = 2.5;
 
+strap_around_over = 1.0;
+strap_around_attach = 2.0;
+
 retainer_walls = [18, 30];
 
 bank_profile_scale_bodge = 1.0;
@@ -64,6 +70,10 @@ liner_th = 0.8;
 brace_r = 5;
 brace_len = 50;
 
+straps_y_adj = [ 3.5,
+		 0,
+		 0 ];
+
 // calculated
 
 straps_y = [ -bank_y_sz * 0.25, // these entries are special and used
@@ -93,8 +103,11 @@ bank_bot_y = strap_above + strap_th;
 strap_r = strap_th;
 
 brace_total_len = brace_len + main_th;
-brace_ctrs_y = [ straps_y[0] - (brace_r + strap_w/2),
-		 straps_y[1] + (brace_r + strap_w/2) ];
+brace_ctrs_y_nom = [ straps_y[0] - (brace_r + strap_w/2),
+		     straps_y[1] + (brace_r + strap_w/2) ];
+
+brace_ctrs_y = [ (straps_y + straps_y_adj)[0] + (brace_r + strap_w/2),
+		 (straps_y + straps_y_adj)[1] + (brace_r + strap_w/2) ];
 
 clamp_width_actual = max(clamp_min_width, holder_x_sz);
 
@@ -136,23 +149,28 @@ module TubeClampLeft() { ////toplevel
     TubeClampLeftPlan();
 }
 
-module TubeClampRightPlan(){
+module TubeClampRightPlan(){ ////toplevel
   difference(){
-    union(){
-      rectfromto([ clamp_gap/2,                   bot_y ],
-		 [ clamp_gap/2 + behind_knob_th,  0     ]);
-      intersection(){
-	circle(r= main_r);
-	union(){
-	  rectfromto([0,0],
-		      main_r *  [5,5]);
-	  rectfromto([ clamp_gap/2, main_r*5 ],
-		      main_r * [2,-5]);
+    // It broke at the inside corner, round these a bit
+    offset(r=-corner_rounding_r)
+    offset(r=+corner_rounding_r)
+    difference(){
+      union(){
+	rectfromto([ clamp_gap/2,                   bot_y ],
+		   [ clamp_gap/2 + behind_knob_th,  0     ]);
+	intersection(){
+	  circle(r= main_r); // maybe split off from main_r and increase?
+	  union(){
+	    rectfromto([0,0],
+		       main_r *  [5,5]);
+	    rectfromto([ clamp_gap/2, main_r*5 ],
+		       main_r * [2,-5]);
+	  }
 	}
+	HingeBodyPlan();
       }
-      HingeBodyPlan();
+      TubePlan();
     }
-    TubePlan();
     HingePinPlan();
   }
 }
@@ -168,7 +186,7 @@ module Screws(){
 	      screw_nut_across / cos(30) + bridge_slop*2],
 	     center=true);
 
-    linextr_x_yz(-200,
+    linextr_x_yz(-(main_r + bank_recess_y + screw_head_space_above),
 		 -(clamp_gap/2 + screw_nut_th + screw_head_behind))
       square([screw_head, screw_head + bridge_slop*2],
 	     center=true);
@@ -297,13 +315,28 @@ module PowerBankHolder(){ ////toplevel
 	  }
 	}
 	translate([brace_total_len, 0,0])
-	  linextr_y_xz(brace_ctrs_y[0] - brace_r,
-		       brace_ctrs_y[1] + brace_r)
+	  linextr_y_xz(brace_ctrs_y_nom[0] - brace_r,
+		       brace_ctrs_y_nom[1] + brace_r)
 	  BraceTubePlan();
       }
+
+      for (strap_y = straps_y + straps_y_adj) {
+	translate([0, strap_y, 0]) {
+	  linextr(-holder_x_sz/2,
+		  +holder_x_sz/2){
+	    hull(){
+	      for (dy = [-1,+1] *
+		     (strap_w/2 + strap_around_attach - strap_around_over)) {
+		translate([0, dy, 0])
+		  circle(r=strap_around_over);
+	      }
+	    }
+	  }
+	}
+      }
     }
 
-    for (strap_y = straps_y)
+    for (strap_y = straps_y + straps_y_adj)
       translate([0, strap_y, 0])
 	rotate([0,0,-90])
 	rotate([0,90,0])