lacker / seticore

A high-performance implementation of some core SETI algorithms that can be included in other programs.
MIT License
3 stars 7 forks source link

Add a GPU ID command line parameter (main.cpp) #6

Closed texadactyl closed 2 years ago

texadactyl commented 2 years ago

dedoppler should be passed a GPU ID parameter for use with cudaSetDevice(gpu_id) Command line default: 0

lacker commented 2 years ago

Is it good enough to use the environment variable CUDA_VISIBLE_DEVICES ? I'd rather not support extra flags if there's already some way to do it.

texadactyl commented 2 years ago

rawspec, turbo_seti, and hyperseti have set a precedent by providing this flexibility

texadactyl commented 2 years ago

Should seticore become successful, this won't be the last user interface change

texadactyl commented 2 years ago

e.g. min_drift_rate which might be set quite high in the case of Mars

texadactyl commented 2 years ago

The author has the right to design how a specific device is selected. Therefore, I am closing this issue and opening a PR that just amends README.md to add a section on how to deterministically select a specific device using environment variables.