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

addSeries() combined with boost.js throws console errors. #7499

Closed
Denyllon opened this Issue Dec 7, 2017 · 4 comments

Comments

Projects
None yet
3 participants
@Denyllon
Contributor

Denyllon commented Dec 7, 2017

Actual behaviour

While using addSeries() combined with boost.js module there are errors throwed in console, and one of added series isn't visible.

Might be related to 7309

Live demo with steps to reproduce

https://jsfiddle.net/daniel_s/vgoq9f4r/

Occurence

v6.0.0 and above

@Denyllon Denyllon changed the title from addSeries() combined with boost.js throwing console errors. to addSeries() combined with boost.js throws console errors. Dec 7, 2017

@KacperMadej

This comment has been minimized.

Show comment
Hide comment
@KacperMadej
Contributor

KacperMadej commented Dec 8, 2017

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Dec 11, 2017

Collaborator

First of all, I don't know if it resolves the actual use case, but the demo doesn't adhere to best practice. In stead of removing the two series, setting new data to the option set and add new series, it is recommended to use Series.setData. This also resolves the issue with the console error: https://jsfiddle.net/highcharts/vgoq9f4r/1/.

Collaborator

TorsteinHonsi commented Dec 11, 2017

First of all, I don't know if it resolves the actual use case, but the demo doesn't adhere to best practice. In stead of removing the two series, setting new data to the option set and add new series, it is recommended to use Series.setData. This also resolves the issue with the console error: https://jsfiddle.net/highcharts/vgoq9f4r/1/.

@TorsteinHonsi TorsteinHonsi added the Bug label Dec 11, 2017

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi
Collaborator

TorsteinHonsi commented Dec 11, 2017

@TorsteinHonsi TorsteinHonsi self-assigned this Dec 11, 2017

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Dec 11, 2017

Collaborator

Debugging so far points to the shouldForceChartSeriesBoosting function. If first returns false as expected, but when adding the second series it returns true, which is not expected as we only have two series each with four points in the chart. Will discuss this with @cvasseng.

Collaborator

TorsteinHonsi commented Dec 11, 2017

Debugging so far points to the shouldForceChartSeriesBoosting function. If first returns false as expected, but when adding the second series it returns true, which is not expected as we only have two series each with four points in the chart. Will discuss this with @cvasseng.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment