manodeep / Corrfunc

⚡️⚡️⚡️Blazing fast correlation functions on the CPU.
https://corrfunc.readthedocs.io
MIT License
164 stars 50 forks source link

Add per cell timer #95

Closed manodeep closed 7 years ago

manodeep commented 7 years ago

I ran into this requirement while creating plots for the paper -- a per-cell timer gives a fine-grained look into the time spent in each pair of cells opened.

There will likely be massive performance impacts if this per-cell timer is enabled. Probably needs to have a warning printed once per API call.

manodeep commented 7 years ago

As noted in 9133c643c66ad34b364732d1628e5668208ce064, majority of users are unlikely to use this feature. I only wanted it to generate the plots for the paper -- and for that only one pair-counter is required. Therefore, closing this issue for now.