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

Bubble series with 'sizeByAbsoluteValue' and similar 'z' values shows empty chart #8092

Closed
ppotaczek opened this issue Mar 28, 2018 · 3 comments
Closed
Assignees
Labels

Comments

@ppotaczek
Copy link
Contributor

@ppotaczek ppotaczek commented Mar 28, 2018

Expected behaviour

Chart should render independently from 'z' value.
https://jsfiddle.net/BlackLabel/ktp0f7nL/

Actual behaviour

If 'z' data values are different in two points and the difference of these values is not greater than approximately 0.1, the chart does not render.

Live demo with steps to reproduce

https://jsfiddle.net/BlackLabel/f6mb4ubt/

Product version

Highcharts v6.0.7

Affected browser(s)

All

@sebastianbochan
Copy link
Contributor

@sebastianbochan sebastianbochan commented Mar 28, 2018

Workaround
Set the zThreshold parameter.

Demo:

@pawelfus pawelfus added the Type: Bug label Mar 28, 2018
@denbald
Copy link

@denbald denbald commented Mar 28, 2018

Looks like that the problem is in this piece of code in getRadii method:

// When sizing by threshold, the absolute value of z determines the size
// of the bubble.
if (options.sizeByAbsoluteValue && value !== null) {
    value = Math.abs(value - zThreshold);
    zMax = Math.max(zMax - zThreshold, Math.abs(zMin - zThreshold));
    zMin = 0;       
}

After update zMax and zMin values, should we not also update zRange?

@denbald
Copy link

@denbald denbald commented Mar 28, 2018

Workaround
Set the zThreshold parameter.

This won't work if difference in z values is 0.01

@pawelfus pawelfus self-assigned this Apr 6, 2018
pawelfus added a commit that referenced this issue Apr 6, 2018
KacperMadej added a commit that referenced this issue Apr 9, 2018
TorsteinHonsi added a commit that referenced this issue Apr 10, 2018
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
4 participants