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.

Sign up for GitHub

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

Jump to bottom

Improve performance of updating data #838

Closed
timocov opened this issue Sep 7, 2021 · 2 comments · Fixed by #866
Closed

Improve performance of updating data #838

timocov opened this issue Sep 7, 2021 · 2 comments · Fixed by #866
Assignees
@timocov
Labels
enhancement Feature requests, and general improvements.
Milestone
3.7

Comments

@timocov
Copy link
Contributor

timocov commented Sep 7, 2021

We significantly have improved performance of setting/updating data of series. It needs to migrate these fixes to lightweight-charts as well.
@timocov timocov added the enhancement Feature requests, and general improvements. label Sep 7, 2021
@timocov timocov added this to the 4.0 milestone Sep 7, 2021
@timocov timocov self-assigned this Sep 7, 2021
@timocov timocov modified the milestones: 4.0, 3.7 Sep 8, 2021
@timocov timocov mentioned this issue Oct 11, 2021
Merged
3 tasks
@timocov
Copy link
Contributor Author

timocov commented Nov 5, 2021

Related to #418

@vritme
Copy link

vritme commented May 29, 2022 •
edited
Loading

Done some tests on the production project.
3.5.0 - setting data for 9.845s.
3.7.0 - setting data for 6.045s.
38.6% improvement. Fantastic!)
Also in my case I have noticed, that 3.8.0 was a little bit slower than 3.7.0.

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

@timocov timocov

Labels
enhancement Feature requests, and general improvements.
Projects
None yet
Milestone
3.7
Development

Successfully merging a pull request may close this issue.

Improved performance of setting/updating series data tradingview/lightweight-charts
2 participants
@timocov @vritme