-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename dask-config
CLI for clarity
#1038
Comments
Additionally, this is broken since the |
This raises another good question. Is it being used (considering some functionality in it is currently broken and I'm guessing we are not hearing about it)? |
It was only recently introduced, so probably no-one knows about it yet. I recommend renaming (via the new |
+1 to renaming |
In PR ( #981 ), @jacobtomlinson suggested using |
Did we settle on a name? cc @quasiben |
Either my or @wence-'s suggestions seem fine. |
Currently Dask-CUDA exposes a CLI called
dask-config
dask-cuda/setup.py
Line 61 in 2c99f5a
That points to a similarly named module
As it may not be clear that this CLI comes from Dask-CUDA (say in a busy environment), wonder if it makes sense to rename this for clarity
The text was updated successfully, but these errors were encountered: