From 33c44476fbfbc3fea40eed0e1b029dad6d83e54d Mon Sep 17 00:00:00 2001 From: Matthew Fallshaw Date: Wed, 27 May 2020 20:41:39 +1000 Subject: [PATCH] pie_slice can't accept `d` parameter Since `r=10` is supplied as a default parameter in the module argument list, a call supplying a `d` parameter (eg. `pie_slice(d=10,h=10,ang=90);`) will error on the call to `get_radius` on line 1428 with `ERROR: Assertion 'is_undef(d)' failed: "Conflicting or redundant radius/diameter arguments given."`. Since the calls to `get_radius` on lines 1428 and 1429 both supply `dflt=10`, line 1422 can just change to supplying `r=undef`. --- shapes.scad | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/shapes.scad b/shapes.scad index cd3d49f..d80476a 100644 --- a/shapes.scad +++ b/shapes.scad @@ -1419,7 +1419,7 @@ module noop(spin=0, orient=UP) attachable(CENTER,spin,orient, d=0.01) {nil(); ch // pie_slice(ang=60, l=20, d1=50, d2=70); module pie_slice( ang=30, l=undef, - r=10, r1=undef, r2=undef, + r=undef, r1=undef, r2=undef, d=undef, d1=undef, d2=undef, h=undef, center, anchor, spin=0, orient=UP