build123d icon indicating copy to clipboard operation
build123d copied to clipboard

Thicken normal incorrect

Open MatthiasJ1 opened this issue 1 year ago • 2 comments

l1 = JernArc((0,0), (-1,0), 1, 180).edge().reversed()
l2 = JernArc((0,0), (1,0), 2, -90)
s = RadiusArc((0,0), (0,-1), 1)
f = sweep((l1^0)*s, l1) + sweep((l2^0)*s, l2) # Workaround for #622
1
p = thicken(f, 0.4)
2

MatthiasJ1 avatar May 09 '24 22:05 MatthiasJ1

There is a normal_override parameter of thicken to help cope with these situations:

p = thicken(f, 0.4, normal_override=(0, 1, 0))

image

When thicken was developed controlling the normal was found to be difficult so this override was added as the best solution at the time.

gumyr avatar May 10 '24 12:05 gumyr

Shouldn't it just thicken in the face normal?

MatthiasJ1 avatar May 10 '24 18:05 MatthiasJ1