Skip to content

Grid performance over 10 columns and 25 rows and number of watchers #5547

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

Open
zalayeto opened this issue Jul 13, 2016 · 4 comments
Open

Grid performance over 10 columns and 25 rows and number of watchers #5547

zalayeto opened this issue Jul 13, 2016 · 4 comments

Comments

@zalayeto
Copy link

zalayeto commented Jul 13, 2016

Hi,

We're were trying to use our angular application with ui-grid on a big screen, being the grid the only element on the screen. Unfortunately, we've realized that the grid in this scenario becomes really sluggish. Looking into the issue, we've realized that the number of watchers is around 8000.

We've noticed that it seems that, roughly, every cell is adding around 10 watchers. This way, for example, for a normal screen, a grid taking up the whole space (let's say 12 columns and 25 rows) the number of watchers would be roughly 12 * 25 * 10 = 3000', well over the recommended level of 2000, and causing performance issues.

This behavior is happening as well in the tutorial so for example the grid (only the controller MainCtrl that holds the grid) in the tutorial 404 ([http://ui-grid.info/docs/#/tutorial/404_large_data_sets_and_performance]) has around 450 watchers showing 4 columns and 10 rows.

Why ui-grid needs 10 watchers per cell? Is there any way of reducing this?

Thanks,

David.

@zalayeto zalayeto changed the title Grid performance over 10 columns and 40 rows Grid performance over 10 columns and 25 rows and number of watchers Jul 13, 2016
@Ruud-cb
Copy link

Ruud-cb commented Jul 27, 2016

Hi Zalayeto, any progress on this? Did you found some improvements?

@zalayeto
Copy link
Author

zalayeto commented Jul 29, 2016

Hi,

Nothing definitive yet. So far we've done this 4612 and having a go with this 4554 (@YonatanKra 's suggestion ).

@Ruud-cb
Copy link

Ruud-cb commented Jul 29, 2016

@zalayeto Hmm, well I eventually went with ag grid, perhaps read #5585 if you're interested to my conclusions.

@VioricaMihai
Copy link

Hi,
Any news regarding this issue? I have the sane problem with performance in my application (3 grids with 25columns and 25 rows) -over 6000 watchers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants