non-indexed column chart are not working if not starting at x=0 and having values greater than the data array length

[cjolif]

With #15308 was introduced in Dojo 1.8 the ability to render non-indexed column plots. However if the first value of the series is at x != 0 and having values greater than the data array length the rendering is incorrect.

[cjolif]

test case that does reproduce the issue

[cjolif]

This comes from the fact the for loop on the values in Columns.js starts with min that is the first x-value in the data but last for getDataLength() which is (if the data array length is less than the maximum value of the data) the number of items in the data not the max - min x-value.

[Mathevet julien]

fixed bar ?

[Mathevet julien]

I have differents fix in my dojox source repository. I will try to extract patch that fix this.

Is this the expected graph ? (because I am not sure you also need offset to center bar on x value)

[Mathevet julien]

I think it's also due to bar width, see #15477

Could you try to apply fix in #15477, thank's

[cjolif]

Well bar width is for me a different "issue". With keeping bar width as it is the expected result is the following (ignoring the values on the x-axis):

[Mathevet julien]

We don't need to use min and max because we iterate over data elements

[cjolif]

thanks moogle, I'm going to look at your patch (CLA). Also I suspect the same issue exist on Bar plots as well.

[Mathevet julien]

Sorry, I updated patch

[cjolif]

Thanks a lot for the updated patch.

I see two other "issues":

• StackedColumns/StackedBars now have a getDataLength() method which is never called (and a thus useless maxRunLength variable).
• we have no more optimization in Columns/Bars chart for a chart that is zoomed. The purpose of the min/max was on an indexed chart that is zoomed to avoid iterating over all the values for nothing but just iterate from the first to the last visible value. This is obviously not working on non indexed as we have seen, but I suspect this is not a good idea to remove that optimization for ones that can benefit from it (and maybe we can make everyone benefit from it).

[Mathevet julien]

Last update skip data outside axis range.

[cjolif]

moogle, thanks for your latest patch it seems it wants to accommodate the optimization to non-indexed series as well as to Default (line) plots. At this is meant to be committed in 1.8 branch I want to keep the changes minimal and will take inspiration in your patch to create a simpler version. We should however keep that in mind for 1.9.

[cjolif]

In [29615]:

refs #15915. Fixes an issue with non-indexed bar/columns plots while keeping indexed plots optimization for zoomed charts. Based on moogle (CLA) patch.

[cjolif]

Fixed in trunk, I will commit in 1.8 branch tomorrow if no regression or problem on this patch are reported by then.

[cjolif]

In [29616]:

fixes #15915. Backport to 1.8.x fixes for issue with non-indexed bar/columns plots while keeping indexed plots optimization for zoomed charts. Based on moogle (CLA) patch.