gif-encoder icon indicating copy to clipboard operation
gif-encoder copied to clipboard

Add performance tests

Open twolfson opened this issue 12 years ago • 3 comments

During the development of gifsockets, performance was a major priority. We should embrace performance driven development as a core principle for this repository.

Things to perf:

  • Verify using ByteCapacitor over pushing for every byte is more efficient
    • Test via both .read() and .on('data')
  • Switch to ndarray over typed arrays for NeuQuant

twolfson avatar Nov 12 '13 10:11 twolfson

  • Reuse of indexedPixels instead of reallocating them for every analyzePixels

twolfson avatar Nov 12 '13 10:11 twolfson

  • Performance of removing alpha channel vs iterating of rgba array (this.pixels)
    • https://github.com/twolfson/gif-encoder/blob/0.3.0/README.md#setimagepixelspixels-options

twolfson avatar Nov 12 '13 10:11 twolfson

In 0.4.1, we discovered that frame based buffers are required since byte by byte buffers will fall over. This takes care of the first bullet point.

twolfson avatar Nov 21 '13 11:11 twolfson