search

magland / ml_ms4alg

MountainSort v4
7 stars 19 forks source link

Edits to use Dask rather than multiprocessing for internal parallelism #25

Closed lfrank closed 4 years ago

lfrank commented 4 years ago

These edits allow parallel execution in SpikeInterface to work

magland commented 4 years ago

This is great @lfrank! Could you also add "dask" to the install_requires in setup.py?

lfrank commented 4 years ago

I think that should now be added.

Loren M. Frank

Professor
Howard Hughes Medical Institute Kavli Institute for Fundamental Neuroscience Department of Physiology University of California, San Francisco Office Phone: 415-502-7357

On May 29, 2020, at 12:50 PM, Jeremy Magland notifications@github.com wrote:

This is great @lfrank https://github.com/lfrank! Could you also add "dask" to the install_requires in setup.py?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/magland/ml_ms4alg/pull/25#issuecomment-636157583, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV4PSPRYFRBFY4YXITKSUTRUAGYNANCNFSM4NOJRW4A.

magland commented 4 years ago

We need a , after 'dask' @lfrank

lfrank commented 4 years ago

Let me know if there is anything else...

Loren M. Frank

Professor
Howard Hughes Medical Institute Kavli Institute for Fundamental Neuroscience Department of Physiology University of California, San Francisco Office Phone: 415-502-7357

On May 29, 2020, at 12:58 PM, Jeremy Magland notifications@github.com wrote:

We need a , after 'dask' @lfrank https://github.com/lfrank — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/magland/ml_ms4alg/pull/25#issuecomment-636162676, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV4PSISJYDF3KDHT7FNM3DRUAHXVANCNFSM4NOJRW4A.