js-framework-benchmark icon indicating copy to clipboard operation
js-framework-benchmark copied to clipboard

Improve Non-Keyed Vanilla Implementations

Open ryansolid opened this issue 2 years ago • 6 comments

I tend to treat that table like it doesn't exist so I haven't looked at it for months (years?), but noticed Remove Rows in the vanilla implementation seems to be slower than a handful of implementations. We should look at improving it.

If anyone in the community familiar with Non-Keyed wants to contribute I think this is a worthwhile effort. Vanilla should be the fastest and without it being so it compromises the value of the particular tests.

ryansolid avatar Aug 30 '21 08:08 ryansolid

I'm interested on improving this, although I was curious as to why the Remove Row seems to be slower than the keyed version

lxsmnsyc avatar Aug 30 '21 09:08 lxsmnsyc

I had written my own take on the vanillajs implementation image

Tho it seems to me that both implementations are just patching elements and not actually replacing them. Is that a valid approach?

lxsmnsyc avatar Aug 30 '21 11:08 lxsmnsyc

image

Anyways, I didn't saw the note above so I just reworked it a bit.

lxsmnsyc avatar Aug 30 '21 12:08 lxsmnsyc

@LXSMNSYC Would you like to submit a PR for that?

krausest avatar Sep 12 '21 19:09 krausest

@LXSMNSYC Would you like to submit a PR for that?

It's actually my own take. I didn't try to update both vanillajs and vanillajs-1 due to their select implementation. In a nutshell, my implementation assumes that data and view are both in sync.

lxsmnsyc avatar Sep 13 '21 08:09 lxsmnsyc

That would be fine. You could just submit a new vanillajs implementation and I'd remove the others when I checked that it's the fastest.

krausest avatar Sep 13 '21 11:09 krausest