consider removing RenderMode.mixed and RenderMode.raster about flutter-vector-map-tiles HOT 5 CLOSED

This sounds like a good idea to me. Best to remove the feature early when not too many people depend on it.
I haven't used the raster and mixed RenderModes much, though, so I don't know if there are certain cases where they are beneficial.

from flutter-vector-map-tiles.

One thing to consider is that once a tile is rendered to a raster image and cached to disk, future use of the tile might be faster. This could be useful for older/underpowered devices.

from flutter-vector-map-tiles.

Firstly, great stuff on all the work.

One thing I did do previously to experiment with my old stuff iirc, was to render to a raster in an isolate, save to disk, and then use that when available from the main thread (so just passing a ready confirmation via the isolate port). However, every raster type solution felt like clutter in the end and never really liked hybrid solutions and interfered with getting pure vectors working optimally which I think is similar to this thread.

I feel like it would be nice to keep the labels, and feel like it "should" be feasible, but maybe not just yet. Had a quick glance at the code, so I'm adding a few thoughts to the performance thread, even if it's closed, rather than clutter here, just in case any of it is helpful to someone.

from flutter-vector-map-tiles.

@ibrierley thanks for the feedback!

I'm not sure what you mean about "keep the labels", feel free to elaborate.

from flutter-vector-map-tiles.

Heya, apologies, yes that wasn't clear. That was meant to be part of the other performance thread so would have made more sense there, but I missed that in my cut/paste! I was using Labels to mean the text and TextPainter type code. So street labels and city labels. Basically I think I was trying to work out if code like _createTextPainter() is called every build/canvas draw, or whether it is just called once (per text/coord and then cached/reused and then just the render() method used only).

from flutter-vector-map-tiles.