Merge v1.38.3 into master

CHANGELOG.md

@@ -10,6 +10,17 @@ https://github.com/plotly/plotly.js/compare/vX.Y.Z...master
 where X.Y.Z is the semver of most recent plotly.js release.
 
 
+## [1.38.3] -- 2018-06-11
+
+### Fixed
+- Fix `cone` axis padding when under `sizemode: 'absolute'` [#2715]
+- Fix `cone` scaling on irregular grids [#2715]
+- Fix `cone` `sizemode: 'absolute'` scaling and attribute description [#2715]
+- Improve `cone` hover picking [#2715]
+- Fix exception during histogram cross-trace computation [#2724]
+- Fix handling of custom transforms that make their own data arrays [#2714]
+
+
 ## [1.38.2] -- 2018-06-04
 
 ### Fixed

dist/README.md

@@ -38,7 +38,7 @@ You can grab the relevant MathJax files in `./dist/extras/mathjax/`.
 Plotly.js defaults to US English (en-US) and includes British English (en) in the standard bundle.
 Many other localizations are available - here is an example using Swiss-German (de-CH),
 see the contents of this directory for the full list.
-They are also available on our CDN as https://cdn.plot.ly/plotly-locale-de-ch-latest.js OR https://cdn.plot.ly/plotly-locale-de-ch-1.38.2.js
+They are also available on our CDN as https://cdn.plot.ly/plotly-locale-de-ch-latest.js OR https://cdn.plot.ly/plotly-locale-de-ch-1.38.3.js
 Note that the file names are all lowercase, even though the region is uppercase when you apply a locale.
 
 *After* the plotly.js script tag, add:
@@ -61,11 +61,11 @@ The main plotly.js bundle includes all the official (non-beta) trace modules.
 
 It be can imported as minified javascript
 - using dist file `dist/plotly.min.js`
-- using CDN URL https://cdn.plot.ly/plotly-latest.min.js OR https://cdn.plot.ly/plotly-1.38.2.min.js
+- using CDN URL https://cdn.plot.ly/plotly-latest.min.js OR https://cdn.plot.ly/plotly-1.38.3.min.js
 
 or as raw javascript:
 - using dist file `dist/plotly.js`
-- using CDN URL https://cdn.plot.ly/plotly-latest.js OR https://cdn.plot.ly/plotly-1.38.2.js
+- using CDN URL https://cdn.plot.ly/plotly-latest.js OR https://cdn.plot.ly/plotly-1.38.3.js
 - using CommonJS with `require('plotly.js')`
 
 If you would like to have access to the attribute meta information (including attribute descriptions as on the [schema reference page](https://plot.ly/javascript/reference/)), use dist file `dist/plotly-with-meta.js`
@@ -74,7 +74,7 @@ The main plotly.js bundle weights in at:
 
 | plotly.js | plotly.min.js | plotly.min.js + gzip | plotly-with-meta.js |
 |-----------|---------------|----------------------|---------------------|
-| 6.3 MB | 2.6 MB | 791.7 kB | 6.5 MB |
+| 6.3 MB | 2.6 MB | 791.6 kB | 6.5 MB |
 
 ## Partial bundles
 
@@ -98,13 +98,13 @@ The `basic` partial bundle contains the `scatter`, `bar` and `pie` trace modules
 | dist bundle (minified) | `dist/plotly-basic.min.js` |
 | CDN URL (latest) | https://cdn.plot.ly/plotly-basic-latest.js |
 | CDN URL (latest minified) | https://cdn.plot.ly/plotly-basic-latest.min.js |
-| CDN URL (tagged) | https://cdn.plot.ly/plotly-basic-1.38.2.js |
-| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-basic-1.38.2.min.js |
+| CDN URL (tagged) | https://cdn.plot.ly/plotly-basic-1.38.3.js |
+| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-basic-1.38.3.min.js |
 | CommonJS | `require('plotly.js/lib/index-basic')` |
 
 | Raw size | Minified size | Minified + gzip size |
 |------|-----------------|------------------------|
-| 2.1 MB | 749.8 kB | 244.2 kB |
+| 2.1 MB | 750 kB | 244.3 kB |
 
 ### plotly.js cartesian
 
@@ -116,13 +116,13 @@ The `cartesian` partial bundle contains the `scatter`, `bar`, `box`, `heatmap`,
 | dist bundle (minified) | `dist/plotly-cartesian.min.js` |
 | CDN URL (latest) | https://cdn.plot.ly/plotly-cartesian-latest.js |
 | CDN URL (latest minified) | https://cdn.plot.ly/plotly-cartesian-latest.min.js |
-| CDN URL (tagged) | https://cdn.plot.ly/plotly-cartesian-1.38.2.js |
-| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-cartesian-1.38.2.min.js |
+| CDN URL (tagged) | https://cdn.plot.ly/plotly-cartesian-1.38.3.js |
+| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-cartesian-1.38.3.min.js |
 | CommonJS | `require('plotly.js/lib/index-cartesian')` |
 
 | Raw size | Minified size | Minified + gzip size |
 |------|-----------------|------------------------|
-| 2.4 MB | 860.9 kB | 278.8 kB |
+| 2.4 MB | 861 kB | 278.9 kB |
 
 ### plotly.js geo
 
@@ -134,13 +134,13 @@ The `geo` partial bundle contains the `scatter`, `scattergeo` and `choropleth` t
 | dist bundle (minified) | `dist/plotly-geo.min.js` |
 | CDN URL (latest) | https://cdn.plot.ly/plotly-geo-latest.js |
 | CDN URL (latest minified) | https://cdn.plot.ly/plotly-geo-latest.min.js |
-| CDN URL (tagged) | https://cdn.plot.ly/plotly-geo-1.38.2.js |
-| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-geo-1.38.2.min.js |
+| CDN URL (tagged) | https://cdn.plot.ly/plotly-geo-1.38.3.js |
+| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-geo-1.38.3.min.js |
 | CommonJS | `require('plotly.js/lib/index-geo')` |
 
 | Raw size | Minified size | Minified + gzip size |
 |------|-----------------|------------------------|
-| 2.1 MB | 773.6 kB | 253.4 kB |
+| 2.1 MB | 773.7 kB | 253.4 kB |
 
 ### plotly.js gl3d
 
@@ -152,8 +152,8 @@ The `gl3d` partial bundle contains the `scatter`, `scatter3d`, `surface`, `mesh3
 | dist bundle (minified) | `dist/plotly-gl3d.min.js` |
 | CDN URL (latest) | https://cdn.plot.ly/plotly-gl3d-latest.js |
 | CDN URL (latest minified) | https://cdn.plot.ly/plotly-gl3d-latest.min.js |
-| CDN URL (tagged) | https://cdn.plot.ly/plotly-gl3d-1.38.2.js |
-| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-gl3d-1.38.2.min.js |
+| CDN URL (tagged) | https://cdn.plot.ly/plotly-gl3d-1.38.3.js |
+| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-gl3d-1.38.3.min.js |
 | CommonJS | `require('plotly.js/lib/index-gl3d')` |
 
 | Raw size | Minified size | Minified + gzip size |
@@ -170,8 +170,8 @@ The `gl2d` partial bundle contains the `scatter`, `scattergl`, `splom`, `pointcl
 | dist bundle (minified) | `dist/plotly-gl2d.min.js` |
 | CDN URL (latest) | https://cdn.plot.ly/plotly-gl2d-latest.js |
 | CDN URL (latest minified) | https://cdn.plot.ly/plotly-gl2d-latest.min.js |
-| CDN URL (tagged) | https://cdn.plot.ly/plotly-gl2d-1.38.2.js |
-| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-gl2d-1.38.2.min.js |
+| CDN URL (tagged) | https://cdn.plot.ly/plotly-gl2d-1.38.3.js |
+| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-gl2d-1.38.3.min.js |
 | CommonJS | `require('plotly.js/lib/index-gl2d')` |
 
 | Raw size | Minified size | Minified + gzip size |
@@ -188,8 +188,8 @@ The `mapbox` partial bundle contains the `scatter` and `scattermapbox` trace mod
 | dist bundle (minified) | `dist/plotly-mapbox.min.js` |
 | CDN URL (latest) | https://cdn.plot.ly/plotly-mapbox-latest.js |
 | CDN URL (latest minified) | https://cdn.plot.ly/plotly-mapbox-latest.min.js |
-| CDN URL (tagged) | https://cdn.plot.ly/plotly-mapbox-1.38.2.js |
-| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-mapbox-1.38.2.min.js |
+| CDN URL (tagged) | https://cdn.plot.ly/plotly-mapbox-1.38.3.js |
+| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-mapbox-1.38.3.min.js |
 | CommonJS | `require('plotly.js/lib/index-mapbox')` |
 
 | Raw size | Minified size | Minified + gzip size |
@@ -206,13 +206,13 @@ The `finance` partial bundle contains the `scatter`, `bar`, `histogram`, `pie`,
 | dist bundle (minified) | `dist/plotly-finance.min.js` |
 | CDN URL (latest) | https://cdn.plot.ly/plotly-finance-latest.js |
 | CDN URL (latest minified) | https://cdn.plot.ly/plotly-finance-latest.min.js |
-| CDN URL (tagged) | https://cdn.plot.ly/plotly-finance-1.38.2.js |
-| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-finance-1.38.2.min.js |
+| CDN URL (tagged) | https://cdn.plot.ly/plotly-finance-1.38.3.js |
+| CDN URL (tagged minified) | https://cdn.plot.ly/plotly-finance-1.38.3.min.js |
 | CommonJS | `require('plotly.js/lib/index-finance')` |
 
 | Raw size | Minified size | Minified + gzip size |
 |------|-----------------|------------------------|
-| 2.2 MB | 780 kB | 253.3 kB |
+| 2.2 MB | 780.1 kB | 253.3 kB |
 
 ----------------
 

dist/plot-schema.json

@@ -17612,15 +17612,14 @@
                     "role": "info",
                     "editType": "calc",
                     "dflt": "scaled",
-                    "description": "Sets the mode by which the cones are sized. If *scaled*, `sizeref` scales such that the reference cone size for the maximum vector magnitude is 1. If *absolute*, `sizeref` scales such that the reference cone size for vector magnitude 1 is one grid unit."
+                    "description": "Determines whether `sizeref` is set as a *scaled* (i.e unitless) scalar (normalized by the max u/v/w norm in the vector field) or as *absolute* value (in the same units as the vector field)."
                 },
                 "sizeref": {
                     "valType": "number",
                     "role": "info",
                     "editType": "calc",
                     "min": 0,
-                    "dflt": 1,
-                    "description": "Sets the cone size reference value."
+                    "description": "Adjusts the cone size scaling. The size of the cones is determined by their u/v/w norm multiplied a factor and `sizeref`. This factor (computed internally) corresponds to the minimum \"time\" to travel across two successive x/y/z positions at the average velocity of those two successive positions. All cones in a given trace use the same factor. With `sizemode` set to *scaled*, `sizeref` is unitless, its default value is *0.5* With `sizemode` set to *absolute*, `sizeref` has the same units as the u/v/w vector field, its the default value is half the sample's maximum vector norm."
                 },
                 "anchor": {
                     "valType": "enumerated",

dist/plotly-basic.js

@@ -1,5 +1,5 @@
 /**
-* plotly.js (basic) v1.38.2
+* plotly.js (basic) v1.38.3
 * Copyright 2012-2018, Plotly, Inc.
 * All rights reserved.
 * Licensed under the MIT license
@@ -33021,7 +33021,7 @@ exports.svgAttrs = {
 'use strict';
 
 // package version injected by `npm run preprocess`
-exports.version = '1.38.2';
+exports.version = '1.38.3';
 
 // inject promise polyfill
 require('es6-promise').polyfill();
@@ -55885,11 +55885,29 @@ plots.supplyTraceDefaults = function(traceIn, colorIndex, layout, traceInIndex)
     return traceOut;
 };
 
+/**
+ * hasMakesDataTransform: does this trace have a transform that makes its own
+ * data, either by grabbing it from somewhere else or by creating it from input
+ * parameters? If so, we should still keep going with supplyDefaults
+ * even if the trace is invisible, which may just be because it has no data yet.
+ */
+function hasMakesDataTransform(traceIn) {
+    var transformsIn = traceIn.transforms;
+    if(Array.isArray(transformsIn) && transformsIn.length) {
+        for(var i = 0; i < transformsIn.length; i++) {
+            var _module = transformsRegistry[transformsIn[i].type];
+            if(_module && _module.makesData) return true;
+        }
+    }
+    return false;
+}
+
 plots.supplyTransformDefaults = function(traceIn, traceOut, layout) {
     // For now we only allow transforms on 1D traces, ie those that specify a _length.
     // If we were to implement 2D transforms, we'd need to have each transform
     // describe its own applicability and disable itself when it doesn't apply.
-    if(!traceOut._length) return;
+    // Also allow transforms that make their own data, but not in globalTransforms
+    if(!(traceOut._length || hasMakesDataTransform(traceIn))) return;
 
     var globalTransforms = layout._globalTransforms || [];
     var transformModules = layout._transformModules || [];

dist/plotly-cartesian.js

@@ -1,5 +1,5 @@
 /**
-* plotly.js (cartesian) v1.38.2
+* plotly.js (cartesian) v1.38.3
 * Copyright 2012-2018, Plotly, Inc.
 * All rights reserved.
 * Licensed under the MIT license
@@ -33171,7 +33171,7 @@ exports.svgAttrs = {
 'use strict';
 
 // package version injected by `npm run preprocess`
-exports.version = '1.38.2';
+exports.version = '1.38.3';
 
 // inject promise polyfill
 require('es6-promise').polyfill();
@@ -56035,11 +56035,29 @@ plots.supplyTraceDefaults = function(traceIn, colorIndex, layout, traceInIndex)
     return traceOut;
 };
 
+/**
+ * hasMakesDataTransform: does this trace have a transform that makes its own
+ * data, either by grabbing it from somewhere else or by creating it from input
+ * parameters? If so, we should still keep going with supplyDefaults
+ * even if the trace is invisible, which may just be because it has no data yet.
+ */
+function hasMakesDataTransform(traceIn) {
+    var transformsIn = traceIn.transforms;
+    if(Array.isArray(transformsIn) && transformsIn.length) {
+        for(var i = 0; i < transformsIn.length; i++) {
+            var _module = transformsRegistry[transformsIn[i].type];
+            if(_module && _module.makesData) return true;
+        }
+    }
+    return false;
+}
+
 plots.supplyTransformDefaults = function(traceIn, traceOut, layout) {
     // For now we only allow transforms on 1D traces, ie those that specify a _length.
     // If we were to implement 2D transforms, we'd need to have each transform
     // describe its own applicability and disable itself when it doesn't apply.
-    if(!traceOut._length) return;
+    // Also allow transforms that make their own data, but not in globalTransforms
+    if(!(traceOut._length || hasMakesDataTransform(traceIn))) return;
 
     var globalTransforms = layout._globalTransforms || [];
     var transformModules = layout._transformModules || [];
@@ -69500,7 +69518,7 @@ var oneMonth = require('../../constants/numerical').ONEAVGMONTH;
 var getBinSpanLabelRound = require('./bin_label_vals');
 
 module.exports = function calc(gd, trace) {
-    // ignore as much processing as possible (and including in autorange) if bar is not visible
+    // ignore as much processing as possible (and including in autorange) if not visible
     if(trace.visible !== true) return;
 
     // depending on orientation, set position and size axes and data ranges
@@ -69912,6 +69930,7 @@ function getConnectedHistograms(gd, trace) {
     for(var i = 0; i < fullData.length; i++) {
         var tracei = fullData[i];
         if(tracei.type === 'histogram' &&
+            tracei.visible === true &&
             tracei.orientation === orientation &&
             tracei.xaxis === xid && tracei.yaxis === yid
         ) {

dist/plotly-finance.js

@@ -1,5 +1,5 @@
 /**
-* plotly.js (finance) v1.38.2
+* plotly.js (finance) v1.38.3
 * Copyright 2012-2018, Plotly, Inc.
 * All rights reserved.
 * Licensed under the MIT license
@@ -33063,7 +33063,7 @@ exports.svgAttrs = {
 'use strict';
 
 // package version injected by `npm run preprocess`
-exports.version = '1.38.2';
+exports.version = '1.38.3';
 
 // inject promise polyfill
 require('es6-promise').polyfill();
@@ -55927,11 +55927,29 @@ plots.supplyTraceDefaults = function(traceIn, colorIndex, layout, traceInIndex)
     return traceOut;
 };
 
+/**
+ * hasMakesDataTransform: does this trace have a transform that makes its own
+ * data, either by grabbing it from somewhere else or by creating it from input
+ * parameters? If so, we should still keep going with supplyDefaults
+ * even if the trace is invisible, which may just be because it has no data yet.
+ */
+function hasMakesDataTransform(traceIn) {
+    var transformsIn = traceIn.transforms;
+    if(Array.isArray(transformsIn) && transformsIn.length) {
+        for(var i = 0; i < transformsIn.length; i++) {
+            var _module = transformsRegistry[transformsIn[i].type];
+            if(_module && _module.makesData) return true;
+        }
+    }
+    return false;
+}
+
 plots.supplyTransformDefaults = function(traceIn, traceOut, layout) {
     // For now we only allow transforms on 1D traces, ie those that specify a _length.
     // If we were to implement 2D transforms, we'd need to have each transform
     // describe its own applicability and disable itself when it doesn't apply.
-    if(!traceOut._length) return;
+    // Also allow transforms that make their own data, but not in globalTransforms
+    if(!(traceOut._length || hasMakesDataTransform(traceIn))) return;
 
     var globalTransforms = layout._globalTransforms || [];
     var transformModules = layout._transformModules || [];
@@ -63867,7 +63885,7 @@ var oneMonth = require('../../constants/numerical').ONEAVGMONTH;
 var getBinSpanLabelRound = require('./bin_label_vals');
 
 module.exports = function calc(gd, trace) {
-    // ignore as much processing as possible (and including in autorange) if bar is not visible
+    // ignore as much processing as possible (and including in autorange) if not visible
     if(trace.visible !== true) return;
 
     // depending on orientation, set position and size axes and data ranges
@@ -64279,6 +64297,7 @@ function getConnectedHistograms(gd, trace) {
     for(var i = 0; i < fullData.length; i++) {
         var tracei = fullData[i];
         if(tracei.type === 'histogram' &&
+            tracei.visible === true &&
             tracei.orientation === orientation &&
             tracei.xaxis === xid && tracei.yaxis === yid
         ) {