Use Future or Isolate? about thumbhash HOT 3 OPEN

Anyway we should add caching. With current straightforward implementation,

Image(
  image: ThumbHash.fromBase64('3OcRJYB4d3h/iIeHeEh3eIhw+j3A').toImage(),
)

seems to make decoding (plus encoding to bmp) on each build.

Instead of this, we should make API like

Image(
  image: ThumbHashImage.fromBase64('3OcRJYB4d3h/iIeHeEh3eIhw+j3A'),
)

where ThumbHashImage is ImageProvider which will decode to bmp (in isolate or not) once and cache the result.

from thumbhash.

Yes, using ImageProvider would be way better. Doing it on every build a big performance issue.

As far as the Isolate goes we can leave this open and do the benchmarks when there is time.

from thumbhash.

Thank you, it's interesting to think about.

Just wrapping functions with Future won't help because actual decoding will be still synchronous.

However, it's worth benchmarking decoding algorithm. Maybe it takes short time because thumbhash image is decoded maximum to 32x32 pixels. If it turns out that it takes long time, then I will consider decoding in Isolate for flutter_thumbhash.

from thumbhash.