Fix data table worker

[bramkragten]

Proposed change

Restore the intended behavior of the data table filter/sort worker.

Type of change

• Dependency upgrade
• Bugfix (non-breaking change which fixes an issue)
• New feature (thank you!)
• Breaking change (fix/feature causing existing functionality to break)
• Code quality improvements to existing code or addition of tests

Example configuration

Additional information

• This PR fixes or closes issue: fixes #
• This PR is related to issue:
• Link to documentation pull request:

Checklist

• The code change is tested and works locally.
• There is no commented out code in this PR.
• Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

• Documentation added/updated for www.home-assistant.io

[balloob]

This is good.

I think that we could make it slightly faster if we make the worker do some more memoization.

If worker had an API like this:

interface Worker {
  createDataSet(data: Rows[]): number;
  sortAndFilter(dataSet: number, filter, sort)
  destroyDataSet(dataSet: number)
}

We would only send the data over once, and we could still memoize on a per data set basis.

let setId = 0;
let sets = {};

function createDataSet(data: Rows[]) {
  setId++;
  sets[setId] = {
    data,
    filterData: memoizeOne(filter => filterData(data, filter)),
  }
}

[balloob]

Alternative is to use a worker per data table, so you don't have to worry about cleaning up data sets.

[bramkragten]

I like that idea, let's experiment with that in a new PR :-)