But according to scene.cpp
the light manager is updated before the integrator?
If light manager is updated after the integrator I agree it would be better to remove the other one. But yes if…
Usually description
is used here, but looks rather lengthy if there is already description for RNA properties, so I use name
instead.
I find this get_unique
concept quite strange, it is here because gizmos are allowed to have multiple properties, and if there are multiple I don't know which one just got edited, and could it be that only one of them has undo.
I find it a bit confusing that there are two groups of getter/setters. The new transform_set_fn
could modify the value, and I don't want to drop the const
specifier for value_set_fn
. Or could there be better function names?
@slwk1d thank you for your feedback.
- Point light theoretically should have a radius of 0, so I don't see a problem of a default value of 0. But it seems a bug that the move gizmo is drawn over…
The pull request has been merged as a3e6cb5dab