Investigate memory leak in Firefox
Reported by Jonathan Lister | October 10th, 2010 @ 01:17 PM | in 0.8
I think there is a memory leak - after a while, the performance drops as FF's memory usage grows.
Comments and changes to this ticket
-

-

Jonathan Lister October 10th, 2010 @ 08:14 PM
- State changed from new to open
Step 1: reduce use of large arrays.
Getting rid of large arrays: axisx & data
mapDiffaToRaphael:
array with 2-minute intervals' timestamps
this is used to bucket fdData into clusteres, which go into 'clusters' array
DONEsetupHeatmapConfig:
sticks axisx in config
axisx.length used to get total number of increments
DONEdrawXAxis:
'now' is created from the last entry in axisx (and then not used!) axisx is iterated through to map to an array that contains those values that round to the nearest hour
DONEdrawBlobs:
axisx.length is used as the upper limit on a loop
the loop only looks at data points; each data point's timestamp is needed, which is pulled from axisx
DONEscrollHeatmapTo:
axisx.length is used
DONE----- what we'd need to save -----
axisx.length
now (=axisx[axisx.length-1])
the value of axisx for each data point -

Ben Hood October 10th, 2010 @ 08:19 PM
- State changed from open to resolved
(from [847e997b13a085462b2023fbe05384515ffedba1]) Removed use of large arrays axisx & data. This has had a reasonably positive effect on the performance. There is still some work to do to remove the circular reference between each blob (a SVG element) and its cluster (a JS object). I am going to open another ticket for that and set it to v1.0.
There has also been some miscellaneous tidying up, mainly to make sure that variables are not duplicated or inconsistently named.
[[#122 state:resolved]] http://github.com/lshift/diffa/commit/847e997b13a085462b2023fbe0538...
-

Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
A Real Time Differencing Tool.
People watching this ticket
- Nobody is watching this ticket.
Referenced by
-
122
Investigate memory leak in Firefox
[[#122]]
http://github.com/lshift/diffa/commit/3cc6d77c6...
-
122
Investigate memory leak in Firefox
[[#122 state:resolved]]
http://github.com/lshift/diffa/c...
-
123
Remove circular reference between blobs and clusters
Following on from #122 commit message...