psteinb / sota_on_uncertainties

trying to obtain uncertainties from training accuracies using timm
BSD 3-Clause "New" or "Revised" License
9 stars 0 forks source link

Error in jusuf Slurm config #4

Closed zyzzyxdonta closed 2 years ago

zyzzyxdonta commented 2 years ago

I couldn't try it because I don't have access to Jusuf. But I'm fairly certain that there is an error in it. You specified the CPUs per task twice: Once with -c, once with --cpus-per-task.

https://github.com/psteinb/sota_on_uncertainties/blob/ddd513b2aea18b7ad5bf031e8ec15ef924a2a401/config/slurm/jusuf/config.yaml#L2

Also, the ngpu setting is not used in the cluster string.

psteinb commented 2 years ago

thanks for spotting this. slurm didn't throw an error. Fixed with 84f9668