sdc icon indicating copy to clipboard operation
sdc copied to clipboard

[WIP]: Tweak Int64Index.reindex() performance

Open kozlov-alexey opened this issue 3 years ago • 1 comments

Motivation: current implementation of Int64Index reindex via building map_positions as a typed.Dict and filling the result indexer in a prange scales poorly and has performance that is far from ideal. This PR improves performance by ~2x by using native TBB based implementation.

kozlov-alexey avatar Aug 17 '21 20:08 kozlov-alexey

Hello @kozlov-alexey! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 69:1: E266 too many leading '#' for block comment Line 71:1: E302 expected 2 blank lines, found 0 Line 76:1: E305 expected 2 blank lines after class or function definition, found 1 Line 1175:59: E231 missing whitespace after ','

pep8speaks avatar Aug 17 '21 20:08 pep8speaks