magland / figurl-franklab-views

0 stars 0 forks source link

Memoize data validation #10

Open jsoules opened 1 year ago

jsoules commented 1 year ago

See e.g. PR #9 for the start of this problem.

View selection depends on validating the type of the data coming in, which is currently tightly integrated with the view loader. This can cause (potentially quite expensive) validations on large data sets with every redraw.

We should do a couple things to memoize this and ensure better performance:

Production deployments seem to have optimized this out for some reason, so it's not an urgent need (just something that makes development environments more representative), but probably a medium priority since the changes won't be super extensive and ought to be echoed to other projects in the ecosystem.