Fix #103387: Radius affects curves bounding box #105154

Merged
Hans Goudey merged 1 commits from HooglyBoogly/blender:fix-curves-bounds-radius into blender-v3.5-release 2023-02-27 00:01:13 +01:00

1 Commits

Author SHA1 Message Date
Hans Goudey 6eaaceda80 Fix #103387: Radius affects curves bounding box
buildbot/vexp-code-patch-coordinator Build done. Details
e8f4010611 unified the bounds computation for the new curves
object type and the rest of the curves system used by geometry nodes.
In the process, it made bounds affected by the control point radius.
In theory that makes sense; the bounds are supposed to be the extents
of the visible geometry. But in practice the change wasn't expected,
for a few reasons:
 - The radius has never affected the bounds for the legacy curve type
 - The default radius of legacy curve objects is absurdly large at 1.0m
 - Only the new curve object has visible radius, and only in "strip"
   mode or when rendering with Cycles

Currently the bounds are only used for the "Bounding Box" geometry node
and the panel in the 3D viewport sidebar, so there isn't any incentive
to choose less intuitive behavior yet.

Long term, the _correct_ behavior is probably to include the radius in
the bounds, but this commit postpones that change to when it works
better with the rest of the curves system.
2023-02-23 15:18:25 -05:00