-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
add CUDA support #76
add CUDA support #76
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
c5d4f79
to
5a3fddd
Compare
Windows and Linux builds work fine. The OSX build fails while detecting the C compiler for some reason:
I didn't touch the OSX parts though and the OSX configuration files are unchanged, except that a Any suggestions? |
For some reason, pip installs cmake, setuptools and wheel; I reckon this causes trouble. This was not the case in the latest build on main.
|
d1c8bcb
to
6bb6a49
Compare
I tried to explicitly specify
|
This fixes the OSX build.
…nda-forge-pinning 2023.12.14.20.48.22
6bb6a49
to
526bfab
Compare
I'm not sure if the issue is related to conda/conda-build#3993 or if it is caused by something else. I modified |
Nice work! Is there any way to check whether cuda support is enabled in the test section to avoid future regression? |
We should also use track features to prefer the cuda builds (or alternatively increase the build number for cuda builds by 100 |
And finally, totally optional, you might want to consider manually adding cuda 11.8 and 12 migrations |
I based most of the modification on the recipe of tensorflow: It looks like pytorch has them? https://github.com/conda-forge/pytorch-cpu-feedstock/blob/main/recipe/meta.yaml Neither of the recipes has CUDA 11.8/CUDA 12 support, so I'd prefer to add the migrator later. Neither does check for GPU support in the tests. I don't know if checking for GPU support is even possible in the CI. |
If you want track features to be used, can you show me a recipe that does them in the way that is state-of-the-art on Conda Forge? |
@conda-forge-admin, please rerender |
…nda-forge-pinning 2023.12.17.14.24.54
@conda-forge-admin, please rerender |
…nda-forge-pinning 2023.12.17.14.24.54
@conda-forge-admin, please rerender |
…nda-forge-pinning 2023.12.17.14.24.54
I've added the |
Hi! This is the friendly conda-forge automerge bot! I considered the following status checks when analyzing this PR:
Thus the PR was passing and merged! Have a great day! |
Thanks for merging and having a look. 🙂 |
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)Resolves #54