Differential D15405

Curves: use consistent default radius for Cycles, Eevee, Set Curve Radius node
ClosedPublic
Actions

Authored by Brecht Van Lommel (brecht) on Jul 8 2022, 3:58 PM.

Details

Reviewers

Hans Goudey (HooglyBoogly)

Maniphest Tasks

T97584: cycles can not start sampling while a new curve object with the default radius in the scene

Commits

rB8159e0a666d3: Curves: use consistent default radius for Cycles, Eevee, Set Curve Radius node

Summary

To avoid Cycles not showing any hair by default, and to avoid very slow render
due to many overlaps with the previous 1 meter default in the node.

Fixes T97584

Diff Detail

Repository

rB Blender

Branch

master

Build Status

Buildable 22912
Build 22912: arc lint + arc unit

Event Timeline

Brecht Van Lommel (brecht) requested review of this revision.Jul 8 2022, 3:58 PM

Brecht Van Lommel (brecht) created this revision.

Harbormaster completed remote builds in B22912: Diff 53437.Jul 8 2022, 3:58 PM

Was just about to look into this and I noticed this patch, thanks!

This revision is now accepted and ready to land.Jul 8 2022, 4:16 PM

Closed by commit rB8159e0a666d3: Curves: use consistent default radius for Cycles, Eevee, Set Curve Radius node (authored by Brecht Van Lommel (brecht)). · Explain WhyJul 8 2022, 4:42 PM

This revision was automatically updated to reflect the committed changes.

Brecht Van Lommel (brecht) added a commit: rB8159e0a666d3: Curves: use consistent default radius for Cycles, Eevee, Set Curve Radius node.

Revision Contents
Changeset List

Path

Size

intern/

cycles/

blender/

curves.cpp

2 lines

source/

blender/

nodes/

geometry/

nodes/

node_geo_set_curve_radius.cc

2 lines

Diff 53437

View Options

intern/cycles/blender/curves.cpp