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

Multiple Series error in svg rendering [HighStock -> Highchart] #5892

Closed
avdaredevil opened this Issue Oct 27, 2016 · 3 comments

Comments

Projects
None yet
4 participants
@avdaredevil

avdaredevil commented Oct 27, 2016

Expected behaviour

Spline plot added multiple series, and as I append data to each series everything should run smoothly

Actual behaviour

Huge performance hit, and these errors (in counts of hundreds):

highstock.js:54 Error: <path> attribute d: Expected number, "….14627380267771 C 315.0616246498…".

Live demo with steps to reproduce

Go here: https://avdaredevil.github.io/highcharts-chart/

  • Scroll to the first chart (spline chart).
  • Enable Real-Time
  • Open your dev console
  • Click the center of the chart multiple times (6-7 times)
  • Click Add Data Point a few times
  • Notice all the errors

My Conjecture: It is caused when the series start using the pushData over addData [API: Implementation is to type app.$.splineChart.pushData in your console on the page]. ie. When the series[2..n] shift add data rather than appending it

Affected browser(s)

No idea, just tested on chrome.

@TorsteinHonsi

This comment has been minimized.

Show comment
Hide comment
@TorsteinHonsi

TorsteinHonsi Oct 27, 2016

Collaborator

Thanks for reporting, but pushData is not a Highcharts API. Can you reproduce this issue outside your wrapper environment? The best starting point would be if you can modify the featured sample to showcase your problem.

Collaborator

TorsteinHonsi commented Oct 27, 2016

Thanks for reporting, but pushData is not a Highcharts API. Can you reproduce this issue outside your wrapper environment? The best starting point would be if you can modify the featured sample to showcase your problem.

@LithMage

This comment has been minimized.

Show comment
Hide comment
@LithMage

LithMage Nov 23, 2016

Made an example since i am getting same (or very similar) error. It happens when you try to add data point by point and doesnt show any errors if data is loaded as one array (not in example).

In my case both data points and series are added dynamically
JsFidle just press button and look into console

LithMage commented Nov 23, 2016

Made an example since i am getting same (or very similar) error. It happens when you try to add data point by point and doesnt show any errors if data is loaded as one array (not in example).

In my case both data points and series are added dynamically
JsFidle just press button and look into console

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Nov 24, 2016

Contributor

Thanks @LithMage !

I have simplified your demo, replacing AJAX with random data and redrawing chart only once, after all modifications: http://jsfiddle.net/tueqcd4r/3/

Issue still exists. It's caused by animation, when disabled it works fine: http://jsfiddle.net/tueqcd4r/1/

Edit:
It's general issue, not only for Highstock, demo: http://jsfiddle.net/tueqcd4r/5/

Internal note:
Something is wrong in function append(arr, other) { ... } method, standalone demo: http://jsfiddle.net/f50w12a2/

Contributor

pawelfus commented Nov 24, 2016

Thanks @LithMage !

I have simplified your demo, replacing AJAX with random data and redrawing chart only once, after all modifications: http://jsfiddle.net/tueqcd4r/3/

Issue still exists. It's caused by animation, when disabled it works fine: http://jsfiddle.net/tueqcd4r/1/

Edit:
It's general issue, not only for Highstock, demo: http://jsfiddle.net/tueqcd4r/5/

Internal note:
Something is wrong in function append(arr, other) { ... } method, standalone demo: http://jsfiddle.net/f50w12a2/

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