check memory is specified as numeric value

mschubert / clustermq

R package to send function calls as jobs on LSF, SGE, Slurm, PBS/Torque, or each via SSH

https://mschubert.github.io/clustermq/

Apache License 2.0

146 stars 27 forks source link

check memory is specified as numeric value #233

Closed sam217pa closed 3 years ago

sam217pa commented 3 years ago

Hi, I'm on a slurm cluster and tried specifying memory usage with template = list(memory = "4G"), as slurm understands this kind of argument. I got the "Worker needs about 23Mb overhead ..." error because memory wasn't passed a numeric value.

I guess a simple condition like this could solve this problem; I don't really know how the rest of the code would interpret it though.

thanks for clustermq anyway ;)

mschubert commented 3 years ago

Thank you for the PR, @sam217pa!

I will, however, try to fix this in a way that clustermq understands shorthands such as "4G" for v0.9

Related: #137