griefith/test
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Depsgraph: Fix dependency cycle when rigid body is involved

Browse Source 
Was introduced by point cache reset on manual edits. Needed to
split evaluation and introduce an explicit init key, which allows
to hook up relations which are "monitoring" manual edits to the
channel.

Noticed while looking into T61190.
This commit is contained in:
Sergey Sharybin
2019-02-06 17:04:29 +01:00
parent 49e07dfdfb
commit fb15dfbddf
4 changed files with 58 additions and 50 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
source/blender/depsgraph/intern/builder/deg_builder_nodes.cc
View File

@@ -743,13 +743,16 @@ void DepsgraphNodeBuilder::build_object_transform(Object *object)
{
OperationNode *op_node;
Object *ob_cow = get_cow_datablock(object);
/* Local transforms (from transform channels - loc/rot/scale + deltas). */
op_node = add_operation_node(&object->id, NodeType::TRANSFORM,
OperationCode::TRANSFORM_LOCAL,
function_bind(BKE_object_eval_local_transform,
_1,
ob_cow));
/* Transform entry operation. */
op_node = add_operation_node(
&object->id, NodeType::TRANSFORM, OperationCode::TRANSFORM_INIT);
op_node->set_as_entry();
/* Local transforms (from transform channels - loc/rot/scale + deltas). */
add_operation_node(&object->id, NodeType::TRANSFORM,
OperationCode::TRANSFORM_LOCAL,
function_bind(BKE_object_eval_local_transform,
_1,
ob_cow));
/* Object parent. */
if (object->parent != NULL) {
add_operation_node(&object->id, NodeType::TRANSFORM,
@@ -762,7 +765,7 @@ void DepsgraphNodeBuilder::build_object_transform(Object *object)
}
/* Rest of transformation update. */
add_operation_node(&object->id, NodeType::TRANSFORM,
OperationCode::TRANSFORM_OBJECT_UBEREVAL,
OperationCode::TRANSFORM_EVAL,
function_bind(BKE_object_eval_uber_transform,
_1,
ob_cow));

79
source/blender/depsgraph/intern/builder/deg_builder_relations.cc
View File

@@ -570,20 +570,19 @@ void DepsgraphRelationBuilder::build_object(Base *base, Object *object)
}
/* Object Transforms */
OperationCode base_op = (object->parent) ? OperationCode::TRANSFORM_PARENT
: OperationCode::TRANSFORM_LOCAL;
: OperationCode::TRANSFORM_LOCAL;
OperationKey base_op_key(&object->id, NodeType::TRANSFORM, base_op);
OperationKey local_transform_key(&object->id,
NodeType::TRANSFORM,
OperationCode::TRANSFORM_LOCAL);
OperationKey parent_transform_key(&object->id,
NodeType::TRANSFORM,
OperationCode::TRANSFORM_PARENT);
OperationKey final_transform_key(&object->id,
NodeType::TRANSFORM,
OperationCode::TRANSFORM_FINAL);
OperationKey ob_ubereval_key(&object->id,
NodeType::TRANSFORM,
OperationCode::TRANSFORM_OBJECT_UBEREVAL);
OperationKey init_transform_key(
&object->id, NodeType::TRANSFORM, OperationCode::TRANSFORM_INIT);
OperationKey local_transform_key(
&object->id, NodeType::TRANSFORM, OperationCode::TRANSFORM_LOCAL);
OperationKey parent_transform_key(
&object->id, NodeType::TRANSFORM, OperationCode::TRANSFORM_PARENT);
OperationKey final_transform_key(
&object->id, NodeType::TRANSFORM, OperationCode::TRANSFORM_FINAL);
OperationKey ob_eval_key(
&object->id, NodeType::TRANSFORM, OperationCode::TRANSFORM_EVAL);
add_relation(init_transform_key, local_transform_key, "Transform Init");
/* Various flags, flushing from bases/collections. */
build_object_flags(base, object);
/* Parenting. */
@@ -635,20 +634,19 @@ void DepsgraphRelationBuilder::build_object(Base *base, Object *object)
/* operation order */
add_relation(base_op_key, constraint_key, "ObBase-> Constraint Stack");
add_relation(constraint_key, final_transform_key, "ObConstraints -> Done");
// XXX
add_relation(constraint_key, ob_ubereval_key, "Temp Ubereval");
add_relation(ob_ubereval_key, final_transform_key, "Temp Ubereval");
add_relation(constraint_key, ob_eval_key, "Eval");
add_relation(ob_eval_key, final_transform_key, "Eval");
}
else {
/* NOTE: Keep an eye here, we skip some relations here to "streamline"
* dependencies and avoid transitive relations which causes overhead.
* But once we get rid of uber eval node this will need reconsideration. */
if (object->rigidbody_object == NULL) {
/* Rigid body will hook up another node inbetween, so skip
/* Rigid body will hook up another node in between, so skip
* relation here to avoid transitive relation. */
add_relation(base_op_key, ob_ubereval_key, "Temp Ubereval");
add_relation(base_op_key, ob_eval_key, "Eval");
}
add_relation(ob_ubereval_key, final_transform_key, "Temp Ubereval");
add_relation(ob_eval_key, final_transform_key, "Eval");
}
/* Animation data */
build_animdata(&object->id);
@@ -661,16 +659,19 @@ void DepsgraphRelationBuilder::build_object(Base *base, Object *object)
/* Proxy object to copy from. */
if (object->proxy_from != NULL) {
build_object(NULL, object->proxy_from);
ComponentKey ob_transform_key(&object->proxy_from->id, NodeType::TRANSFORM);
ComponentKey ob_transform_key(
&object->proxy_from->id, NodeType::TRANSFORM);
ComponentKey proxy_transform_key(&object->id, NodeType::TRANSFORM);
add_relation(ob_transform_key, proxy_transform_key, "Proxy Transform");
}
if (object->proxy_group != NULL) {
build_object(NULL, object->proxy_group);
OperationKey proxy_group_ubereval_key(&object->proxy_group->id,
NodeType::TRANSFORM,
OperationCode::TRANSFORM_OBJECT_UBEREVAL);
add_relation(proxy_group_ubereval_key, final_transform_key, "Proxy Group Transform");
OperationKey proxy_group_eval_key(&object->proxy_group->id,
NodeType::TRANSFORM,
OperationCode::TRANSFORM_EVAL);
add_relation(proxy_group_eval_key,
final_transform_key,
"Proxy Group Transform");
}
/* Object dupligroup. */
if (object->dup_group != NULL) {
@@ -912,7 +913,9 @@ void DepsgraphRelationBuilder::build_object_pointcache(Object *object)
int flag = -1;
if (ptcache_id->type == PTCACHE_TYPE_RIGIDBODY) {
flag = FLAG_TRANSFORM;
ComponentKey transform_key(&object->id, NodeType::TRANSFORM);
OperationKey transform_key(&object->id,
NodeType::TRANSFORM,
OperationCode::TRANSFORM_LOCAL);
add_relation(point_cache_key,
transform_key,
"Point Cache -> Rigid Body");
@@ -934,13 +937,13 @@ void DepsgraphRelationBuilder::build_object_pointcache(Object *object)
}
/* Manual edits to any dependency (or self) should reset the point cache. */
if (!BLI_listbase_is_empty(&ptcache_id_list)) {
OperationKey transform_local_key(&object->id,
NodeType::TRANSFORM,
OperationCode::TRANSFORM_LOCAL);
OperationKey transform_init_key(&object->id,
NodeType::TRANSFORM,
OperationCode::TRANSFORM_INIT);
OperationKey geometry_init_key(&object->id,
NodeType::GEOMETRY,
OperationCode::GEOMETRY_EVAL_INIT);
add_relation(transform_local_key,
add_relation(transform_init_key,
point_cache_key,
"Transform Local -> Point Cache",
RELATION_FLAG_FLUSH_USER_EDIT_ONLY);
@@ -1637,7 +1640,8 @@ void DepsgraphRelationBuilder::build_rigidbody(Scene *scene)
* - passive don't change, so may need to know full transform... */
OperationKey rbo_key(&object->id, NodeType::TRANSFORM, OperationCode::RIGIDBODY_TRANSFORM_COPY);
OperationCode trans_opcode = object->parent ? OperationCode::TRANSFORM_PARENT : OperationCode::TRANSFORM_LOCAL;
OperationCode trans_opcode = object->parent ? OperationCode::TRANSFORM_PARENT
: OperationCode::TRANSFORM_LOCAL;
OperationKey trans_op(&object->id, NodeType::TRANSFORM, trans_opcode);
add_relation(sim_key, rbo_key, "Rigidbody Sim Eval -> RBO Sync");
@@ -1663,14 +1667,13 @@ void DepsgraphRelationBuilder::build_rigidbody(Scene *scene)
add_relation(rbo_key, constraint_key, "RBO Sync -> Ob Constraints");
}
else {
/* Final object transform depends on rigidbody.
*
* NOTE: Currently we consider final here an ubereval node.
* If it is gone we'll need to reconsider relation here. */
OperationKey uber_key(&object->id,
NodeType::TRANSFORM,
OperationCode::TRANSFORM_OBJECT_UBEREVAL);
add_relation(rbo_key, uber_key, "RBO Sync -> Uber (Temp)");
/* Transform evaluation depends on rigidbody. */
OperationKey transform_eval_key(&object->id,
NodeType::TRANSFORM,
OperationCode::TRANSFORM_EVAL);
add_relation(rbo_key,
transform_eval_key,
"RBO Sync -> Transform Eval");
}
/* Needed to get correct base values. */

4
source/blender/depsgraph/intern/node/deg_node_operation.cc
View File

@@ -49,13 +49,13 @@ const char *operationCodeAsString(OperationCode opcode)
/* Object related. */
case OperationCode::OBJECT_BASE_FLAGS: return "OBJECT_BASE_FLAGS";
/* Transform. */
case OperationCode::TRANSFORM_INIT: return "TRANSFORM_INIT";
case OperationCode::TRANSFORM_LOCAL: return "TRANSFORM_LOCAL";
case OperationCode::TRANSFORM_PARENT: return "TRANSFORM_PARENT";
case OperationCode::TRANSFORM_CONSTRAINTS:
return "TRANSFORM_CONSTRAINTS";
case OperationCode::TRANSFORM_FINAL: return "TRANSFORM_FINAL";
case OperationCode::TRANSFORM_OBJECT_UBEREVAL:
return "TRANSFORM_OBJECT_UBEREVAL";
case OperationCode::TRANSFORM_EVAL: return "TRANSFORM_EVAL";
/* Rigid body. */
case OperationCode::RIGIDBODY_REBUILD: return "RIGIDBODY_REBUILD";
case OperationCode::RIGIDBODY_SIM: return "RIGIDBODY_SIM";

8
source/blender/depsgraph/intern/node/deg_node_operation.h
View File

@@ -61,16 +61,18 @@ enum class OperationCode {
OBJECT_BASE_FLAGS,
/* Transform. ----------------------------------------------------------- */
/* Transform entry point - local transforms only */
/* Transform entry point. */
TRANSFORM_INIT,
/* Local transforms only */
TRANSFORM_LOCAL,
/* Parenting */
TRANSFORM_PARENT,
/* Constraints */
TRANSFORM_CONSTRAINTS,
/* Handle object-level updates, mainly proxies hacks and recalc flags. */
TRANSFORM_EVAL,
/* Transform exit point */
TRANSFORM_FINAL,
/* Handle object-level updates, mainly proxies hacks and recalc flags. */
TRANSFORM_OBJECT_UBEREVAL,
/* Rigid body. ---------------------------------------------------------- */
/* Perform Simulation */