From 462993cfd9d9e11b95607bbdcd0e21da47ca9a3d Mon Sep 17 00:00:00 2001 From: Maxime Beauchemin Date: Thu, 26 Oct 2017 00:05:55 +0000 Subject: [PATCH] Fix box_plot NaN issue fixes https://github.com/apache/incubator-superset/issues/3712 --- superset/assets/visualizations/nvd3_vis.js | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/superset/assets/visualizations/nvd3_vis.js b/superset/assets/visualizations/nvd3_vis.js index 42fe9d255279a..69c7c64a4c996 100644 --- a/superset/assets/visualizations/nvd3_vis.js +++ b/superset/assets/visualizations/nvd3_vis.js @@ -296,9 +296,7 @@ function nvd3Vis(slice, payload) { case 'box_plot': colorKey = 'label'; chart = nv.models.boxPlotChart(); - chart.x(function (d) { - return d.label; - }); + chart.x(d => d.label); chart.staggerLabels(true); chart.maxBoxWidth(75); // prevent boxes from being incredibly wide break; @@ -353,7 +351,8 @@ function nvd3Vis(slice, payload) { chart.x2Axis.tickFormat(xAxisFormatter); height += 30; } - if (vizType !== 'dist_bar' && chart.xAxis && chart.xAxis.tickFormat) { + const isXAxisString = ['dist_bar', 'box_plot'].includes(vizType) >= 0; + if (!isXAxisString && chart.xAxis && chart.xAxis.tickFormat) { chart.xAxis.tickFormat(xAxisFormatter); }