Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add iteratees. #2

Open
Fuuzetsu opened this issue Mar 19, 2018 · 3 comments
Open

Add iteratees. #2

Fuuzetsu opened this issue Mar 19, 2018 · 3 comments

Comments

@Fuuzetsu
Copy link

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

@harendra-kumar
Copy link
Member

harendra-kumar commented Mar 19, 2018

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.

@fumieval
Copy link
Contributor

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

@harendra-kumar
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants