nfnitloop / better-iterators Goto Github PK

For mapParUnordered, I yield all settled Promises before returning to filling the work queue. This means that, depending on where iteration pauses, there could be chunks of time where there are work slots available, but not running any tasks.

Should update that for better parallelism. Only yield one settled Promise at a time, then start refilling the pipeline. Could probably use partition to help simplify here. :)

Right now, associateBy and groupBy always return T, but if we've got say, [key, value], and want to convert that to a map, it would be nice to just grab out the value for storing in the map.

For lazy iteration that ends early (ex: first(), or limit()) we should make sure to call .return() on the inner Iterable if it's a Generator, so that it can clean up its resources.

For example, here's how Array.filter() does it:

interface Array {
    filter<S extends T>(predicate: (value: T, index: number, array: T[]) => value is S, thisArg?: any): S[];
}