Fix T64681: evaluate curves with generative modifiers and no keys.
Introduce a new function and use it everywhere, including automatic curve deletion checks to guarantee consistency.
This commit is contained in:
parent
e097845a6f
commit
6519982876
Notes:
blender-bot
2023-02-14 08:38:11 +01:00
Referenced by issue #64681, Cycles: Multi step motion blur test failed on all platforms.
|
@ -340,6 +340,7 @@ float evaluate_fcurve_driver(struct PathResolvedRNA *anim_rna,
|
|||
struct FCurve *fcu,
|
||||
struct ChannelDriver *driver_orig,
|
||||
float evaltime);
|
||||
bool BKE_fcurve_is_empty(struct FCurve *fcu);
|
||||
/* evaluate fcurve and store value */
|
||||
float calculate_fcurve(struct PathResolvedRNA *anim_rna, struct FCurve *fcu, float evaltime);
|
||||
|
||||
|
|
|
@ -1894,7 +1894,7 @@ static void animsys_evaluate_fcurves(Depsgraph *depsgraph,
|
|||
continue;
|
||||
}
|
||||
/* Skip empty curves, as if muted. */
|
||||
if (fcu->totvert == 0) {
|
||||
if (BKE_fcurve_is_empty(fcu)) {
|
||||
continue;
|
||||
}
|
||||
PathResolvedRNA anim_rna;
|
||||
|
@ -2009,7 +2009,7 @@ void animsys_evaluate_action_group(PointerRNA *ptr, bAction *act, bActionGroup *
|
|||
/* calculate then execute each curve */
|
||||
for (fcu = agrp->channels.first; (fcu) && (fcu->grp == agrp); fcu = fcu->next) {
|
||||
/* check if this curve should be skipped */
|
||||
if ((fcu->flag & (FCURVE_MUTED | FCURVE_DISABLED)) == 0 && fcu->totvert != 0) {
|
||||
if ((fcu->flag & (FCURVE_MUTED | FCURVE_DISABLED)) == 0 && !BKE_fcurve_is_empty(fcu)) {
|
||||
PathResolvedRNA anim_rna;
|
||||
if (animsys_store_rna_setting(ptr, fcu->rna_path, fcu->array_index, &anim_rna)) {
|
||||
const float curval = calculate_fcurve(&anim_rna, fcu, ctime);
|
||||
|
@ -3105,7 +3105,7 @@ static void nlastrip_evaluate_actionclip(PointerRNA *ptr,
|
|||
if ((fcu->grp) && (fcu->grp->flag & AGRP_MUTED)) {
|
||||
continue;
|
||||
}
|
||||
if (fcu->totvert == 0) {
|
||||
if (BKE_fcurve_is_empty(fcu)) {
|
||||
continue;
|
||||
}
|
||||
|
||||
|
@ -3334,7 +3334,7 @@ static void nla_eval_domain_action(PointerRNA *ptr,
|
|||
if ((fcu->grp) && (fcu->grp->flag & AGRP_MUTED)) {
|
||||
continue;
|
||||
}
|
||||
if (fcu->totvert == 0) {
|
||||
if (BKE_fcurve_is_empty(fcu)) {
|
||||
continue;
|
||||
}
|
||||
|
||||
|
|
|
@ -3070,14 +3070,20 @@ float evaluate_fcurve_driver(PathResolvedRNA *anim_rna,
|
|||
return evaluate_fcurve_ex(fcu, evaltime, cvalue);
|
||||
}
|
||||
|
||||
/* Checks if the curve has valid keys, drivers or modifiers that produce an actual curve. */
|
||||
bool BKE_fcurve_is_empty(FCurve *fcu)
|
||||
{
|
||||
return (fcu->totvert == 0) && (fcu->driver == NULL) &&
|
||||
!list_has_suitable_fmodifier(&fcu->modifiers, 0, FMI_TYPE_GENERATE_CURVE);
|
||||
}
|
||||
|
||||
/* Calculate the value of the given F-Curve at the given frame, and set its curval */
|
||||
float calculate_fcurve(PathResolvedRNA *anim_rna, FCurve *fcu, float evaltime)
|
||||
{
|
||||
/* only calculate + set curval (overriding the existing value) if curve has
|
||||
* any data which warrants this...
|
||||
*/
|
||||
if ((fcu->totvert) || (fcu->driver && !(fcu->driver->flag & DRIVER_FLAG_INVALID)) ||
|
||||
list_has_suitable_fmodifier(&fcu->modifiers, 0, FMI_TYPE_GENERATE_CURVE)) {
|
||||
if (!BKE_fcurve_is_empty(fcu)) {
|
||||
/* calculate and set curval (evaluates driver too if necessary) */
|
||||
float curval;
|
||||
if (fcu->driver) {
|
||||
|
|
|
@ -315,8 +315,7 @@ void clean_fcurve(struct bAnimContext *ac, bAnimListElem *ale, float thresh, boo
|
|||
clear_fcurve_keys(fcu);
|
||||
|
||||
/* check if curve is really unused and if it is, return signal for deletion */
|
||||
if ((list_has_suitable_fmodifier(&fcu->modifiers, 0, FMI_TYPE_GENERATE_CURVE) == 0) &&
|
||||
(fcu->driver == NULL)) {
|
||||
if (BKE_fcurve_is_empty(fcu)) {
|
||||
AnimData *adt = ale->adt;
|
||||
ANIM_fcurve_delete_from_animdata(ac, adt, fcu);
|
||||
ale->key_data = NULL;
|
||||
|
|
|
@ -1557,8 +1557,7 @@ static bool delete_keyframe_fcurve(AnimData *adt, FCurve *fcu, float cfra)
|
|||
delete_fcurve_key(fcu, i, 1);
|
||||
|
||||
/* Only delete curve too if it won't be doing anything anymore */
|
||||
if ((fcu->totvert == 0) &&
|
||||
(list_has_suitable_fmodifier(&fcu->modifiers, 0, FMI_TYPE_GENERATE_CURVE) == 0)) {
|
||||
if (BKE_fcurve_is_empty(fcu)) {
|
||||
ANIM_fcurve_delete_from_animdata(NULL, adt, fcu);
|
||||
}
|
||||
|
||||
|
|
|
@ -83,7 +83,7 @@ void ui_but_anim_flag(uiBut *but, float cfra)
|
|||
if (fcu) {
|
||||
if (!driven) {
|
||||
/* Empty curves are ignored by the animation evaluation system. */
|
||||
if (fcu->totvert == 0) {
|
||||
if (BKE_fcurve_is_empty(fcu)) {
|
||||
return;
|
||||
}
|
||||
|
||||
|
|
|
@ -979,8 +979,7 @@ static bool delete_action_keys(bAnimContext *ac)
|
|||
changed = delete_fcurve_keys(fcu);
|
||||
|
||||
/* Only delete curve too if it won't be doing anything anymore */
|
||||
if ((fcu->totvert == 0) &&
|
||||
(list_has_suitable_fmodifier(&fcu->modifiers, 0, FMI_TYPE_GENERATE_CURVE) == 0)) {
|
||||
if (BKE_fcurve_is_empty(fcu)) {
|
||||
ANIM_fcurve_delete_from_animdata(ac, adt, fcu);
|
||||
ale->key_data = NULL;
|
||||
}
|
||||
|
|
|
@ -1172,9 +1172,7 @@ static bool delete_graph_keys(bAnimContext *ac)
|
|||
}
|
||||
|
||||
/* Only delete curve too if it won't be doing anything anymore */
|
||||
if ((fcu->totvert == 0) &&
|
||||
(list_has_suitable_fmodifier(&fcu->modifiers, 0, FMI_TYPE_GENERATE_CURVE) == 0) &&
|
||||
(fcu->driver == NULL)) {
|
||||
if (BKE_fcurve_is_empty(fcu)) {
|
||||
ANIM_fcurve_delete_from_animdata(ac, adt, fcu);
|
||||
ale->key_data = NULL;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue