Skip to content

refresh cbc - update cuda compiler version#443

Merged
cbouss merged 5 commits intomasterfrom
PKG-13044
Apr 16, 2026
Merged

refresh cbc - update cuda compiler version#443
cbouss merged 5 commits intomasterfrom
PKG-13044

Conversation

@cbouss
Copy link
Copy Markdown
Contributor

@cbouss cbouss commented Apr 15, 2026

Following https://github.com/anaconda/aggregate-duct-tape/pull/38

Copy link
Copy Markdown
Contributor

@ELundby45 ELundby45 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

Comment thread conda_build_config.yaml Outdated
- cuda-nvcc # [linux or win]
cuda_compiler_version: # [linux or win]
- 12.9 # [linux or win]
- 13.1 # [linux or win]
Copy link
Copy Markdown
Contributor

@onurbingol onurbingol Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unpopular (but safer) option: What about using 13.0 instead of 13.1? The reason is simple: CUDA minor version compatibility.

I agree some libraries have to depend on 13.1, e.g. with CUDA Tile dependencies, and some of them, like cuda-bindings require specifying the exact CUDA version. Apart from these, for 13.x, majority of the libraries support the minor version compatibility (or advanced enhanced compatibility).

Use of 13.1 (or later) could be reconsidered in a couple of months, depending on the changes in CTK.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After some research, I agree with you.

The GPU feature list did not change between 13.0 and 13.1:
https://docs.nvidia.com/cuda/archive/13.0.0/cuda-compiler-driver-nvcc/index.html#gpu-feature-list
https://docs.nvidia.com/cuda/archive/13.1.0/cuda-compiler-driver-nvcc/index.html#gpu-feature-list

13.1 requires a slightly higher driver than 13.0.

This doc: https://github.com/AnacondaRecipes/cuda-feedstock/blob/main/recipe/doc/recipe_guide.md#cuda-enhanced-compatibility indicates we could build with 13.1 while keeping maximum range (provided we take the appropriate steps). However, identifying appropriate steps seems error prone.

There is also cuda-compat, but that is linux only.

Considering all this, setting the default to 13.0 does seem to be the right call.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ELundby45 @xkong-anaconda you're good with 13.0 instead of 13.1?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doc: https://github.com/AnacondaRecipes/cuda-feedstock/blob/main/recipe/doc/recipe_guide.md#cuda-enhanced-compatibility indicates we could build with 13.1 while keeping maximum range (provided we take the appropriate steps). However, identifying appropriate steps seems error prone.

That documentation is a hidden gem :)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm good with that. Thanks for flagging and investigating.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 on 13.0

Comment thread conda_build_config.yaml
@cbouss cbouss requested a review from onurbingol April 16, 2026 14:45
Copy link
Copy Markdown
Contributor

@onurbingol onurbingol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you very much @cbouss, @ELundby45 and @xkong-anaconda !

@cbouss cbouss merged commit b2b3922 into master Apr 16, 2026
@cbouss cbouss deleted the PKG-13044 branch April 16, 2026 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants