NVIDIA-Genomics-Research / AtacWorks

Deep learning based processing of Atac-seq data
https://clara-parabricks.github.io/AtacWorks/
Other
128 stars 23 forks source link

Change --gpu to --gpu_id to clarify the functionality #177

Closed ntadimeti closed 4 years ago

vellamike commented 4 years ago

please don't use underscore in longopts, it can get eaten by terminal borders and look like a space, longopts should be --like-this

ntadimeti commented 4 years ago

@vellamike thanks for the input! I'll change it accordingly

ntadimeti commented 4 years ago

@vellamike -- I saw your comment again. We don't have short option and long options. So, we actually access this argument with args.gpu_idx and a hiphen isn't compatible in that case. Am I making sense? Happy to discuss offline