search

tdurham86 / EPICViz

EPICViz - http://cse512-15s.github.io/fp-tdurham-ajh24-chiasson-ningli30/
0 stars 0 forks source link

Consolidate the small multiples plotting functions in to a single function #6

Open tdurham86 opened 7 years ago

tdurham86 commented 7 years ago

If we handle all small multiples plotting in one function, we can probably save a bunch of overhead that we currently incur from doing multiple data bindings.

tdurham86 commented 7 years ago

I'm no longer sure that this is as good of an idea. It seems to me that D3 really is built to bind one DOM object to one piece of data, and the data binding routine is probably very optimized and makes it easy to change/update/remove just the relevant objects in each case. If we were to bind the data to a single group of objects, we would have to use selectors in every callback on every data point to get the relevant DOM object from each other (non-bound) plot and edit their relevant attributes one-by-one. This strikes me as most likely being much less efficient than using D3's iteration over particular classes of objects.

That being said, we should still be keeping an eye out for new ideas about how to improve the efficiency of plotting the data.