echarts 4.0.4 → 4.1.0

package/lib/chart/line/LineView.js

@@ -21,11 +21,33 @@ var Polygon = _poly.Polygon;
 
 var ChartView = require("../../view/Chart");
 
+var _number = require("../../util/number");
+
+var round = _number.round;
+
 var _helper = require("./helper");
 
 var prepareDataCoordInfo = _helper.prepareDataCoordInfo;
 var getStackedOnPoint = _helper.getStackedOnPoint;
 
+/*
+* Licensed to the Apache Software Foundation (ASF) under one
+* or more contributor license agreements.  See the NOTICE file
+* distributed with this work for additional information
+* regarding copyright ownership.  The ASF licenses this file
+* to you under the Apache License, Version 2.0 (the
+* "License"); you may not use this file except in compliance
+* with the License.  You may obtain a copy of the License at
+*
+*   http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing,
+* software distributed under the License is distributed on an
+* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+* KIND, either express or implied.  See the License for the
+* specific language governing permissions and limitations
+* under the License.
+*/
 // FIXME step not support polar
 function isPointsSame(points1, points2) {
   if (points1.length !== points2.length) {
@@ -83,24 +105,33 @@ function getStackedOnPoints(coordSys, data, dataCoordInfo) {
   return points;
 }
 
-function createGridClipShape(cartesian, hasAnimation, seriesModel) {
+function createGridClipShape(cartesian, hasAnimation, forSymbol, seriesModel) {
   var xExtent = getAxisExtentWithGap(cartesian.getAxis('x'));
   var yExtent = getAxisExtentWithGap(cartesian.getAxis('y'));
   var isHorizontal = cartesian.getBaseAxis().isHorizontal();
   var x = Math.min(xExtent[0], xExtent[1]);
   var y = Math.min(yExtent[0], yExtent[1]);
   var width = Math.max(xExtent[0], xExtent[1]) - x;
-  var height = Math.max(yExtent[0], yExtent[1]) - y;
-  var lineWidth = seriesModel.get('lineStyle.width') || 2; // Expand clip shape to avoid clipping when line value exceeds axis
+  var height = Math.max(yExtent[0], yExtent[1]) - y; // Avoid float number rounding error for symbol on the edge of axis extent.
+  // See #7913 and `test/dataZoom-clip.html`.
+
+  if (forSymbol) {
+    x -= 0.5;
+    width += 0.5;
+    y -= 0.5;
+    height += 0.5;
+  } else {
+    var lineWidth = seriesModel.get('lineStyle.width') || 2; // Expand clip shape to avoid clipping when line value exceeds axis
 
-  var expandSize = seriesModel.get('clipOverflow') ? lineWidth / 2 : Math.max(width, height);
+    var expandSize = seriesModel.get('clipOverflow') ? lineWidth / 2 : Math.max(width, height);
 
-  if (isHorizontal) {
-    y -= expandSize;
-    height += expandSize * 2;
-  } else {
-    x -= expandSize;
-    width += expandSize * 2;
+    if (isHorizontal) {
+      y -= expandSize;
+      height += expandSize * 2;
+    } else {
+      x -= expandSize;
+      width += expandSize * 2;
+    }
   }
 
   var clipPath = new graphic.Rect({
@@ -125,18 +156,25 @@ function createGridClipShape(cartesian, hasAnimation, seriesModel) {
   return clipPath;
 }
 
-function createPolarClipShape(polar, hasAnimation, seriesModel) {
+function createPolarClipShape(polar, hasAnimation, forSymbol, seriesModel) {
   var angleAxis = polar.getAngleAxis();
   var radiusAxis = polar.getRadiusAxis();
-  var radiusExtent = radiusAxis.getExtent();
+  var radiusExtent = radiusAxis.getExtent().slice();
+  radiusExtent[0] > radiusExtent[1] && radiusExtent.reverse();
   var angleExtent = angleAxis.getExtent();
-  var RADIAN = Math.PI / 180;
+  var RADIAN = Math.PI / 180; // Avoid float number rounding error for symbol on the edge of axis extent.
+
+  if (forSymbol) {
+    radiusExtent[0] -= 0.5;
+    radiusExtent[1] += 0.5;
+  }
+
   var clipPath = new graphic.Sector({
     shape: {
-      cx: polar.cx,
-      cy: polar.cy,
-      r0: radiusExtent[0],
-      r: radiusExtent[1],
+      cx: round(polar.cx, 1),
+      cy: round(polar.cy, 1),
+      r0: round(radiusExtent[0], 1),
+      r: round(radiusExtent[1], 1),
       startAngle: -angleExtent[0] * RADIAN,
       endAngle: -angleExtent[1] * RADIAN,
       clockwise: angleAxis.inverse
@@ -155,8 +193,8 @@ function createPolarClipShape(polar, hasAnimation, seriesModel) {
   return clipPath;
 }
 
-function createClipShape(coordSys, hasAnimation, seriesModel) {
-  return coordSys.type === 'polar' ? createPolarClipShape(coordSys, hasAnimation, seriesModel) : createGridClipShape(coordSys, hasAnimation, seriesModel);
+function createClipShape(coordSys, hasAnimation, forSymbol, seriesModel) {
+  return coordSys.type === 'polar' ? createPolarClipShape(coordSys, hasAnimation, forSymbol, seriesModel) : createGridClipShape(coordSys, hasAnimation, forSymbol, seriesModel);
 }
 
 function turnPointsIntoStep(points, coordSys, stepTurnAt) {
@@ -287,6 +325,63 @@ function getVisualGradient(data, coordSys) {
   return gradient;
 }
 
+function getIsIgnoreFunc(seriesModel, data, coordSys) {
+  var showAllSymbol = seriesModel.get('showAllSymbol');
+  var isAuto = showAllSymbol === 'auto';
+
+  if (showAllSymbol && !isAuto) {
+    return;
+  }
+
+  var categoryAxis = coordSys.getAxesByScale('ordinal')[0];
+
+  if (!categoryAxis) {
+    return;
+  } // Note that category label interval strategy might bring some weird effect
+  // in some scenario: users may wonder why some of the symbols are not
+  // displayed. So we show all symbols as possible as we can.
+
+
+  if (isAuto // Simplify the logic, do not determine label overlap here.
+  && canShowAllSymbolForCategory(categoryAxis, data)) {
+    return;
+  } // Otherwise follow the label interval strategy on category axis.
+
+
+  var categoryDataDim = data.mapDimension(categoryAxis.dim);
+  var labelMap = {};
+  zrUtil.each(categoryAxis.getViewLabels(), function (labelItem) {
+    labelMap[labelItem.tickValue] = 1;
+  });
+  return function (dataIndex) {
+    return !labelMap.hasOwnProperty(data.get(categoryDataDim, dataIndex));
+  };
+}
+
+function canShowAllSymbolForCategory(categoryAxis, data) {
+  // In mose cases, line is monotonous on category axis, and the label size
+  // is close with each other. So we check the symbol size and some of the
+  // label size alone with the category axis to estimate whether all symbol
+  // can be shown without overlap.
+  var axisExtent = categoryAxis.getExtent();
+  var availSize = Math.abs(axisExtent[1] - axisExtent[0]) / categoryAxis.scale.count();
+  isNaN(availSize) && (availSize = 0); // 0/0 is NaN.
+  // Sampling some points, max 5.
+
+  var dataLen = data.count();
+  var step = Math.max(1, Math.round(dataLen / 5));
+
+  for (var dataIndex = 0; dataIndex < dataLen; dataIndex += step) {
+    if (SymbolClz.getSymbolSize(data, dataIndex // Only for cartesian, where `isHorizontal` exists.
+    )[categoryAxis.isHorizontal() ? 1 : 0] // Empirical number
+    * 1.5 > availSize) {
+      return false;
+    }
+  }
+
+  return true;
+}
+
 var _default = ChartView.extend({
   type: 'line',
   init: function () {
@@ -315,9 +410,7 @@ var _default = ChartView.extend({
     var dataCoordInfo = prepareDataCoordInfo(coordSys, data, valueOrigin);
     var stackedOnPoints = getStackedOnPoints(coordSys, data, dataCoordInfo);
     var showSymbol = seriesModel.get('showSymbol');
-
-    var isSymbolIgnore = showSymbol && !isCoordSysPolar && !seriesModel.get('showAllSymbol') && this._getSymbolIgnoreFunc(data, coordSys); // Remove temporary symbols
-
+    var isIgnoreFunc = showSymbol && !isCoordSysPolar && getIsIgnoreFunc(seriesModel, data, coordSys); // Remove temporary symbols
 
     var oldData = this._data;
     oldData && oldData.eachItemGraphicEl(function (el, idx) {
@@ -337,8 +430,8 @@ var _default = ChartView.extend({
 
     if (!(polyline && prevCoordSys.type === coordSys.type && step === this._step)) {
       showSymbol && symbolDraw.updateData(data, {
-        isIgnore: isSymbolIgnore,
-        clipShape: createClipShape(coordSys, false, seriesModel)
+        isIgnore: isIgnoreFunc,
+        clipShape: createClipShape(coordSys, false, true, seriesModel)
       });
 
       if (step) {
@@ -353,7 +446,7 @@ var _default = ChartView.extend({
         polygon = this._newPolygon(points, stackedOnPoints, coordSys, hasAnimation);
       }
 
-      lineGroup.setClipPath(createClipShape(coordSys, true, seriesModel));
+      lineGroup.setClipPath(createClipShape(coordSys, true, false, seriesModel));
     } else {
       if (isAreaChart && !polygon) {
         // If areaStyle is added
@@ -362,16 +455,15 @@ var _default = ChartView.extend({
         // If areaStyle is removed
         lineGroup.remove(polygon);
         polygon = this._polygon = null;
-      }
+      } // Update clipPath
 
-      var coordSysClipShape = createClipShape(coordSys, false, seriesModel); // Update clipPath
 
-      lineGroup.setClipPath(coordSysClipShape); // Always update, or it is wrong in the case turning on legend
+      lineGroup.setClipPath(createClipShape(coordSys, false, false, seriesModel)); // Always update, or it is wrong in the case turning on legend
       // because points are not changed
 
       showSymbol && symbolDraw.updateData(data, {
-        isIgnore: isSymbolIgnore,
-        clipShape: coordSysClipShape
+        isIgnore: isIgnoreFunc,
+        clipShape: createClipShape(coordSys, false, true, seriesModel)
       }); // Stop symbol animation and sync with line points
       // FIXME performance?
 
@@ -556,17 +648,6 @@ var _default = ChartView.extend({
     return polygon;
   },
 
-  /**
-   * @private
-   */
-  _getSymbolIgnoreFunc: function (data, coordSys) {
-    var categoryAxis = coordSys.getAxesByScale('ordinal')[0]; // `getLabelInterval` is provided by echarts/component/axis
-
-    if (categoryAxis && categoryAxis.isLabelIgnored) {
-      return zrUtil.bind(categoryAxis.isLabelIgnored, categoryAxis);
-    }
-  },
-
   /**
    * @private
    */

package/lib/chart/line/helper.js

@@ -6,6 +6,25 @@ var _util = require("zrender/lib/core/util");
 
 var map = _util.map;
 
+/*
+* Licensed to the Apache Software Foundation (ASF) under one
+* or more contributor license agreements.  See the NOTICE file
+* distributed with this work for additional information
+* regarding copyright ownership.  The ASF licenses this file
+* to you under the Apache License, Version 2.0 (the
+* "License"); you may not use this file except in compliance
+* with the License.  You may obtain a copy of the License at
+*
+*   http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing,
+* software distributed under the License is distributed on an
+* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+* KIND, either express or implied.  See the License for the
+* specific language governing permissions and limitations
+* under the License.
+*/
+
 /**
  * @param {Object} coordSys
  * @param {module:echarts/data/List} data
@@ -20,16 +39,32 @@ function prepareDataCoordInfo(coordSys, data, valueOrigin) {
   var valueDim = data.mapDimension(valueAxisDim);
   var baseDim = data.mapDimension(baseAxisDim);
   var baseDataOffset = valueAxisDim === 'x' || valueAxisDim === 'radius' ? 1 : 0;
-  var stacked = isDimensionStacked(data, valueDim, baseDim);
-  var dataDimsForPoint = map(coordSys.dimensions, function (coordDim) {
+  var dims = map(coordSys.dimensions, function (coordDim) {
     return data.mapDimension(coordDim);
   });
+  var stacked;
+  var stackResultDim = data.getCalculationInfo('stackResultDimension');
+
+  if (stacked |= isDimensionStacked(data, dims[0]
+  /*, dims[1]*/
+  )) {
+    // jshint ignore:line
+    dims[0] = stackResultDim;
+  }
+
+  if (stacked |= isDimensionStacked(data, dims[1]
+  /*, dims[0]*/
+  )) {
+    // jshint ignore:line
+    dims[1] = stackResultDim;
+  }
+
   return {
-    dataDimsForPoint: dataDimsForPoint,
+    dataDimsForPoint: dims,
     valueStart: valueStart,
     valueAxisDim: valueAxisDim,
     baseAxisDim: baseAxisDim,
-    stacked: stacked,
+    stacked: !!stacked,
     valueDim: valueDim,
     baseDim: baseDim,
     baseDataOffset: baseDataOffset,

package/lib/chart/line/lineAnimationDiff.js

@@ -3,6 +3,24 @@ var _helper = require("./helper");
 var prepareDataCoordInfo = _helper.prepareDataCoordInfo;
 var getStackedOnPoint = _helper.getStackedOnPoint;
 
+/*
+* Licensed to the Apache Software Foundation (ASF) under one
+* or more contributor license agreements.  See the NOTICE file
+* distributed with this work for additional information
+* regarding copyright ownership.  The ASF licenses this file
+* to you under the Apache License, Version 2.0 (the
+* "License"); you may not use this file except in compliance
+* with the License.  You may obtain a copy of the License at
+*
+*   http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing,
+* software distributed under the License is distributed on an
+* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+* KIND, either express or implied.  See the License for the
+* specific language governing permissions and limitations
+* under the License.
+*/
 // var arrayDiff = require('zrender/src/core/arrayDiff');
 // 'zrender/src/core/arrayDiff' has been used before, but it did
 // not do well in performance when roam with fixed dataZoom window.

package/lib/chart/line/poly.js

@@ -4,6 +4,24 @@ var vec2 = require("zrender/lib/core/vector");
 
 var fixClipWithShadow = require("zrender/lib/graphic/helper/fixClipWithShadow");
 
+/*
+* Licensed to the Apache Software Foundation (ASF) under one
+* or more contributor license agreements.  See the NOTICE file
+* distributed with this work for additional information
+* regarding copyright ownership.  The ASF licenses this file
+* to you under the Apache License, Version 2.0 (the
+* "License"); you may not use this file except in compliance
+* with the License.  You may obtain a copy of the License at
+*
+*   http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing,
+* software distributed under the License is distributed on an
+* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+* KIND, either express or implied.  See the License for the
+* specific language governing permissions and limitations
+* under the License.
+*/
 // Poly path support NaN point
 var vec2Min = vec2.min;
 var vec2Max = vec2.max;
@@ -19,18 +37,29 @@ function isPointNull(p) {
 }
 
 function drawSegment(ctx, points, start, segLen, allLen, dir, smoothMin, smoothMax, smooth, smoothMonotone, connectNulls) {
-  if (smoothMonotone == null) {
-    if (isMono(points, 'x')) {
-      return drawMono(ctx, points, start, segLen, allLen, dir, smoothMin, smoothMax, smooth, 'x', connectNulls);
-    } else if (isMono(points, 'y')) {
-      return drawMono(ctx, points, start, segLen, allLen, dir, smoothMin, smoothMax, smooth, 'y', connectNulls);
-    } else {
-      return drawNonMono.apply(this, arguments);
-    }
-  } else if (smoothMonotone !== 'none' && isMono(points, smoothMonotone)) {
-    return drawMono.apply(this, arguments);
-  } else {
+  // if (smoothMonotone == null) {
+  //     if (isMono(points, 'x')) {
+  //         return drawMono(ctx, points, start, segLen, allLen,
+  //             dir, smoothMin, smoothMax, smooth, 'x', connectNulls);
+  //     }
+  //     else if (isMono(points, 'y')) {
+  //         return drawMono(ctx, points, start, segLen, allLen,
+  //             dir, smoothMin, smoothMax, smooth, 'y', connectNulls);
+  //     }
+  //     else {
+  //         return drawNonMono.apply(this, arguments);
+  //     }
+  // }
+  // else if (smoothMonotone !== 'none' && isMono(points, smoothMonotone)) {
+  //     return drawMono.apply(this, arguments);
+  // }
+  // else {
+  //     return drawNonMono.apply(this, arguments);
+  // }
+  if (smoothMonotone === 'none' || !smoothMonotone) {
     return drawNonMono.apply(this, arguments);
+  } else {
+    return drawMono.apply(this, arguments);
   }
 }
 /**
@@ -44,32 +73,29 @@ function drawSegment(ctx, points, start, segLen, allLen, dir, smoothMin, smoothM
  *                                    If is undefined, either being monotone
  *                                    in 'x' or 'y' will call `drawMono`.
  */
+// function isMono(points, smoothMonotone) {
+//     if (points.length <= 1) {
+//         return true;
+//     }
+//     var dim = smoothMonotone === 'x' ? 0 : 1;
+//     var last = points[0][dim];
+//     var lastDiff = 0;
+//     for (var i = 1; i < points.length; ++i) {
+//         var diff = points[i][dim] - last;
+//         if (!isNaN(diff) && !isNaN(lastDiff)
+//             && diff !== 0 && lastDiff !== 0
+//             && ((diff >= 0) !== (lastDiff >= 0))
+//         ) {
+//             return false;
+//         }
+//         if (!isNaN(diff) && diff !== 0) {
+//             lastDiff = diff;
+//             last = points[i][dim];
+//         }
+//     }
+//     return true;
+// }
 
-
-function isMono(points, smoothMonotone) {
-  if (points.length <= 1) {
-    return true;
-  }
-
-  var dim = smoothMonotone === 'x' ? 0 : 1;
-  var last = points[0][dim];
-  var lastDiff = 0;
-
-  for (var i = 1; i < points.length; ++i) {
-    var diff = points[i][dim] - last;
-
-    if (!isNaN(diff) && !isNaN(lastDiff) && diff !== 0 && lastDiff !== 0 && diff >= 0 !== lastDiff >= 0) {
-      return false;
-    }
-
-    if (!isNaN(diff) && diff !== 0) {
-      lastDiff = diff;
-      last = points[i][dim];
-    }
-  }
-
-  return true;
-}
 /**
  * Draw smoothed line in monotone, in which only vertical or horizontal bezier
  * control points will be used. This should be used when points are monotone

package/lib/chart/line.js

@@ -12,6 +12,24 @@ var dataSample = require("../processor/dataSample");
 
 require("../component/gridSimple");
 
+/*
+* Licensed to the Apache Software Foundation (ASF) under one
+* or more contributor license agreements.  See the NOTICE file
+* distributed with this work for additional information
+* regarding copyright ownership.  The ASF licenses this file
+* to you under the Apache License, Version 2.0 (the
+* "License"); you may not use this file except in compliance
+* with the License.  You may obtain a copy of the License at
+*
+*   http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing,
+* software distributed under the License is distributed on an
+* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+* KIND, either express or implied.  See the License for the
+* specific language governing permissions and limitations
+* under the License.
+*/
 // In case developer forget to include grid component
 echarts.registerVisual(visualSymbol('line', 'circle', 'line'));
 echarts.registerLayout(layoutPoints('line')); // Down sample after filter

package/lib/chart/lines/LinesSeries.js

@@ -18,9 +18,26 @@ var encodeHTML = _format.encodeHTML;
 
 var CoordinateSystem = require("../../CoordinateSystem");
 
-var globalObj = typeof window === 'undefined' ? global : window;
-var Uint32Arr = globalObj.Uint32Array || Array;
-var Float64Arr = globalObj.Float64Array || Array;
+/*
+* Licensed to the Apache Software Foundation (ASF) under one
+* or more contributor license agreements.  See the NOTICE file
+* distributed with this work for additional information
+* regarding copyright ownership.  The ASF licenses this file
+* to you under the Apache License, Version 2.0 (the
+* "License"); you may not use this file except in compliance
+* with the License.  You may obtain a copy of the License at
+*
+*   http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing,
+* software distributed under the License is distributed on an
+* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+* KIND, either express or implied.  See the License for the
+* specific language governing permissions and limitations
+* under the License.
+*/
+var Uint32Arr = typeof Uint32Array === 'undefined' ? Array : Uint32Array;
+var Float64Arr = typeof Float64Array === 'undefined' ? Array : Float64Array;
 
 function compatEc2(seriesOpt) {
   var data = seriesOpt.data;
@@ -50,7 +67,9 @@ var LinesSeries = SeriesModel.extend({
   dependencies: ['grid', 'polar'],
   visualColorAccessPath: 'lineStyle.color',
   init: function (option) {
-    // Not using preprocessor because mergeOption may not have series.type
+    // The input data may be null/undefined.
+    option.data = option.data || []; // Not using preprocessor because mergeOption may not have series.type
+
     compatEc2(option);
 
     var result = this._processFlatCoordsArray(option.data);
@@ -65,6 +84,8 @@ var LinesSeries = SeriesModel.extend({
     LinesSeries.superApply(this, 'init', arguments);
   },
   mergeOption: function (option) {
+    // The input data may be null/undefined.
+    option.data = option.data || [];
     compatEc2(option);
 
     if (option.data) {

package/lib/chart/lines/LinesView.js

@@ -18,6 +18,24 @@ var LargeLineDraw = require("../helper/LargeLineDraw");
 
 var linesLayout = require("./linesLayout");
 
+/*
+* Licensed to the Apache Software Foundation (ASF) under one
+* or more contributor license agreements.  See the NOTICE file
+* distributed with this work for additional information
+* regarding copyright ownership.  The ASF licenses this file
+* to you under the Apache License, Version 2.0 (the
+* "License"); you may not use this file except in compliance
+* with the License.  You may obtain a copy of the License at
+*
+*   http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing,
+* software distributed under the License is distributed on an
+* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+* KIND, either express or implied.  See the License for the
+* specific language governing permissions and limitations
+* under the License.
+*/
 var _default = echarts.extendChartView({
   type: 'lines',
   init: function () {},
@@ -76,8 +94,9 @@ var _default = echarts.extendChartView({
   },
   updateTransform: function (seriesModel, ecModel, api) {
     var data = seriesModel.getData();
+    var pipelineContext = seriesModel.pipelineContext;
 
-    if (!this._finished || seriesModel.pipelineContext.large) {
+    if (!this._finished || pipelineContext.large || pipelineContext.progressiveRender) {
       // TODO Don't have to do update in large mode. Only do it when there are millions of data.
       return {
         update: true

package/lib/chart/lines/linesLayout.js

@@ -1,5 +1,23 @@
 var createRenderPlanner = require("../helper/createRenderPlanner");
 
+/*
+* Licensed to the Apache Software Foundation (ASF) under one
+* or more contributor license agreements.  See the NOTICE file
+* distributed with this work for additional information
+* regarding copyright ownership.  The ASF licenses this file
+* to you under the Apache License, Version 2.0 (the
+* "License"); you may not use this file except in compliance
+* with the License.  You may obtain a copy of the License at
+*
+*   http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing,
+* software distributed under the License is distributed on an
+* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+* KIND, either express or implied.  See the License for the
+* specific language governing permissions and limitations
+* under the License.
+*/
 var _default = {
   seriesType: 'lines',
   plan: createRenderPlanner(),
@@ -24,7 +42,7 @@ var _default = {
 
           points = new Float32Array(segCount + totalCoordsCount * 2);
         } else {
-          points = new Float32Array(segCount * 2);
+          points = new Float32Array(segCount * 4);
         }
 
         var offset = 0;

package/lib/chart/lines/linesVisual.js

@@ -1,3 +1,21 @@
+/*
+* Licensed to the Apache Software Foundation (ASF) under one
+* or more contributor license agreements.  See the NOTICE file
+* distributed with this work for additional information
+* regarding copyright ownership.  The ASF licenses this file
+* to you under the Apache License, Version 2.0 (the
+* "License"); you may not use this file except in compliance
+* with the License.  You may obtain a copy of the License at
+*
+*   http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing,
+* software distributed under the License is distributed on an
+* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+* KIND, either express or implied.  See the License for the
+* specific language governing permissions and limitations
+* under the License.
+*/
 function normalize(a) {
   if (!(a instanceof Array)) {
     a = [a, a];

package/lib/chart/lines.js

@@ -8,5 +8,23 @@ var linesLayout = require("./lines/linesLayout");
 
 var linesVisual = require("./lines/linesVisual");
 
+/*
+* Licensed to the Apache Software Foundation (ASF) under one
+* or more contributor license agreements.  See the NOTICE file
+* distributed with this work for additional information
+* regarding copyright ownership.  The ASF licenses this file
+* to you under the Apache License, Version 2.0 (the
+* "License"); you may not use this file except in compliance
+* with the License.  You may obtain a copy of the License at
+*
+*   http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing,
+* software distributed under the License is distributed on an
+* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+* KIND, either express or implied.  See the License for the
+* specific language governing permissions and limitations
+* under the License.
+*/
 echarts.registerLayout(linesLayout);
 echarts.registerVisual(linesVisual);