streaming-benchmarks icon indicating copy to clipboard operation
streaming-benchmarks copied to clipboard

Published 20 hours ago • verified publisher icon composewell

Add iteratees.

Open Fuuzetsu opened this issue 7 years ago • 3 comments

I'd love to see iteratees added to the list.

Fuuzetsu avatar Mar 19 '18 13:03 Fuuzetsu

I would appreciate any help in doing that. A PR is welcome. BTW, the charts in the readme are not all that is available in the package. It has benchmarks for logict, list-t, list-transformer as well. I disabled simple-conduit because I was having difficulty in compiling it along with the latest version of all other dependencies.

harendra-kumar avatar Mar 19 '18 13:03 harendra-kumar

One problem I foresee is that it would be about 10x slower, and would make the charts harder to read.

fumieval avatar Mar 29 '18 07:03 fumieval

It can still be added, we do not necessarily need to add it to the charts, there are other libraries that are not in the charts but are benchmarked. One can run the benchmark to see the results, compare it with other libs or chart it if they want to. You can select which libraries you want to chart.

For publishing the results, we can keep the slower libraries in separate charts. We can bunch libraries with similar perf together.

harendra-kumar avatar Mar 29 '18 07:03 harendra-kumar