scatter3d now performs axis transformation on error bars

Author: antoinerg

src/traces/scatter3d/calc_errors.js

@@ -10,7 +10,7 @@
 
 var Registry = require('../../registry');
 
-function calculateAxisErrors(data, params, scaleFactor) {
+function calculateAxisErrors(data, params, scaleFactor, axis, calendar) {
     if(!params || !params.visible) return null;
 
     var computeError = Registry.getComponentMethod('errorbars', 'makeComputeError')(params);
@@ -19,9 +19,10 @@ function calculateAxisErrors(data, params, scaleFactor) {
     for(var i = 0; i < data.length; i++) {
         var errors = computeError(+data[i], i);
 
+        var point = axis.d2l(data[i], 0, calendar) * scaleFactor; // A bit wasteful
         result[i] = [
-            -errors[0] * scaleFactor,
-            errors[1] * scaleFactor
+            (axis.d2l(data[i] - errors[0], 0, calendar) * scaleFactor) - point || -point,
+            (axis.d2l(data[i] + errors[1], 0, calendar) * scaleFactor) - point || -point
         ];
     }
 
@@ -35,11 +36,11 @@ function dataLength(array) {
     return 0;
 }
 
-function calculateErrors(data, scaleFactor) {
+function calculateErrors(data, scaleFactor, sceneLayout) {
     var errors = [
-        calculateAxisErrors(data.x, data.error_x, scaleFactor[0]),
-        calculateAxisErrors(data.y, data.error_y, scaleFactor[1]),
-        calculateAxisErrors(data.z, data.error_z, scaleFactor[2])
+        calculateAxisErrors(data.x, data.error_x, scaleFactor[0], sceneLayout.xaxis, data.xcalendar),
+        calculateAxisErrors(data.y, data.error_y, scaleFactor[1], sceneLayout.yaxis, data.ycalendar),
+        calculateAxisErrors(data.z, data.error_z, scaleFactor[2], sceneLayout.zaxis, data.zcalendar)
     ];
 
     var n = dataLength(errors);

src/traces/scatter3d/convert.js

@@ -252,7 +252,7 @@ function convertPlotlyOptions(scene, data) {
         }
     }
 
-    params.errorBounds = calculateError(data, scaleFactor);
+    params.errorBounds = calculateError(data, scaleFactor, sceneLayout, points);
 
     var errorParams = calculateErrorParams([data.error_x, data.error_y, data.error_z]);
     params.errorColor = errorParams.color;

test/image/mocks/gl3d_error_bars_log.json

@@ -0,0 +1,26 @@
+{
+    "data": [{
+        "x": [0, 1, 2, 3, 4, 5, 6, 7, 8, 9],
+        "y": [0, 0, 0, 0, 0, 0, 0, 0, 0, 0],
+        "z": [1e+00, 1e+01, 1e+02, 1e+03, 1e+04,
+            1e+05, 1e+06, 1e+07, 1e+08, 1e+09
+        ],
+        "type": "scatter3d",
+        "error_z": {
+            "array": [1e-01, 2e+01, 1e+1, 9e+03, 5e+03,
+                0.9e+05, 1e+05, 1e+07, 9e+08, 1e+09
+            ],
+            "type": "data",
+            "visible": true
+        }
+    }],
+    "layout": {
+        "scene": {
+            "zaxis": {
+                "type": "log"
+            }
+        },
+        "width": 800,
+        "height": 800
+    }
+}