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

Compile IMap[Unordered] with Cython #1154

Merged
merged 2 commits into from Mar 25, 2018

Conversation

Projects
None yet
1 participant
@jamadden
Member

jamadden commented Mar 24, 2018

This gets us another 20-30% faster:

Benchmark 27_pool_opts 27_pool_cython2
imap_unordered_seq 897 us 694 us: 1.29x faster (-23%)
imap_unordered_par 539 us 363 us: 1.49x faster (-33%)
imap_seq 1.00 ms 714 us: 1.41x faster (-29%)
imap_par 612 us 404 us: 1.52x faster (-34%)
map_seq 382 us 349 us: 1.09x faster (-9%)
map_par 267 us 252 us: 1.06x faster (-6%)
apply 427 us 406 us: 1.05x faster (-5%)
spawn 397 us 360 us: 1.10x faster (-9%)
Benchmark 36_pool_opts 36_pool_cython6
imap_unordered_seq 686 us 534 us: 1.28x faster (-22%)
imap_unordered_par 389 us 286 us: 1.36x faster (-27%)
imap_seq 729 us 579 us: 1.26x faster (-21%)
imap_par 398 us 306 us: 1.30x faster (-23%)
map_seq 293 us 280 us: 1.04x faster (-4%)
map_par 199 us 194 us: 1.02x faster (-2%)
apply 326 us 320 us: 1.02x faster (-2%)
spawn 298 us 307 us: 1.03x slower (+3%)

jamadden added some commits Mar 24, 2018

Compile IMap[Unordered] with Cython
This gets us another 20-30% faster:

| Benchmark          | 27_pool_opts | 27_pool_cython2             |
|--------------------|--------------|-----------------------------|
| imap_unordered_seq | 897 us       | 694 us: 1.29x faster (-23%) |
| imap_unordered_par | 539 us       | 363 us: 1.49x faster (-33%) |
| imap_seq           | 1.00 ms      | 714 us: 1.41x faster (-29%) |
| imap_par           | 612 us       | 404 us: 1.52x faster (-34%) |
| map_seq            | 382 us       | 349 us: 1.09x faster (-9%)  |
| map_par            | 267 us       | 252 us: 1.06x faster (-6%)  |
| apply              | 427 us       | 406 us: 1.05x faster (-5%)  |
| spawn              | 397 us       | 360 us: 1.10x faster (-9%)  |

@jamadden jamadden merged commit 8916cda into master Mar 25, 2018

5 checks passed

continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.08%) to 84.112%
Details

@jamadden jamadden deleted the threadpool-opts branch Mar 25, 2018

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