jerch commented 1 year ago

WIP to get rid of the worker for sixels.

May close #7.

TODO:

jerch commented 1 year ago

Now without worker:

sixel processing ~1.5x faster
latency got much better (making xtermdoom playable in firefox, too), also we now have quite instant cursor moves in jexer
downside: decoding happens in mainthread (eats precious main CPU), still it is streamlined, thus should not create perceivable blocking
10 kB less on final packages
no more separate worker file (much easier to deal with on integrator side)

The workerless rework actually surfaced another issue - #37. Should be fixed before we merge the PR and do another release.

jerch commented 1 year ago

37 is still not fully fixed. It needs a more involved change in the decoder package (see #37 for details).

pmp-p commented 1 year ago

is it available for testing ?

jerch commented 1 year ago

@pmp-p No not yet. Well you can build it from this PR, if you want (worker already gone).

37 is currently blocking this PR, but I might do an independent release, if the fix gets too involving.

jerch commented 1 year ago

Postponing #37, as the adjustments needed to get that properly working are quite involved (needs a fundamental patch in the underlying lib).