[Bf-codereview] MDef BVH Support (issue 4529048)

Daniel Salazar - 3Developer.com zanqdo at gmail.com
Mon Sep 26 03:42:33 CEST 2011


Gah, sadly patch is failing in svn

patching file source/blender/editors/armature/meshlaplacian.c
Hunk #1 succeeded at 1106 (offset 19 lines).
Hunk #2 FAILED at 1165.
Hunk #3 succeeded at 1249 (offset 19 lines).
Hunk #4 succeeded at 1763 (offset 19 lines).
Hunk #5 succeeded at 1880 (offset 19 lines).
1 out of 5 hunks FAILED -- saving rejects to file
source/blender/editors/armature/meshlaplacian.c.rej

Daniel Salazar
3Developer.com



On Sun, Sep 11, 2011 at 5:17 PM, Daniel Salazar - 3Developer.com
<zanqdo at gmail.com> wrote:
> i'll develop a test suite for comparison based on this then
>
> cheers
>
> Daniel Salazar
> 3Developer.com
>
>
>
> On Sat, Sep 10, 2011 at 2:10 AM,  <brechtvanlommel at gmail.com> wrote:
>> The function meshdeform_tri_intersect has a big epsilon to ensure no
>> rays slip through (float precision issues). Actually my main concern
>> with this patch is that the BVH epsilon is possibly too low. It should
>> be tested to give the same results on geometry with mixed small/big
>> faces, faces close to and nearly overlapping the cage boundary.
>>
>> http://codereview.appspot.com/4529048/
>> _______________________________________________
>> Bf-codereview mailing list
>> Bf-codereview at blender.org
>> http://lists.blender.org/mailman/listinfo/bf-codereview
>>
>


More information about the Bf-codereview mailing list