JRCLUST icon indicating copy to clipboard operation
JRCLUST copied to clipboard

Manual curation is slower in v4 compared to v3

Open LiuDaveLiu opened this issue 6 years ago • 4 comments
trafficstars

Everything is much slower in the manual curation of v4 compared to v3. Merging and splitting takes 5s per action.

If you have a usage or support question, come join our Gitter and ask your question there. If you found a bug, please provide all of the following:

Output of ver

Same as before

Output of gpuDevice

Same as before

The command you used

(e.g., jrc detect-sort myparam.prm)

If manually sorting, what it is you tried to do

Merging units is much slower than in v3. Computing waveform correlation... took 1.7s Computing self correlation ................................................................................................................................................................................................................................................................................................................................................................................................................ took 4.9s Calculating cluster quality... took 0.1s Computing waveform correlation... took 3.1s Computing self correlation ............................................................................................................................................................................................................................................................................................................................................................................................................... took 4.9s Calculating cluster quality... took 0.0s Computing waveform correlation... took 0.2s Computing self correlation .............................................................................................................................................................................................................................................................................................................................................................................................................. took 4.9s Calculating cluster quality... took 0.0s Computing waveform correlation... took 0.1s Computing self correlation ............................................................................................................................................................................................................................................................................................................................................................................................................. took 5.2s Calculating cluster quality... took 0.1s Computing waveform correlation... took 3.9s Computing self correlation ............................................................................................................................................................................................................................................................................................................................................................................................................ took 5.6s Calculating cluster quality... took 0.0s

Console output

LiuDaveLiu avatar Feb 09 '19 01:02 LiuDaveLiu

I'll need to profile this and see where the bottleneck is.

aliddell avatar Feb 10 '19 02:02 aliddell

Try running ‘jrc unit-test manual’ and it should create a profiler report. It should be compared between version 3 and 4 to determine what’s slowing down.

-James

On Feb 8, 2019, at 8:27 PM, LiuDaveLiu [email protected] wrote:

Everything is much slower in the manual curation of v4 compared to v3. Merging, splitting, saving, etc.

If you have a usage or support question, come join our Gitter and ask your question there. If you found a bug, please provide all of the following:

Output of ver

Same as before

Output of gpuDevice

Same as before

The command you used

(e.g., jrc detect-sort myparam.prm)

If manually sorting, what it is you tried to do

Merging units is much slower than in v3. Computing waveform correlation... took 1.7s Computing self correlation ................................................................................................................................................................................................................................................................................................................................................................................................................ took 4.9s Calculating cluster quality... took 0.1s Computing waveform correlation... took 3.1s Computing self correlation ............................................................................................................................................................................................................................................................................................................................................................................................................... took 4.9s Calculating cluster quality... took 0.0s Computing waveform correlation... took 0.2s Computing self correlation .............................................................................................................................................................................................................................................................................................................................................................................................................. took 4.9s Calculating cluster quality... took 0.0s Computing waveform correlation... took 0.1s Computing self correlation ............................................................................................................................................................................................................................................................................................................................................................................................................. took 5.2s Calculating cluster quality... took 0.1s Computing waveform correlation... took 3.9s Computing self correlation ............................................................................................................................................................................................................................................................................................................................................................................................................ took 5.6s Calculating cluster quality... took 0.0s

Console output

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

jamesjun avatar Feb 10 '19 19:02 jamesjun

Will do

aliddell avatar Feb 11 '19 13:02 aliddell

This is partially addressed in 4539c5f2bd5dc06b6743b7c1cfabbd0998075493. Auto-merge is still much slower, that's a work in progress.

aliddell avatar Feb 19 '19 00:02 aliddell