vedeu icon indicating copy to clipboard operation
vedeu copied to clipboard

dsl_vertical_alignment example takes seconds to render each update

Open dymk opened this issue 9 years ago • 3 comments

Tested with:

Vedeu 0.8.16 OSX 10.11.3 iTerm 2 (v3 Beta) and Terminal.app

This is a pretty bad performance bug, running the dsl_vertical_alignment.rb is horrendously slow (each refresh takes ~4 seconds), when pressing one of 'hjkl' to move around the middle square. Is there any low hanging fruit here to pluck?

dymk avatar Feb 20 '16 00:02 dymk

The biggest cost is the fact that both dsl_vertical_alignment.rb and dsl_horizontal_alignment.rb use multiple renderers. I disabled the text renderer which writes to the filesystem (Vedeu::Renderers::Text) and my refresh time is reduced to a still unacceptable 350ms. Ideally, I'd like to get this down to 60fps (16.67ms), so there is work to be done.

Pull requests are always welcome, but in their absence, patience is appreciated :-D

Gav

gavinlaking avatar Feb 22 '16 22:02 gavinlaking

Refresh time for me is between 250ms and 300ms now, still a fair way to go to reach ~16ms.

gavinlaking avatar May 07 '16 00:05 gavinlaking

Refresh time for me is between 150ms and 200ms as of 612eb37.

gavinlaking avatar Jul 03 '16 19:07 gavinlaking