100pah edited a comment on issue #13678: URL: https://github.com/apache/incubator-echarts/issues/13678#issuecomment-757116526
 To fix this issue, we might: + See `LogScale['unionExtentFromData']`: we might add param to the method `List['getApproximateExtent']` and `List['getDataExtent']` to union data without that `<= 0`. Otherwise the logScale extent will not be correct (e.g., [-Infinity, 10]). + Either + In `BarView`, check wether the el has invalid layout (has NaN) and mark as `ignore`. + In `zrender`, if the Path has invalid shape params (NaN), do not render them. + Fix the animation that from a NaN prop. Test case: ```js require([ 'echarts' ], function (echarts) { var data = [0, 1, 10, 100, 1000]; var option = { xAxis: { type: 'log' }, yAxis: { type: 'category', data: ['a', 'b', 'c', 'd', 'e'] }, series: [{ type: 'bar', data: data, label: { show: true, position: 'right' } }] }; var chart = testHelper.create(echarts, 'main1', { title: [ 'click "all +10"', 'check the x extent should be correct', 'the 0 bar should be displayed with animation' ], option: option, buttons: [{ text: 'all +10', onclick: function () { for (var i = 0; i < data.length; i++) { data[i] += 10; } chart.setOption({ series: { data: data } }); } }] }); }); ``` @pissang What's your opinion? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
