mirror of
https://github.com/PixarAnimationStudios/OpenSubdiv
synced 2024-11-10 06:10:07 +00:00
Removed use of "full topology" flag with Far::AdaptiveOptions:
- it was always ignored as full topology is required for PatchTables - it will soon be removed from AdaptiveOptions entirely
This commit is contained in:
parent
d758d572f7
commit
9222c9e169
@ -323,7 +323,6 @@ createMesh(ShapeDesc const & shapeDesc, int level) {
|
||||
refiner->RefineUniform(options);
|
||||
} else {
|
||||
Far::TopologyRefiner::AdaptiveOptions options(level);
|
||||
options.fullTopologyInLastLevel = false;
|
||||
options.useSingleCreasePatch = false;
|
||||
refiner->RefineAdaptive(options);
|
||||
}
|
||||
|
@ -758,7 +758,6 @@ createVtrMesh(Shape * shape, int maxlevel) {
|
||||
|
||||
if (g_Adaptive) {
|
||||
OpenSubdiv::Far::TopologyRefiner::AdaptiveOptions options(maxlevel);
|
||||
options.fullTopologyInLastLevel = true;
|
||||
options.useSingleCreasePatch = false;
|
||||
refiner->RefineAdaptive(options);
|
||||
} else {
|
||||
|
@ -98,14 +98,14 @@ protected:
|
||||
|
||||
static inline void refineMesh(Far::TopologyRefiner & refiner, int level, bool adaptive, bool singleCreasePatch) {
|
||||
|
||||
bool fullTopologyInLastLevel = refiner.GetNumFVarChannels()>0;
|
||||
|
||||
if (adaptive) {
|
||||
Far::TopologyRefiner::AdaptiveOptions options(level);
|
||||
options.fullTopologyInLastLevel = fullTopologyInLastLevel;
|
||||
options.useSingleCreasePatch = singleCreasePatch;
|
||||
refiner.RefineAdaptive(options);
|
||||
} else {
|
||||
// This dependency on FVar channels should not be necessary
|
||||
bool fullTopologyInLastLevel = refiner.GetNumFVarChannels()>0;
|
||||
|
||||
Far::TopologyRefiner::UniformOptions options(level);
|
||||
options.fullTopologyInLastLevel = fullTopologyInLastLevel;
|
||||
refiner.RefineUniform(options);
|
||||
|
Loading…
Reference in New Issue
Block a user