You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@echarts.apache.org by sh...@apache.org on 2019/10/30 17:17:56 UTC

[incubator-echarts] branch master updated: feat(clip): candlestick add clip option (#11529)

This is an automated email from the ASF dual-hosted git repository.

shenyi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-echarts.git


The following commit(s) were added to refs/heads/master by this push:
     new 17f7bfd  feat(clip): candlestick add clip option (#11529)
17f7bfd is described below

commit 17f7bfd02692ee545c3643468f199e40081e828d
Author: Yi Shen <bm...@gmail.com>
AuthorDate: Thu Oct 31 01:17:27 2019 +0800

    feat(clip): candlestick add clip option (#11529)
    
    * feat(clip): candlestick add clip option
    
    * a small performance optimization on the clipping of candlestick.
---
 src/chart/candlestick/CandlestickSeries.js     |   2 +
 src/chart/candlestick/CandlestickView.js       |  45 ++-
 src/chart/helper/createClipPathFromCoordSys.js |   6 +-
 src/coord/cartesian/Cartesian2D.js             |  13 -
 test/clip.html                                 | 373 +++++++++++++++++++++++++
 test/runTest/actions/__meta__.json             |   2 +-
 test/runTest/actions/clip.json                 |   2 +-
 7 files changed, 426 insertions(+), 17 deletions(-)

diff --git a/src/chart/candlestick/CandlestickSeries.js b/src/chart/candlestick/CandlestickSeries.js
index 52511e7..4276643 100644
--- a/src/chart/candlestick/CandlestickSeries.js
+++ b/src/chart/candlestick/CandlestickSeries.js
@@ -59,6 +59,8 @@ var CandlestickSeries = SeriesModel.extend({
 
         layout: null, // 'horizontal' or 'vertical'
 
+        clip: true,
+
         itemStyle: {
             color: '#c23531', // 阳线 positive
             color0: '#314656', // 阴线 negative     '#c23531', '#314656'
diff --git a/src/chart/candlestick/CandlestickView.js b/src/chart/candlestick/CandlestickView.js
index 1556c86..de5f19e 100644
--- a/src/chart/candlestick/CandlestickView.js
+++ b/src/chart/candlestick/CandlestickView.js
@@ -21,17 +21,20 @@ import * as zrUtil from 'zrender/src/core/util';
 import ChartView from '../../view/Chart';
 import * as graphic from '../../util/graphic';
 import Path from 'zrender/src/graphic/Path';
+import {createClipPath} from '../helper/createClipPathFromCoordSys';
 
 var NORMAL_ITEM_STYLE_PATH = ['itemStyle'];
 var EMPHASIS_ITEM_STYLE_PATH = ['emphasis', 'itemStyle'];
 var SKIP_PROPS = ['color', 'color0', 'borderColor', 'borderColor0'];
 
-
 var CandlestickView = ChartView.extend({
 
     type: 'candlestick',
 
     render: function (seriesModel, ecModel, api) {
+        // If there is clipPath created in large mode. Remove it.
+        this.group.removeClipPath();
+
         this._updateDrawMode(seriesModel);
 
         this._isLargeDraw
@@ -64,6 +67,10 @@ var CandlestickView = ChartView.extend({
         var group = this.group;
         var isSimpleBox = data.getLayout('isSimpleBox');
 
+        var needsClip = seriesModel.get('clip', true);
+        var coord = seriesModel.coordinateSystem;
+        var clipArea = coord.getArea && coord.getArea();
+
         // There is no old data only when first rendering or switching from
         // stream mode to normal mode, where previous elements should be removed.
         if (!this._data) {
@@ -76,12 +83,18 @@ var CandlestickView = ChartView.extend({
                     var el;
 
                     var itemLayout = data.getItemLayout(newIdx);
+
+                    if (needsClip && isNormalBoxClipped(clipArea, itemLayout)) {
+                        return;
+                    }
+
                     el = createNormalBox(itemLayout, newIdx, true);
                     graphic.initProps(el, {shape: {points: itemLayout.ends}}, seriesModel, newIdx);
 
                     setBoxCommon(el, data, newIdx, isSimpleBox);
 
                     group.add(el);
+
                     data.setItemGraphicEl(newIdx, el);
                 }
             })
@@ -95,6 +108,11 @@ var CandlestickView = ChartView.extend({
                 }
 
                 var itemLayout = data.getItemLayout(newIdx);
+                if (needsClip && isNormalBoxClipped(clipArea, itemLayout)) {
+                    group.remove(el);
+                    return;
+                }
+
                 if (!el) {
                     el = createNormalBox(itemLayout, newIdx);
                 }
@@ -118,7 +136,19 @@ var CandlestickView = ChartView.extend({
 
     _renderLarge: function (seriesModel) {
         this._clear();
+
         createLarge(seriesModel, this.group);
+
+        var clipPath = seriesModel.get('clip', true)
+            ? createClipPath(seriesModel.coordinateSystem, false, seriesModel)
+            : null;
+        if (clipPath) {
+            this.group.setClipPath(clipPath);
+        }
+        else {
+            this.group.removeClipPath();
+        }
+
     },
 
     _incrementalRenderNormal: function (params, seriesModel) {
@@ -184,6 +214,7 @@ var NormalBoxPath = Path.extend({
     }
 });
 
+
 function createNormalBox(itemLayout, dataIndex, isInit) {
     var ends = itemLayout.ends;
     return new NormalBoxPath({
@@ -196,6 +227,18 @@ function createNormalBox(itemLayout, dataIndex, isInit) {
     });
 }
 
+function isNormalBoxClipped(clipArea, itemLayout) {
+    var clipped = true;
+    for (var i = 0; i < itemLayout.ends.length; i++) {
+        // If any point are in the region.
+        if (clipArea.contain(itemLayout.ends[i][0], itemLayout.ends[i][1])) {
+            clipped = false;
+            break;
+        }
+    }
+    return clipped;
+}
+
 function setBoxCommon(el, data, dataIndex, isSimpleBox) {
     var itemModel = data.getItemModel(dataIndex);
     var normalItemStyleModel = itemModel.getModel(NORMAL_ITEM_STYLE_PATH);
diff --git a/src/chart/helper/createClipPathFromCoordSys.js b/src/chart/helper/createClipPathFromCoordSys.js
index 0a6fcd0..feabeb2 100644
--- a/src/chart/helper/createClipPathFromCoordSys.js
+++ b/src/chart/helper/createClipPathFromCoordSys.js
@@ -97,4 +97,8 @@ function createClipPath(coordSys, hasAnimation, seriesModel) {
     return null;
 }
 
-export {createGridClipPath, createPolarClipPath, createClipPath};
\ No newline at end of file
+export {
+    createGridClipPath,
+    createPolarClipPath,
+    createClipPath
+};
\ No newline at end of file
diff --git a/src/coord/cartesian/Cartesian2D.js b/src/coord/cartesian/Cartesian2D.js
index 55ae631..a162d2e 100644
--- a/src/coord/cartesian/Cartesian2D.js
+++ b/src/coord/cartesian/Cartesian2D.js
@@ -22,19 +22,6 @@ import * as zrUtil from 'zrender/src/core/util';
 import BoundingRect from 'zrender/src/core/BoundingRect';
 import Cartesian from './Cartesian';
 
-// A helper function to calculate extent of axis.
-function getAxisExtentWithGap(axis) {
-    var extent = axis.getGlobalExtent();
-    if (axis.onBand) {
-        // Remove extra 1px to avoid line miter in clipped edge
-        var halfBandWidth = axis.getBandWidth() / 2 - 1;
-        var dir = extent[1] > extent[0] ? 1 : -1;
-        extent[0] += dir * halfBandWidth;
-        extent[1] -= dir * halfBandWidth;
-    }
-    return extent;
-}
-
 function Cartesian2D(name) {
 
     Cartesian.call(this, name);
diff --git a/test/clip.html b/test/clip.html
index ad7dd3d..0a2ad60 100644
--- a/test/clip.html
+++ b/test/clip.html
@@ -957,5 +957,378 @@ under the License.
 
 
         <div class="chart" id="candlestick-clip"></div>
+        <script>
+            require([
+                'echarts'
+            ], function (echarts) {
+                var upColor = '#ec0000';
+                var upBorderColor = '#8A0000';
+                var downColor = '#00da3c';
+                var downBorderColor = '#008F28';
+
+                // 数据意义:开盘(open),收盘(close),最低(lowest),最高(highest)
+                var data0 = splitData([
+                    ['2013/1/24', 2320.26,2320.26,2287.3,2362.94],
+                    ['2013/1/25', 2300,2291.3,2288.26,2308.38],
+                    ['2013/1/28', 2295.35,2346.5,2295.35,2346.92],
+                    ['2013/1/29', 2347.22,2358.98,2337.35,2363.8],
+                    ['2013/1/30', 2360.75,2382.48,2347.89,2383.76],
+                    ['2013/1/31', 2383.43,2385.42,2371.23,2391.82],
+                    ['2013/2/1', 2377.41,2419.02,2369.57,2421.15],
+                    ['2013/2/4', 2425.92,2428.15,2417.58,2440.38],
+                    ['2013/2/5', 2411,2433.13,2403.3,2437.42],
+                    ['2013/2/6', 2432.68,2434.48,2427.7,2441.73],
+                    ['2013/2/7', 2430.69,2418.53,2394.22,2433.89],
+                    ['2013/2/8', 2416.62,2432.4,2414.4,2443.03],
+                    ['2013/2/18', 2441.91,2421.56,2415.43,2444.8],
+                    ['2013/2/19', 2420.26,2382.91,2373.53,2427.07],
+                    ['2013/2/20', 2383.49,2397.18,2370.61,2397.94],
+                    ['2013/2/21', 2378.82,2325.95,2309.17,2378.82],
+                    ['2013/2/22', 2322.94,2314.16,2308.76,2330.88],
+                    ['2013/2/25', 2320.62,2325.82,2315.01,2338.78],
+                    ['2013/2/26', 2313.74,2293.34,2289.89,2340.71],
+                    ['2013/2/27', 2297.77,2313.22,2292.03,2324.63],
+                    ['2013/2/28', 2322.32,2365.59,2308.92,2366.16],
+                    ['2013/3/1', 2364.54,2359.51,2330.86,2369.65],
+                    ['2013/3/4', 2332.08,2273.4,2259.25,2333.54],
+                    ['2013/3/5', 2274.81,2326.31,2270.1,2328.14],
+                    ['2013/3/6', 2333.61,2347.18,2321.6,2351.44],
+                    ['2013/3/7', 2340.44,2324.29,2304.27,2352.02],
+                    ['2013/3/8', 2326.42,2318.61,2314.59,2333.67],
+                    ['2013/3/11', 2314.68,2310.59,2296.58,2320.96],
+                    ['2013/3/12', 2309.16,2286.6,2264.83,2333.29],
+                    ['2013/3/13', 2282.17,2263.97,2253.25,2286.33],
+                    ['2013/3/14', 2255.77,2270.28,2253.31,2276.22],
+                    ['2013/3/15', 2269.31,2278.4,2250,2312.08],
+                    ['2013/3/18', 2267.29,2240.02,2239.21,2276.05],
+                    ['2013/3/19', 2244.26,2257.43,2232.02,2261.31],
+                    ['2013/3/20', 2257.74,2317.37,2257.42,2317.86],
+                    ['2013/3/21', 2318.21,2324.24,2311.6,2330.81],
+                    ['2013/3/22', 2321.4,2328.28,2314.97,2332],
+                    ['2013/3/25', 2334.74,2326.72,2319.91,2344.89],
+                    ['2013/3/26', 2318.58,2297.67,2281.12,2319.99],
+                    ['2013/3/27', 2299.38,2301.26,2289,2323.48],
+                    ['2013/3/28', 2273.55,2236.3,2232.91,2273.55],
+                    ['2013/3/29', 2238.49,2236.62,2228.81,2246.87],
+                    ['2013/4/1', 2229.46,2234.4,2227.31,2243.95],
+                    ['2013/4/2', 2234.9,2227.74,2220.44,2253.42],
+                    ['2013/4/3', 2232.69,2225.29,2217.25,2241.34],
+                    ['2013/4/8', 2196.24,2211.59,2180.67,2212.59],
+                    ['2013/4/9', 2215.47,2225.77,2215.47,2234.73],
+                    ['2013/4/10', 2224.93,2226.13,2212.56,2233.04],
+                    ['2013/4/11', 2236.98,2219.55,2217.26,2242.48],
+                    ['2013/4/12', 2218.09,2206.78,2204.44,2226.26],
+                    ['2013/4/15', 2199.91,2181.94,2177.39,2204.99],
+                    ['2013/4/16', 2169.63,2194.85,2165.78,2196.43],
+                    ['2013/4/17', 2195.03,2193.8,2178.47,2197.51],
+                    ['2013/4/18', 2181.82,2197.6,2175.44,2206.03],
+                    ['2013/4/19', 2201.12,2244.64,2200.58,2250.11],
+                    ['2013/4/22', 2236.4,2242.17,2232.26,2245.12],
+                    ['2013/4/23', 2242.62,2184.54,2182.81,2242.62],
+                    ['2013/4/24', 2187.35,2218.32,2184.11,2226.12],
+                    ['2013/4/25', 2213.19,2199.31,2191.85,2224.63],
+                    ['2013/4/26', 2203.89,2177.91,2173.86,2210.58],
+                    ['2013/5/2', 2170.78,2174.12,2161.14,2179.65],
+                    ['2013/5/3', 2179.05,2205.5,2179.05,2222.81],
+                    ['2013/5/6', 2212.5,2231.17,2212.5,2236.07],
+                    ['2013/5/7', 2227.86,2235.57,2219.44,2240.26],
+                    ['2013/5/8', 2242.39,2246.3,2235.42,2255.21],
+                    ['2013/5/9', 2246.96,2232.97,2221.38,2247.86],
+                    ['2013/5/10', 2228.82,2246.83,2225.81,2247.67],
+                    ['2013/5/13', 2247.68,2241.92,2231.36,2250.85],
+                    ['2013/5/14', 2238.9,2217.01,2205.87,2239.93],
+                    ['2013/5/15', 2217.09,2224.8,2213.58,2225.19],
+                    ['2013/5/16', 2221.34,2251.81,2210.77,2252.87],
+                    ['2013/5/17', 2249.81,2282.87,2248.41,2288.09],
+                    ['2013/5/20', 2286.33,2299.99,2281.9,2309.39],
+                    ['2013/5/21', 2297.11,2305.11,2290.12,2305.3],
+                    ['2013/5/22', 2303.75,2302.4,2292.43,2314.18],
+                    ['2013/5/23', 2293.81,2275.67,2274.1,2304.95],
+                    ['2013/5/24', 2281.45,2288.53,2270.25,2292.59],
+                    ['2013/5/27', 2286.66,2293.08,2283.94,2301.7],
+                    ['2013/5/28', 2293.4,2321.32,2281.47,2322.1],
+                    ['2013/5/29', 2323.54,2324.02,2321.17,2334.33],
+                    ['2013/5/30', 2316.25,2317.75,2310.49,2325.72],
+                    ['2013/5/31', 2320.74,2300.59,2299.37,2325.53],
+                    ['2013/6/3', 2300.21,2299.25,2294.11,2313.43],
+                    ['2013/6/4', 2297.1,2272.42,2264.76,2297.1],
+                    ['2013/6/5', 2270.71,2270.93,2260.87,2276.86],
+                    ['2013/6/6', 2264.43,2242.11,2240.07,2266.69],
+                    ['2013/6/7', 2242.26,2210.9,2205.07,2250.63],
+                    ['2013/6/13', 2190.1,2148.35,2126.22,2190.1]
+                ]);
+
+                function splitData(rawData) {
+                    var categoryData = [];
+                    var values = []
+                    for (var i = 0; i < rawData.length; i++) {
+                        categoryData.push(rawData[i].splice(0, 1)[0]);
+                        values.push(rawData[i])
+                    }
+                    return {
+                        categoryData: categoryData,
+                        values: values
+                    };
+                }
+
+                function calculateMA(dayCount) {
+                    var result = [];
+                    for (var i = 0, len = data0.values.length; i < len; i++) {
+                        if (i < dayCount) {
+                            result.push('-');
+                            continue;
+                        }
+                        var sum = 0;
+                        for (var j = 0; j < dayCount; j++) {
+                            sum += data0.values[i - j][1];
+                        }
+                        result.push(sum / dayCount);
+                    }
+                    return result;
+                }
+
+
+
+                option = {
+                    title: {
+                        text: '上证指数',
+                        left: 0
+                    },
+                    tooltip: {
+                        trigger: 'axis',
+                        axisPointer: {
+                            type: 'cross'
+                        }
+                    },
+                    legend: {
+                        data: ['日K', 'MA5', 'MA10', 'MA20', 'MA30']
+                    },
+                    grid: {
+                        left: '10%',
+                        right: '10%',
+                        bottom: '15%'
+                    },
+                    xAxis: {
+                        type: 'category',
+                        data: data0.categoryData,
+                        scale: true,
+                        boundaryGap : false,
+                        axisLine: {onZero: false},
+                        splitLine: {show: false},
+                        splitNumber: 20,
+                        min: 'dataMin',
+                        max: 'dataMax'
+                    },
+                    yAxis: {
+                        scale: true,
+                        splitArea: {
+                            show: true
+                        }
+                    },
+                    dataZoom: [
+                        {
+                            type: 'inside',
+                            filterMode: 'none',
+                            start: 50,
+                            end: 100
+                        },
+                    ],
+                    series: [
+                        {
+                            name: '日K',
+                            type: 'candlestick',
+                            data: data0.values,
+                            itemStyle: {
+                                normal: {
+                                    color: upColor,
+                                    color0: downColor,
+                                    borderColor: upBorderColor,
+                                    borderColor0: downBorderColor
+                                }
+                            }
+                        },
+                        {
+                            name: 'MA5',
+                            type: 'line',
+                            data: calculateMA(5),
+                            smooth: true,
+                            lineStyle: {
+                                normal: {opacity: 0.5}
+                            }
+                        },
+                        {
+                            name: 'MA10',
+                            type: 'line',
+                            data: calculateMA(10),
+                            smooth: true,
+                            lineStyle: {
+                                normal: {opacity: 0.5}
+                            }
+                        },
+                        {
+                            name: 'MA20',
+                            type: 'line',
+                            data: calculateMA(20),
+                            smooth: true,
+                            lineStyle: {
+                                normal: {opacity: 0.5}
+                            }
+                        },
+                        {
+                            name: 'MA30',
+                            type: 'line',
+                            data: calculateMA(30),
+                            smooth: true,
+                            lineStyle: {
+                                normal: {opacity: 0.5}
+                            }
+                        },
+
+                    ]
+                };
+
+                var chart = testHelper.create(echarts, 'candlestick-clip', {
+                    title: [
+                        'Clip candlestick',
+                        'Case from #11010'
+                    ],
+                    option: option,
+                    height: 400,
+                    buttons: makeToggleChartButtons(function (clip) {
+                        chart.setOption({
+                            series: [{
+                                clip: clip
+                            }]
+                        })
+                    })
+                });
+            });
+        </script>
+
+
+        <div class="chart" id="candlestick-large-clip"></div>
+        <script>
+            require([
+                'echarts',
+                'data/stock-DJI.json'
+            ], function (echarts, rawData) {
+                var option = {
+                    dataset: {
+                        source: rawData
+                    },
+                    backgroundColor: '#eee',
+                    legend: {
+                        left: 0,
+                    },
+                    tooltip: {
+                        trigger: 'axis',
+                        axisPointer: {
+                            type: 'cross'
+                        },
+                        backgroundColor: 'rgba(245, 245, 245, 0.8)',
+                        borderWidth: 1,
+                        borderColor: '#ccc',
+                        padding: 10,
+                        textStyle: {
+                            color: '#000'
+                        },
+                        position: function (pos, params, el, elRect, size) {
+                            var obj = {top: 10};
+                            obj[['left', 'right'][+(pos[0] < size.viewSize[0] / 2)]] = 30;
+                            return obj;
+                        }
+                    },
+                    axisPointer: {
+                        link: {xAxisIndex: 'all'},
+                        label: {
+                            backgroundColor: '#777'
+                        }
+                    },
+                    toolbox: {
+                        feature: {
+                            dataZoom: {
+                                yAxisIndex: false
+                            },
+                            brush: {
+                                type: ['polygon', 'rect', 'lineX', 'lineY', 'keep', 'clear']
+                            }
+                        }
+                    },
+                    brush: {
+                        xAxisIndex: 'all',
+                        brushLink: 'all',
+                        outOfBrush: {
+                            colorAlpha: 0.1
+                        }
+                    },
+                    grid: [
+                        {
+                            left: '10%',
+                            right: '10%',
+                            height: 300
+                        }
+                    ],
+                    xAxis: [
+                        {
+                            type: 'category',
+                            scale: true,
+                            boundaryGap : false,
+                            axisLine: {onZero: false},
+                            splitLine: {show: false},
+                            splitNumber: 20,
+                            min: 'dataMin',
+                            max: 'dataMax'
+                        }
+                    ],
+                    yAxis: [
+                        {
+                            scale: true
+                        }
+                    ],
+                    dataZoom: [
+                        {
+                            show: true,
+                            type: 'slider',
+                            filterMode: 'none',
+                            bottom: 10,
+                            start: 50
+                        }
+                    ],
+                    animation: false,
+                    series: [
+                        {
+                            name: 'Dow-Jones index',
+                            type: 'candlestick',
+                            large: true,
+                            encode: {
+                                x: 0,
+                                y: [1, 2, 3, 4]
+                            },
+                            itemStyle: {
+                                normal: {
+                                    borderColor: null,
+                                    borderColor0: null
+                                }
+                            }
+                        }
+                    ]
+                };
+
+                var chart = testHelper.create(echarts, 'candlestick-large-clip', {
+                    title: [
+                        'Clip large candlestick'
+                    ],
+                    option: option,
+                    height: 400,
+                    buttons: makeToggleChartButtons(function (clip) {
+                        chart.setOption({
+                            series: [{
+                                clip: clip
+                            }]
+                        })
+                    })
+                });
+            });
+        </script>
     </body>
 </html>
\ No newline at end of file
diff --git a/test/runTest/actions/__meta__.json b/test/runTest/actions/__meta__.json
index 42be38a..72d35ac 100644
--- a/test/runTest/actions/__meta__.json
+++ b/test/runTest/actions/__meta__.json
@@ -39,7 +39,7 @@
   "candlestick-large": 4,
   "candlestick-large2": 1,
   "candlestickConnect": 4,
-  "clip": 9,
+  "clip": 11,
   "color-mix-aqi": 1,
   "connect": 1,
   "connect-dynamic": 2,
diff --git a/test/runTest/actions/clip.json b/test/runTest/actions/clip.json
index 53e9f6e..b4246ef 100644
--- a/test/runTest/actions/clip.json
+++ b/test/runTest/actions/clip.json
@@ -1 +1 @@
-[{"name":"Action 1","ops":[{"type":"mousedown","time":447,"x":92,"y":75},{"type":"mouseup","time":559,"x":92,"y":75},{"time":560,"delay":400,"type":"screenshot-auto"},{"type":"mousemove","time":749,"x":90,"y":75},{"type":"mousemove","time":949,"x":43,"y":75},{"type":"mousemove","time":1157,"x":34,"y":75},{"type":"mousedown","time":1225,"x":34,"y":75},{"type":"mouseup","time":1322,"x":34,"y":75},{"time":1323,"delay":400,"type":"screenshot-auto"}],"scrollY":0,"scrollX":0,"timestamp":156888 [...]
\ No newline at end of file
+[{"name":"Action 1","ops":[{"type":"mousedown","time":447,"x":92,"y":75},{"type":"mouseup","time":559,"x":92,"y":75},{"time":560,"delay":400,"type":"screenshot-auto"},{"type":"mousemove","time":749,"x":90,"y":75},{"type":"mousemove","time":949,"x":43,"y":75},{"type":"mousemove","time":1157,"x":34,"y":75},{"type":"mousedown","time":1225,"x":34,"y":75},{"type":"mouseup","time":1322,"x":34,"y":75},{"time":1323,"delay":400,"type":"screenshot-auto"}],"scrollY":0,"scrollX":0,"timestamp":156888 [...]
\ No newline at end of file


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org