Jacopo Scazzosi
Jacopo Scazzosi
Indeed, chained buffering steps might explain the slowdowns I’ve seen. Normally I would expect performance to slow down linearly with the number of chained transforms but often I see slowdowns...
> I was thinking that one way around this is to have a 'chained transform' which applies multiple map/filter/transform operations in one go Would this not require knowing what those...
> I was thinking that methods like map, filter and transform would add to the internal set of operations that need to be done and then return the same iterator....
@jeswr could you have a look at this gist of mine? https://gist.github.com/jacoscaz/e20812991714092dbb20cfd432d1bf00 Either I got something wrong or there's virtually no performance difference between `.map()` and a dedicated mapping iterator.
> Sorry to keep spamming this thread @jeswr if all spam were like this we'd all be much happier about spam. Seriously, awesome work. > To summarize the results I...
@jeswr as I was investigating this, I came up with the same idea as https://github.com/RubenVerborgh/AsyncIterator/issues/38#issuecomment-1077584185 and had a go at it in https://github.com/jacoscaz/AsyncIterator/blob/42311b68988a971fc072c896100d7082b4162470/asynciterator.ts#L744-L783 . I'll make a PR later but...
> I'd say this should be a new library of its own e.g. 'asynciterator-compose.js' Makes sense! > https://github.com/comunica/comunica/issues/965 In _theory_, this should be mitigated by #46 although my testing has...
In addition to the above plan shared by @jeswr , if nobody else needs these changes to live within a single branch for testing purposes I would suggest breaking the...
> I'm not sold yet on SynchronousTransformIterator; I'd need to see a clean version of the code, but things like maybeMap sound like over-optimizations @RubenVerborgh is it the overall approach...
A few comments to keep everyone in the loop and for posterity... While working on **5. Faster cloning**, I've realized that my assumptions about the relative performance of `Array#push()` and...