fix for hover on interpolated z values

Author: archmoj

src/traces/surface/convert.js

@@ -33,6 +33,7 @@ function SurfaceTrace(scene, surface, uid) {
     this.dataScaleX = 1.0;
     this.dataScaleY = 1.0;
     this.refineData = true;
+    this._interpolatedZ = false;
 }
 
 var proto = SurfaceTrace.prototype;
@@ -62,9 +63,11 @@ proto.getYat = function(a, b, calendar, axis) {
 };
 
 proto.getZat = function(a, b, calendar, axis) {
-    var v = (
-        this.data.z[b][a]
-    );
+    var v = this.data.z[b][a];
+
+    if(v === null && this.data.connectgaps && this.data._interpolatedZ) {
+        v = this.data._interpolatedZ[b][a];
+    }
 
     return (calendar === undefined) ? v : axis.d2l(v, 0, calendar);
 };
@@ -410,6 +413,14 @@ proto.update = function(data) {
     if(data.connectgaps) {
         data._emptypoints = findEmpties(rawCoords[2]);
         interp2d(rawCoords[2], data._emptypoints);
+
+        data._interpolatedZ = [];
+        for(j = 0; j < xlen; j++) {
+            data._interpolatedZ[j] = [];
+            for(k = 0; k < ylen; k++) {
+                data._interpolatedZ[j][k] = rawCoords[2][j][k];
+            }
+        }
     }
 
     // Note: log axes are not defined in surfaces yet.

test/image/baselines/gl3d_surface_connectgaps.png

@@ -3,18 +3,18 @@
       {
         "type": "surface",
         "connectgaps": true,
-        "x": [0, 1, 2, 3],
+        "x": [0, 1, 2, 3, 4],
         "y": [0, 1, 2, 3],
         "z": [
-          [1, 1, 1, 1],
-          [1, null, 2, null],
-          [1, 1, 1, 1],
+          [null, 2, 1, 1, 1],
+          [2, 1, null, 2, null],
+          [2, 1, 1, 1],
           [1, null, 0, null]
         ]
       }
     ],
     "layout": {
-        "title": "Surface interpolate gaps",
+        "title": "Surface plot with interpolated gaps<br>(connectgaps: true)",
         "width": 600,
         "height": 400
     }