PeyracheLab / miniscoPy

A package to analyse calcium imaging data recorded with the Miniscope.
GNU General Public License v2.0
32 stars 9 forks source link

Modifications necessary for executing main_cnmf_e.py on both - Windows and Linux #5

Closed DenisPolygalov closed 6 years ago

DenisPolygalov commented 6 years ago

main_cnmf_e.py now can be used on Windows. The main change is to guard everything by if name == 'main': statement otherwise multi-processing dosen't work on Windows. Reference: https://stackoverflow.com/questions/18204782/runtimeerror-on-windows-trying-python-multiprocessing Also, Linux-sspecific code (path seperators) replaced with platform-neutral. Also, it is better to avoid things like "from somewhere import *".

gviejo commented 6 years ago

Thanks Denis