Add CUDA 11.8 - Githubissues

conda-forge / conda-forge-ci-setup-feedstock

A conda-smithy repository for conda-forge-ci-setup.

BSD 3-Clause "New" or "Revised" License

13 stars 51 forks source link

Closed m3vaz closed 1 year ago

m3vaz commented 1 year ago

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase code>@<space/conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

conda-forge-linter commented 1 year ago

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 (recipe) and found it was in an excellent condition.

m3vaz commented 1 year ago

@conda-forge-admin, please rerender

beckermr commented 1 year ago

@jakirkham Can you check this one and merge it?

jakirkham commented 1 year ago

@jaimergp, could you please review as well?

jakirkham commented 1 year ago

Going to go ahead and merge to keep things moving here. Though happy to follow up on anything needed

jakirkham commented 1 year ago

Thanks Mark and Matt! 🙏

leofang commented 1 year ago

@m3vaz I think this component should be installed too: cuda_profiler_api_11.8.

leofang commented 1 year ago

Though it does not explain why CUDA Python 11.7.1 could be compiled fine...

jakirkham commented 1 year ago

Maybe some files got shuffled between components?

leofang commented 1 year ago

Maybe some files got shuffled between components?

Indeed. By extracting the CUDA 11.7.0 installer I see the headers are part of cuda_nvprof:

cuda_nvprof/nvprof/include/cudaProfiler.h
cuda_nvprof/nvprof/include/cuda_profiler_api.h

but with CUDA 11.8.0 they have their own directory cuda_profiler_api:

cuda_profiler_api/cuda_profiler_api/include/cudaProfiler.h
cuda_profiler_api/cuda_profiler_api/include/cuda_profiler_api.h

cc: @m3vaz

leofang commented 1 year ago

Let's see if #221 would fix it.