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

Issue with logarithmic axis #6021

Closed
pliasetski opened this issue Nov 23, 2016 · 4 comments
Closed

Issue with logarithmic axis #6021

pliasetski opened this issue Nov 23, 2016 · 4 comments
Labels

Comments

@pliasetski
Copy link

@pliasetski pliasetski commented Nov 23, 2016

Hello,
Series that belongs for bottom axis is stucked to the left. Bottom axis is logarithmic. When it set it to linear it's workes correct.

Live demo with steps to reproduce

http://jsfiddle.net/Pashe4ka/xkx1mgtc/2/

Affected browser(s)

Latest version of Chrome

@pawelfus
Copy link
Contributor

@pawelfus pawelfus commented Nov 23, 2016

Hi @pliasetski

Thanks, but I think you may have misconfigured the demo. Series 2 & 3 are connected to the yAxis = 1, which is the top yAxis. The scale is correct, because Series 3 has quite high values when comparing to the Series 2. If you hide Series 3, you will see the Series 2 rescaled.

Let me know if I missed something.

@oysteinmoseng
Copy link
Member

@oysteinmoseng oysteinmoseng commented Nov 23, 2016

I think I see the issue, looks like the bottom axis gets NaN for all ticks somehow. Simple example: http://jsfiddle.net/vm8xeg3s/.

Workaround: Set chart.alignTicks: false (http://jsfiddle.net/xkx1mgtc/3/)

@pliasetski
Copy link
Author

@pliasetski pliasetski commented Nov 23, 2016

Hi @pawelfus
The problem is with Series 1. It is stucked to the left. It is only one Series that connected to the yAxis = 0, so why it isn't displayed on the entire chart.

@pawelfus
Copy link
Contributor

@pawelfus pawelfus commented Nov 23, 2016

Ha! @oysteinmoseng is right, I tested this in a small window, like this: http://jsfiddle.net/xkx1mgtc/4/ - when it works fine :)

Internal note:
I guess, alignToOthers() should also consider axis type. Otherwise it tried to set 1800 as a base for Math.pow() which results as Infinity/NaN

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

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.