Blender Git Commit Log
Git Commits -> Revision 28cc31b
Revision 28cc31b by Tamito Kajiyama (master) October 10, 2011, 19:57 (GMT) |
Stability improvements for the Face Smoothness option. The instability considered here is due to a persistent failure of the getFEdge() method in the Interface0D class and its subclasses in the presence of smooth FEdges. When the Face Smoothness option is enabled, the view map is populated with not only sharp FEdges (i.e., edges in the original meshes) but also smooth FEdges (i.e., newly built edges lying on triangular surfaces). The failure of getFEdge() caused many related issues because the method is widely used in other predicates and functions that rely on it. The most prominent example of related user-visible problems is a constant failure of the built-in MaterialF0D. The main issue and related problems were addressed as follows: * A bug in the construction of smooth FEdges was fixed. Individual smooth FEdges, even when they were detected as a series of smooth FEdges that constitute one smooth ViewEdge, may have some irregular geometry in the form of non-uniform OWXFaceLayer::order values. The OWXFaceLayer::order values were used in an inappropriate way, so that resulting smooth ViewEdges may have an FEdge between two subsequent SVertices that were indeed the same SVertex object. This was an unexpected situation that getFEdge() could not handle. * Another issue in the construction of smooth FEdges was resolved. When sharp FEdges are constructed, two SVertices at both ends of an FEdge are generated only when no SVertex exists in a given 3D position (this way, the original mesh topology is reconstructed from a bunch of independent triangles that the BlenderFileLoader class passes to the view map creation process). This sharing of SVertices was used also for the generation of SVertices at the two ends of each smooth FEdge, causing the getFEdge() failure in the presence of smooth FEdges. The workaround implemented here is to simply suppress the sharing of generated SVertices when smooth FEdges are created. * In the Parameter Editor mode, the built-in MaterialF0D was replaced with a better implementation that works well with Curves and Strokes. MaterialF0D does not work with these 1D data types. |
Commit Details:
Full Hash: 28cc31ba11f805c4cac2a250b4181ec9c556b8da
SVN Revision: 40917
Parent Commit: 6c64286
Lines Changed: +71, -44
4 Modified Paths:
/release/scripts/freestyle/style_modules/Functions0D.py (+11, -0) (Diff)
/release/scripts/freestyle/style_modules/parameter_editor.py (+2, -2) (Diff)
/source/blender/freestyle/intern/view_map/ViewEdgeXBuilder.cpp (+55, -41) (Diff)
/source/blender/freestyle/intern/view_map/ViewEdgeXBuilder.h (+3, -1) (Diff)
/release/scripts/freestyle/style_modules/parameter_editor.py (+2, -2) (Diff)
/source/blender/freestyle/intern/view_map/ViewEdgeXBuilder.cpp (+55, -41) (Diff)
/source/blender/freestyle/intern/view_map/ViewEdgeXBuilder.h (+3, -1) (Diff)