Skip to content

CUDA-related updates and cleanups#30433

Merged
abbradar merged 13 commits intoNixOS:masterfrom
abbradar:cudatoolkit
Oct 19, 2017
Merged

CUDA-related updates and cleanups#30433
abbradar merged 13 commits intoNixOS:masterfrom
abbradar:cudatoolkit

Conversation

@abbradar
Copy link
Member

Motivation for this change
  1. Update CUDA toolkit to the latest version, make it default;
  2. Make a unified config flag cudaSupport which controls whether CUDA support is desired in applications;
  3. Update and cleanup various machine learning libraries which make use of CUDA.
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@abbradar abbradar added the 8.has: package (update) This PR updates a package to a newer version label Oct 15, 2017
@abbradar abbradar requested a review from FRidh as a code owner October 15, 2017 12:48
@abbradar
Copy link
Member Author

Notice that I don't have access to NVIDIA GPU now (My Google Cloud free credit ran empty :D) so it'd be nice to have someone test those changes (I used all of them but some time ago and rebase may have broken things).

@abbradar abbradar mentioned this pull request Oct 15, 2017
8 tasks
@abbradar abbradar force-pushed the cudatoolkit branch 4 times, most recently from 7a83f07 to 9120f78 Compare October 16, 2017 09:36
@abbradar
Copy link
Member Author

I've managed to make my laptop's hybrid Nvidia to work with CUDA so these changes are mostly tested now. Let's merge this after #29814 lands so that our default Boost plays nicely with new toolkit.

@abbradar abbradar merged commit 15bfc8c into NixOS:master Oct 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: package (update) This PR updates a package to a newer version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant