Performance of a content renderer for the DynamicParagraph

[igorepst]

The performance of a content renderer for the DynamicParagraph (I imagine the same will hold for DynamicList and DynamicTable) is pretty bad. Please use the attached minimal config file by running xplr -c ~/xplr.conf.txt /usr/bin
xplr.conf.txt
(the extension was changed from lua to txt to suit GitHub requirements, but you don't even need to change it back)
The file has a DynamicParagraph with a renderer that returns some static string. Nevertheless, when I open /usr/bin with 3756 files and just press and hold 'arrow down', the scroll is super jerky. If, however, you comment the DynamicParagraph and uncomment StaticParagraph, the scrolling is smooth as it should be.

scroll.mp4

[sayanarijit]

Runs almost fine on my machine. I'll try it on an old machine.

[sayanarijit]

Ah I see... number of files is the issue.

[sayanarijit]

Not good... We should pass only the visible nodes in the Lua function.

[igorepst]

Yes, exactly, the number of files is what makes a difference, sorry for not being clear enough, as my laptop is not so old ;)

[sayanarijit]

🙊

[sayanarijit]

Not an easy fix. Need to rethink the API. I don't want to break existing the API, so I'll probably create a new API. But this issue will stay for a while.

[sayanarijit]

Note: Today I tried to cache mlua::Value of the directory nodes, but that didn't work. Which means, caching probably isn't a solution.

[sayanarijit]

Finally, found another hack based on caching. Not very pretty, but gets the job done. @igorepst pls test if possible.

[igorepst]

I will, although a bit later. Tnx!

[igorepst]

Seems it works good :)