I hacked this logic (#1283, #1292) in when struggling with some T2T alignments on Slurm, but clearly didn't test it enough since it completely broke cactus on AWS/MESOS. The reason is that it's using (and int-casting) options.maxMemory and options.defaultMemory from Toil, which default to None on AWS/MESOS (but not single machine and slurm where I was testing).
Anyway, this PR should fix it, buy handling None values (and making sure that passed-in values are properly parsed).
I hacked this logic (#1283, #1292) in when struggling with some T2T alignments on Slurm, but clearly didn't test it enough since it completely broke cactus on AWS/MESOS. The reason is that it's using (and int-casting)
options.maxMemoryandoptions.defaultMemoryfrom Toil, which default toNoneon AWS/MESOS (but not single machine and slurm where I was testing).Anyway, this PR should fix it, buy handling None values (and making sure that passed-in values are properly parsed).