Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Behavior of sorting function in series.AddOrderRule #61

Closed
norecces opened this issue Mar 17, 2014 · 3 comments
Closed

Behavior of sorting function in series.AddOrderRule #61

norecces opened this issue Mar 17, 2014 · 3 comments
Labels

Comments

@norecces
Copy link

Hi,
Ive noticed some strange behavior of sorting function while adding series.addOrderRule with chart with two measure axes.
Look at http://jsfiddle.net/norecces/vJaeU/
As it shows connections of points are wrong, but if I reduce amout of rows to 10, it works perfect and in sorting function loops in normal order (1-2, 2-3, ... ,9-10). In case of 14 rows order of loops is a bit strange (1-14, 14-8, ...)

@norecces norecces changed the title Behaivor of sorting function in series.AddOderRule Behaivor of sorting function in series.AddOrderRule Mar 17, 2014
@norecces norecces changed the title Behaivor of sorting function in series.AddOrderRule Behavior of sorting function in series.AddOrderRule Mar 17, 2014
@johnkiernander
Copy link
Member

It is strange, it looks like the values are treated as strings and your order rule is ignored. But on investigation I don't think it is treating them as strings, using a, b, c etc behaves in exactly the same way.

It's definitely a bug, I can't see anything wrong in your approach - I would expect it to do what you expect. Unfortunately I also can't think of a good workaround.

@norecces
Copy link
Author

Thanks for your reply,
for workaround I draw chart as a scatter plot and then manually add paths, but I hope it is not the best solution :)

johnkiernander added a commit that referenced this issue Apr 8, 2014
@johnkiernander
Copy link
Member

Fixed v2.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants