fix(line): using sampling with lttb, NaN point should not be filtered.

[fuchunhui]

…. close #14689

Brief Information

This pull request is in the type of:

• bug fixing
• new feature
• others

What does this PR do?

Fixed issues

Details

Before: What was the problem?

After: How is it fixed in this PR?

store NaN point, append first to every bucket, adjust Uint Array length.

Misc

• The API has been changed (apache/echarts-doc#xxx).
• This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs

N.A.

Others

Merging options

• Please squash the commits into a single one when merging.

Other information

[echarts-bot]

Thanks for your contribution!
The community will review it ASAP. In the meanwhile, please checkout the coding standard and Wiki about How to make a pull request.

[pissang]

Seems when I'm running visual regression test(which has roslution 830x750), the current behavior seems to having unexpected points in the null values.

Before	After

Also I think we can add the case from https://codepen.io/tmscer/pen/yLgGXbo directly.

[fuchunhui]

Seems when I'm running visual regression test(which has roslution 830x750), the current behavior seems to having unexpected points in the null values.

Before After

Also I think we can add the case from https://codepen.io/tmscer/pen/yLgGXbo directly.

fixed. Due to incorrect index order.

[fuchunhui]

// set frameSize = 3；
const dimStore = [21, 22, 20, 33, 44, 55]; 
// IF result is [1, 5]

Demo 1：

const dimStore = [21, null, 20, 33, 44, 55]; 
// Right result is [0, 1, 5]
// Before, it would be [1, 0, 5], not right, this made the wrong case as you mentioned

@pissang

[echarts-bot]

Congratulations! Your PR has been merged. Thanks for your contribution! 👍