[Bf-blender-cvs] [76ebc10794a] master: Increase assert epsilon in versioning for D9551.

Alexander Gavrilov noreply at git.blender.org
Fri Oct 22 16:17:55 CEST 2021


Commit: 76ebc10794a4bb66a1b94339fd8873dc23f1247c
Author: Alexander Gavrilov
Date:   Fri Oct 22 17:17:21 2021 +0300
Branches: master
https://developer.blender.org/rB76ebc10794a4bb66a1b94339fd8873dc23f1247c

Increase assert epsilon in versioning for D9551.

The epsilon was too optimistic. Snapping to hard-coded (0,-1,0) at
singularity should produce max delta 0.0005, but double it to be safe.
This only affects debug builds obviously.

===================================================================

M	source/blender/blenloader/intern/versioning_300.c

===================================================================

diff --git a/source/blender/blenloader/intern/versioning_300.c b/source/blender/blenloader/intern/versioning_300.c
index d5064f411c2..7d93439a44d 100644
--- a/source/blender/blenloader/intern/versioning_300.c
+++ b/source/blender/blenloader/intern/versioning_300.c
@@ -1197,7 +1197,7 @@ static void correct_bone_roll_value(const float head[3],
       /* Recompute roll using legacy code to interpret the old value. */
       legacy_vec_roll_to_mat3_normalized(vec, *r_roll, bone_mat);
       mat3_to_vec_roll(bone_mat, vec2, r_roll);
-      BLI_assert(compare_v3v3(vec, vec2, FLT_EPSILON));
+      BLI_assert(compare_v3v3(vec, vec2, 0.001f));
     }
   }
 }



More information about the Bf-blender-cvs mailing list