Merge pull request #3521 from plotly/speedup-isosurface

Increase the speed of isosurface creation

Author: archmoj

src/traces/isosurface/attributes.js

@@ -10,7 +10,6 @@
 
 var colorscaleAttrs = require('../../components/colorscale/attributes');
 var colorbarAttrs = require('../../components/colorbar/attributes');
-var surfaceAtts = require('../surface/attributes');
 var meshAttrs = require('../mesh3d/attributes');
 var baseAttrs = require('../../plots/attributes');
 
@@ -225,7 +224,7 @@ var attrs = module.exports = overrideAll(extendFlat({
             'If trace `hoverinfo` contains a *text* flag and *hovertext* is not set,',
             'these elements will be seen in the hover labels.'
         ].join(' ')
-    },
+    }
 },
 
 colorscaleAttrs('', {
@@ -236,37 +235,16 @@ colorscaleAttrs('', {
 
     colorbar: colorbarAttrs,
 
-    // Flat shaded mode
-    flatshading: {
-        valType: 'boolean',
-        role: 'style',
-        dflt: false,
-        description: [
-            'Determines whether or not normal smoothing is applied to the isosurfaces,',
-            'creating isosurfaces with an angular, low-poly look via flat reflections.'
-        ].join(' ')
-    },
-
-    contour: {
-        show: extendFlat({}, surfaceAtts.contours.x.show, {
-            description: [
-                'Sets whether or not dynamic contours are shown on hover.',
-                'Contours are more useful when hovering on caps and slices.'
-            ].join(' ')
-        }),
-        color: surfaceAtts.contours.x.color,
-        width: surfaceAtts.contours.x.width
-    },
-
-    lightposition: {
-        x: extendFlat({}, surfaceAtts.lightposition.x, {dflt: 1e5}),
-        y: extendFlat({}, surfaceAtts.lightposition.y, {dflt: 1e5}),
-        z: extendFlat({}, surfaceAtts.lightposition.z, {dflt: 0})
-    },
+    contour: meshAttrs.contour,
+    lightposition: meshAttrs.lightposition,
     lighting: meshAttrs.lighting,
+    flatshading: meshAttrs.flatshading,
 
     hoverinfo: extendFlat({}, baseAttrs.hoverinfo)
 }), 'calc', 'nested');
 
+// required defaults to speed up surface normal calculations
+attrs.flatshading.dflt = true; attrs.lighting.facenormalsepsilon.dflt = 0;
+
 attrs.x.editType = attrs.y.editType = attrs.z.editType = attrs.value.editType = 'calc+clearAxisTypes';
 attrs.transforms = undefined;

src/traces/isosurface/convert.js

@@ -396,9 +396,9 @@ function generateIsosurfaceMesh(data) {
 
     function tryCreateTri(style, xyzv, abc, min, max, isSecondPass) {
 
-        abc = [-1, -1, -1]; // Note: for the moment we had to override indices
-        // for planar surfaces (i.e. caps and slices) due to group shading
-        // bug of gl-mesh3d. But don't worry this would run faster!
+        abc = [-1, -1, -1]; // Note: for the moment we override indices
+        // to run faster! But it is possible to comment this line
+        // to reduce the number of vertices.
 
         var tryDrawTri = function(style, xyzv, abc) {
             if( // we check here if the points are in `real` iso-min/max range

src/traces/isosurface/defaults.js

@@ -95,7 +95,7 @@ module.exports = function supplyDefaults(traceIn, traceOut, defaultColor, layout
         'lightposition.x',
         'lightposition.y',
         'lightposition.z',
-        'flatshading',
+        'flatshading'
     ].forEach(function(x) { coerce(x); });
 
     colorscaleDefaults(traceIn, traceOut, layout, coerce, {prefix: '', cLetter: 'c'});

test/image/baselines/gl3d_isosurface_2surfaces-checker_spaceframe.png

@@ -1878,11 +1878,11 @@
             1.000, 1.450, 1.900, 2.350, 2.800, 3.250, 3.700, 4.150, 4.600, 5.050, 5.500, 5.950, 6.400, 6.850, 7.300, 7.750, 8.200, 8.650, 9.100, 9.550, 10.000
         ],
         "lighting": {
-        "ambient": 0.5,
-        "diffuse": 1.0,
-        "specular": 0.75,
-        "roughness": 1.0,
-        "fresnel": 0.25
+            "ambient": 0.5,
+            "diffuse": 1.0,
+            "specular": 0.75,
+            "roughness": 1.0,
+            "fresnel": 0.25
         },
         "lightposition": {
             "x": 10000,

test/image/baselines/gl3d_isosurface_math.png

@@ -3638,11 +3638,11 @@
             -10, -8, -6, -4, -2, 0, 2, 4, 6, 8, 10
         ],
         "lighting": {
-        "ambient": 0.5,
-        "diffuse": 1.0,
-        "specular": 0.75,
-        "roughness": 1.0,
-        "fresnel": 0.25
+            "ambient": 0.5,
+            "diffuse": 1.0,
+            "specular": 0.75,
+            "roughness": 1.0,
+            "fresnel": 0.25
         },
         "lightposition": {
             "x": 10000,

test/image/baselines/gl3d_isosurface_xycaps_volume_slices.png

@@ -1878,11 +1878,11 @@
             0.000, 0.015, 0.120, 0.405, 0.960, 1.875, 3.240, 5.145, 7.680, 10.935, 15.000, 19.965, 25.920, 32.955, 41.160, 50.625, 61.440, 73.695, 87.480, 102.885, 120.000
         ],
         "lighting": {
-        "ambient": 0.5,
-        "diffuse": 1.0,
-        "specular": 0.75,
-        "roughness": 1.0,
-        "fresnel": 0.25
+            "ambient": 0.5,
+            "diffuse": 1.0,
+            "specular": 0.75,
+            "roughness": 1.0,
+            "fresnel": 0.25
         },
         "lightposition": {
             "x": 10000,

test/image/mocks/gl3d_isosurface_log-axis_slices_surface-fill.json

@@ -1878,11 +1878,11 @@
             -10.000, -9.998, -9.980, -9.932, -9.840, -9.688, -9.460, -9.143, -8.720, -8.178, -7.500, -6.672, -5.680, -4.507, -3.140, -1.563, 0.240, 2.282, 4.580, 7.147, 10.000
         ],
         "lighting": {
-        "ambient": 0.5,
-        "diffuse": 1.0,
-        "specular": 0.75,
-        "roughness": 1.0,
-        "fresnel": 0.25
+            "ambient": 0.5,
+            "diffuse": 1.0,
+            "specular": 0.75,
+            "roughness": 1.0,
+            "fresnel": 0.25
         },
         "lightposition": {
             "x": 10000,