-
Notifications
You must be signed in to change notification settings - Fork 887
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
[REVIEW] Fix issue with nvstrings dependency in branch-0.6 #1252
[REVIEW] Fix issue with nvstrings dependency in branch-0.6 #1252
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few changes needed that should be mirrored in the cuda9.2 environment file as well.
That was a lot more commentary than I expected! Your suggested changes are all in. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the 9.2 env file should be explicit about label/cuda9.2
This fixes #1231 and also fixes #1251.
Update the README with instructions on creating a new conda environment either for CUDA 9.2 or CUDA 10.0. This should resolve issues that users are having with incorrect NVStrings dependencies, which by default was installing NVStrings=0.2.0 for CUDA 9.2.