disarm-dev / douma-app

DiSARM app (2018 and beyond)
MIT License
3 stars 1 forks source link

[aggregations] filter more efficiently #153

Open onlyjsmith opened 6 years ago

onlyjsmith commented 6 years ago

We're iterating every response for every filter. How can we reduce this?

onlyjsmith commented 6 years ago

Looks to me like Array.filter is exactly the same in the transpiled output. (Searched for filter_responses in app.js on deployed stage).

I think this would be a great place to start looking at performance, with a very small surface area of code to touch, and limited chance of side-effects.