feat: make a top and bottom holder with different distances from the mop handle

build.py

@@ -1,12 +1,11 @@
 # pyright: reportUnusedCallResult=false, reportWildcardImportFromLibrary=false
 from build123d import *
 
-center_thickness = 10
+def build_mop_scraper_holder(center_thickness: float):
+    mop_diameter = 23.4
+    mop_arc_size = 230
 
-mop_diameter = 23.7
+    scraper_diameter = 20.3
-mop_arc_size = 220
-
-scraper_diameter = 20.55
     scraper_arc_size = mop_arc_size
 
     arms_thickness = 3
@@ -15,7 +14,7 @@ clamp_length = 30
 
     with BuildPart() as mop_scraper_holder:
         with BuildSketch() as profile:
-        with BuildLine() as connecting_outline:
+            with BuildLine():
                 arc_s = 170
                 mop_arc = CenterArc((0, (mop_diameter + center_thickness) / -2), mop_diameter / 2, 90 - (arc_s / 2), arc_s)
                 scraper_arc = CenterArc((0, (scraper_diameter + center_thickness) / 2), scraper_diameter / 2, (arc_s / 2) - 90, -arc_s)
@@ -43,5 +42,12 @@ with BuildPart() as mop_scraper_holder:
         with Locations(Plane((screw_pos.X, screw_pos.Y, 0)).rotated((-90, 0, 0))):
             CounterBoreHole(3 / 2, 5.6 / 2, 3.2)
 
-assert mop_scraper_holder.part is not None
+    return mop_scraper_holder
-export_step(mop_scraper_holder.part, "mop_scraper_holder.step")
+
+mop_scraper_holder_top = build_mop_scraper_holder(15)
+assert mop_scraper_holder_top.part is not None
+export_step(mop_scraper_holder_top.part, "mop_scraper_holder.step")
+
+mop_scraper_holder_bottom = build_mop_scraper_holder(15)
+assert mop_scraper_holder_bottom.part is not None
+export_step(mop_scraper_holder_bottom.part, "mop_scraper_holder.step")