From cc1b210f89ab95676e4c324c75f64dcf92df2bda Mon Sep 17 00:00:00 2001
From: daniel watson <ozzloy@gmail.com>
Date: Sat, 4 Jun 2016 15:17:40 -0700
Subject: [PATCH] bring in height to sonar binder clip holder

---
 .../sonar-binder-clip-holder-data.scad        | 54 +++++++++++++------
 3d-printables/sonar-binder-clip-holder.scad   |  6 ++-
 2 files changed, 42 insertions(+), 18 deletions(-)

diff --git a/3d-printables/sonar-binder-clip-holder-data.scad b/3d-printables/sonar-binder-clip-holder-data.scad
index f944906..94ba324 100644
--- a/3d-printables/sonar-binder-clip-holder-data.scad
+++ b/3d-printables/sonar-binder-clip-holder-data.scad
@@ -36,6 +36,8 @@ screw_radius_measured = 2.8;
 screw_radius_print_fudge = 0.3;
 screw_radius = screw_radius_measured + screw_radius_print_fudge;
 
+sonar_binder_clip_holder_height = 2;
+
 module sonar_sensors_2d(sonar_radius,
                         between_sonar_centers,
                         between_sonar_centers_variation) {
@@ -57,10 +59,10 @@ module sonar_holder_outline_2d(sonar_radius,
                -holder_radius]) {
       square(holder_diameter); } } }
 
-module sonar_holder(sonar_radius,
-                    between_sonar_centers,
-                    between_sonar_centers_variation,
-                    wall_thickness) {
+module sonar_holder_2d(sonar_radius,
+                       between_sonar_centers,
+                       between_sonar_centers_variation,
+                       wall_thickness) {
   difference() {
     sonar_holder_outline_2d(sonar_radius,
                             between_sonar_centers,
@@ -69,29 +71,49 @@ module sonar_holder(sonar_radius,
                      between_sonar_centers,
                      between_sonar_centers_variation); } }
 
-module binder_clip_holder(binder_clip_holder_length,
-                          screw_radius,
-                          holder_radius){
+module binder_clip_holder_2d(binder_clip_holder_length,
+                             screw_radius,
+                             holder_radius){
   difference() {
   square([binder_clip_holder_length, holder_radius * 2]);
   translate([binder_clip_holder_length - holder_radius, holder_radius]) {
     circle(screw_radius); } } }
 
-module sonar_binder_clip_holder(sonar_radius,
-                                between_sonar_centers,
-                                between_sonar_centers_variation,
-                                wall_thickness) {
-  sonar_holder(sonar_radius,
-               between_sonar_centers,
-               between_sonar_centers_variation,
-               wall_thickness);
+module sonar_binder_clip_holder_2d(sonar_radius,
+                                   between_sonar_centers,
+                                   between_sonar_centers_variation,
+                                   wall_thickness) {
+  sonar_holder_2d(sonar_radius,
+                  between_sonar_centers,
+                  between_sonar_centers_variation,
+                  wall_thickness);
   holder_radius = sonar_radius + wall_thickness;
   binder_clip_holder_length = 2 * holder_radius;
   translate([between_sonar_centers + holder_radius, -holder_radius]){
-    binder_clip_holder(binder_clip_holder_length,
+    binder_clip_holder_2d(binder_clip_holder_length,
                        screw_radius,
                        holder_radius); } }
 
+module sonar_binder_clip_holder(sonar_radius,
+                                between_sonar_centers,
+                                between_sonar_centers_variation,
+                                wall_thickness,
+                                sonar_binder_clip_holder_height) {
+  linear_extrude(height = sonar_binder_clip_holder_height) {
+    sonar_binder_clip_holder_2d(sonar_radius,
+                                between_sonar_centers,
+                                between_sonar_centers_variation,
+                                wall_thickness); } }
+
+module binder_clip_holder(binder_clip_holder_length,
+                          screw_radius,
+                          holder_radius,
+                          binder_clip_holder_height) {
+     linear_extrude(height = binder_clip_holder_height) {
+          binder_clip_holder_2d(binder_clip_holder_length,
+                                screw_radius,
+                                holder_radius); } }
+
 /*
   This file is part of challenge-bot.
 
diff --git a/3d-printables/sonar-binder-clip-holder.scad b/3d-printables/sonar-binder-clip-holder.scad
index bd79585..387334e 100644
--- a/3d-printables/sonar-binder-clip-holder.scad
+++ b/3d-printables/sonar-binder-clip-holder.scad
@@ -21,14 +21,16 @@ include <sonar-binder-clip-holder-data.scad>
 sonar_binder_clip_holder(sonar_radius,
                          between_sonar_centers,
                          between_sonar_centers_variation,
-                         wall_thickness);
+                         wall_thickness,
+                         sonar_binder_clip_holder_height);
 
 holder_radius = sonar_radius + wall_thickness;
 binder_clip_holder_length = 2 * holder_radius;
 translate([0, holder_radius + 2]) {
   binder_clip_holder(binder_clip_holder_length,
                      screw_radius,
-                     holder_radius); }
+                     holder_radius,
+                     sonar_binder_clip_holder_height); }
 
 /*
   This file is part of challenge-bot.
-- 
2.30.2