Revision: 73929
          http://sourceforge.net/p/brlcad/code/73929
Author:   starseeker
Date:     2019-09-14 16:34:14 +0000 (Sat, 14 Sep 2019)
Log Message:
-----------
40 was too high a threshold at which to define a 'high level' box.

Modified Paths:
--------------
    brlcad/trunk/src/libbrep/cdt_surf.cpp

Modified: brlcad/trunk/src/libbrep/cdt_surf.cpp
===================================================================
--- brlcad/trunk/src/libbrep/cdt_surf.cpp       2019-09-14 16:04:27 UTC (rev 
73928)
+++ brlcad/trunk/src/libbrep/cdt_surf.cpp       2019-09-14 16:34:14 UTC (rev 
73929)
@@ -302,7 +302,10 @@
        return false;
     }
 
-    if (nhits > 40) {
+    // The nhits threshold is basically indirectly determining how many 
triangles
+    // from the edge we're willing to have point to a single interior point. 
Too
+    // many and we'll end up with a lot of long, thin triangles.
+    if (nhits > 5) {
        // Lot of edges, probably a high level box we need to split - just 
return the overall min_edge
        (*min_edge) = sinfo->min_edge;
        return true;

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.



_______________________________________________
BRL-CAD Source Commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-commits

Reply via email to