Commit: 3b01dbdeb14a9deca0857ca6c3b038840fb96388
Author: Sergey Sharybin
Date: Tue Feb 18 17:23:52 2020 +0100
Branches: master
https://developer.blender.org/rB3b01dbdeb14a9deca0857ca6c3b038840fb96388
Subdiv: Clarify Subdiv settings
The actual naming might also be a subject to change, especially the one
around `level`. Tricky part here is that at some point in the API there
will be change from Blender modifier's Quality to OpenSubdiv's Level,
but which API level is most suitable for this?
At least now meaning of settings is better documented ans should be
clear what's going on.
===================================================================
M source/blender/blenkernel/BKE_subdiv.h
===================================================================
diff --git a/source/blender/blenkernel/BKE_subdiv.h
b/source/blender/blenkernel/BKE_subdiv.h
index 267a03da25c..d7f3bb5a5bf 100644
--- a/source/blender/blenkernel/BKE_subdiv.h
+++ b/source/blender/blenkernel/BKE_subdiv.h
@@ -53,10 +53,33 @@ typedef enum eSubdivFVarLinearInterpolation {
} eSubdivFVarLinearInterpolation;
typedef struct SubdivSettings {
+ /* Simple subdivision corresponds to "Simple" option in the interface. When
its enabled the
+ * subdivided mesh is not "smoothed": new vertices are added uniformly on
the existing surface.
+ *
+ * On an OpenSubdiv implementation level this translates to a subdivision
scheme:
+ * when is_simple is true OSD_SCHEME_BILINEAR is used, otherwise
OSD_SCHEME_CATMARK. */
bool is_simple;
+
+ /* This refers to an adaptive isolation when creating patches for the
subdivided surface.
+ *
+ * When is set to to false (aka uniform subdivision) fixed depth of
isolation is used, which
+ * allows to iteratively add more subdivisions (uniform subdivision level 2
= uniform subdivision
+ * level 1 + uniform subdivision level 1). Uniform subdivisions will
progressively go to a limit
+ * surface.
+ *
+ * Adaptive isolation generates patches at a limit surface (aka as if
infinite number of uniform
+ * subdivisions have been applied). This setting allows to have matches
normal and tangent space
+ * the same independent of number of subdivisions set in modifier settings.
*/
bool is_adaptive;
+
+ /* Corresponds to Quality option in modifier settings: higher values means
the final surface
+ * will be more accurately represented by patches.
+ *
+ * On an OpenSubdiv implementation level this is an isolation level. */
int level;
+
bool use_creases;
+
eSubdivVtxBoundaryInterpolation vtx_boundary_interpolation;
eSubdivFVarLinearInterpolation fvar_linear_interpolation;
} SubdivSettings;
_______________________________________________
Bf-blender-cvs mailing list
Bf-blender-cvs@blender.org
https://lists.blender.org/mailman/listinfo/bf-blender-cvs
