I'm generating a load of coordinates (made of 3 numbers) within a geographical area. However, using Streams (which should be much more efficient than Lists), fills up the app's memory very quickly, as can be seen in this screenshot from Observatory.
I need a structure where events can go in, and be read out one by one, and when this happens, removed from the structure. As far as I understand, that is what a Stream is. When you add a value, the old one is removed.
Unfortunatley, this doesn't appear to be happening. Instead, the stream just grows larger and larger - or at least something reading it does, but I just run the .length
method on the returned stream, and that's it.
Here's the function that starts the Isolate that returns the stream of coordinate tiles. I'll omit the actual generator, as it's not important: it just sends a Coord
to the SendPort
.
static Stream<Coords<num>> _generateTilesComputer(
DownloadableRegion region,
) async* {
List<List<double>> serialiseOutline(l) => (l as List)
.cast<LatLng>()
.map((e) => [e.latitude, e.latitude])
.toList();
final port = ReceivePort();
final tilesCalc = await Isolate.spawn(
region.type == RegionType.rectangle
? rectangleTiles
: region.type == RegionType.circle
? circleTiles
: lineTiles,
{
'port': port.sendPort,
'rectOutline': region.type != RegionType.rectangle
? null
: serialiseOutline(region.points),
'circleOutline': region.type != RegionType.circle
? null
: serialiseOutline(region.points),
'lineOutline': region.type != RegionType.line
? null
: (region.points as List<List<LatLng>>)
.chunked(4)
.map((e) => e.map(serialiseOutline)),
'minZoom': region.minZoom,
'maxZoom': region.maxZoom,
'crs': region.crs,
'tileSize': region.options.tileSize,
},
);
await for (final Coords<num>? coord in port
.skip(region.start)
.take(((region.end ?? double.maxFinite) - region.start).toInt())
.cast()) {
if (coord == null) {
port.close();
tilesCalc.kill();
return;
}
yield coord;
}
}
}
How can I prevent this memory leak? Happy to add more info if needed, but the full source code can be found at https://github.com/JaffaKetchup/flutter_map_tile_caching.
Turns out, the issue was I was inadvertently storing the output of the stream. I'm not sure how/why/where, as the code in the question was designed to replace the code that stored these coords and outputted them as one List
, but I guess I didn't get to the root cause.
For others trying to find the answer to this issue: it's not the Stream
s fault, you're probably storing the output of the stream.
For a useful pattern of 'multi-threaded' looping through results and processing them as they arrive, see this file from my library (may be outdated).