Change Details

When triangulating meshes, the UV unwrapper was previously using a heuristic to split quads into triangles. If one of the internal angles is greater than 180degrees, ("reflex angle") the heuristic was giving a poor choice of split. Instead of using a special case for quads, this change routes everything through the generic ngon polyfill_beautify method instead. Some corner cases regarding non-manifold geometry are documented in the code. Note that there are two incorrect calls to `param_assert` that are also removed by this differential and replaced by a correct `BLI_assert` instead.

When triangulating meshes, the UV unwrapper was previously using a heuristic to split quads into triangles. If one of the internal angles is greater than 180degrees, ("reflex angle") the heuristic was giving a poor choice of split. Instead of using a special case for quads, this change routes everything through the generic ngon polyfill_beautify method instead.

When triangulating meshes, the UV unwrapper was previously using a heuristic to split quads into triangles. If one of the internal angles is greater than 180degrees, ("reflex angle") the heuristic was giving a poor choice of split. Instead of using a special case for quads, this change routes everything through the generic ngon polyfill_beautify method instead. Some corner cases regarding non-manifold geometry are documented in the code. Note that there are two incorrect calls to `param_assert` that are also removed by this differential and replaced by a correct `BLI_assert` instead.