New nvimcom option: max_cpu_cores

[jalvesaq]

Close #593

[PMassicotte]

Good job. Should we also give an example about using SLURM requested CPUs?

options(nvimcom.max_cpu_cores = as.integer(Sys.getenv("SLURM_CPUS_PER_TASK"))

[jalvesaq]

Should we also give an example about using SLURM requested CPUs?

Yes, if the variable will always be set for Slurm users in the normal way of using Slurm (which, according to Gemini, isn't through SSH). I have zero experience with HPC Clusters.

[PMassicotte]

I will add something.

[PMassicotte]

I think this is good to merge. Ok with you?

[jalvesaq]

It's OK for me. Should we wait a while for @daler's comments?

[PMassicotte]

It's OK for me. Should we wait a while for @daler's comments?

I think so!

[daler]

Tested, and works great -- thanks for such quick turnaround.

I made some adjustments to the docs to make it a little more general while still keeping the concrete Slurm-specific example, and slightly tweaked the example to have a max of 2 in case the env var is unavailable, like if you don't use --cpus-per-task.

[jalvesaq]

Thank you, @daler! Let's wait for @PMassicotte final review.

[PMassicotte]

LGTM! Merging, thank you all for your contributions.