Chart updates should be batched #253

New Issue

Open

opened 2022-11-26 12:18:16 +00:00 by toast · 1 comment

toast commented

2022-11-26 12:18:16 +00:00

Owner

Current code is:

for (const chart of charts) {
	chart.save();
}

in a setInterval.

Current code is: ```js for (const chart of charts) { chart.save(); } ``` in a setInterval.

toast referenced this issue

2022-11-26 12:18:23 +00:00

Chart Datapoint creation slows down instance significantly #237

puniko commented

2022-12-03 12:10:49 +00:00

Contributor

tried to reanable charts to see how they behave with the web-workers. it doesn't lock up fe, which is good. but it still locks up delivery queue, while its gathering data.

question: what will happen, when i reanable it again in the code? i tried today, but it did spike my cpu for an hour (until i disabled it again and restarted the instance, because it clogged up my delivery queue), so could it be, that foundkey is trying to fetch all past missing datapoints or something?

tried to reanable charts to see how they behave with the web-workers. it doesn't lock up fe, which is good. but it still locks up delivery queue, while its gathering data. question: what will happen, when i reanable it again in the code? i tried today, but it did spike my cpu for an hour (until i disabled it again and restarted the instance, because it clogged up my delivery queue), so could it be, that foundkey is trying to fetch all past missing datapoints or something?