search

k3oni / pydash

Small web-based monitoring dashboard for linux in Python and Django
662 stars 125 forks source link

JS memory leak #41

Open mathieufrh opened 8 years ago

mathieufrh commented 8 years ago

Hi,

This little program looks fun. But I just want to let you know that there's an memory issue with the js code. I am really not a js expert but you can check it in chrome using snapshots or other tools. In dashboard.js you keep create charts but you never delete them. I think it would be better to use the update method of Chart.js. You declare the charts in main.html and you update them in dashboard.js using the data returned by the usage/views.py functions.

ghost commented 7 years ago

@mathieufrh could you please provide me with screenshots of the memory leak? I will see if I can find some time to fork the project and fix the issue.