number of cores need to be adjustable

compomics / moFF

A modest Feature Finder (moFF) to extract MS1 intensities from Thermo raw file

Apache License 2.0

33 stars 11 forks source link

number of cores need to be adjustable #33

Closed bgruening closed 6 years ago

bgruening commented 6 years ago

https://github.com/compomics/moFF/blob/master/moff.py#L376 should be adjustable by an admin. Otherwise, this tool will be always killed by a scheduler on a cluster.

Maux82 commented 6 years ago

How can I make it adjustable by an admin ? Do you think is it better to add the number of CPU as input parameter (as default I set it to multiprocessing.cpu_count() ) ?

bgruening commented 6 years ago

Yes, a parameter that is overwriting your default would be nice. Thanks!