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

Call only 'Series.setData()' instead of updating whole series #86

Open
Denyllon opened this issue Jun 19, 2019 · 2 comments
Open

Call only 'Series.setData()' instead of updating whole series #86

Denyllon opened this issue Jun 19, 2019 · 2 comments
Labels
enhancement New feature or request

Comments

@Denyllon
Copy link
Contributor

It's more efficient to use only Series.setData() when only series data has changed, so need to think about the way of verifying what was changed in series, and if it's just the data, then use the mentioned function instead of updating whole chart.

@Denyllon Denyllon added the enhancement New feature or request label Jun 19, 2019
@augnustin
Copy link

I also have issues with chart update along vue state recalculation.

But are we sure that highcharts doesn't call Series.setData() behind the scene? I agree that the addSeries event is never fired, but the documentation of the update oneToOne parameter seems to suggest that this behavior can be expected.

We need more info on how highcharts manages state behind the scene ...

@Denyllon
Copy link
Contributor Author

Denyllon commented Mar 29, 2022

Hi @augnustin,

We need more info on how highcharts manages state behind the scene ...

What do you mean? This wrapper just calls the Chart.update() method once chart options have changed. This issue just describes the way of how we should handle those cases when the series data only changes. It's nothing than just more efficient way of dealing with series data updates.

Kind regards!

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

No branches or pull requests

2 participants