Skip to content
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

Deprecate cub::Traits #3384

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bernhardmgruber
Copy link
Contributor

TODO: This PR contains a lot more changes at the moment to test the feasibility of this deprecation.

Fixes: #3381

TODO: This PR contains a lot more changes at the moment to test the feasibility of this deprecation.

Fixes: NVIDIA#3381
Copy link

copy-pr-bot bot commented Jan 14, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@bernhardmgruber bernhardmgruber added the cub For all items related to CUB label Jan 14, 2025
@bernhardmgruber
Copy link
Contributor Author

/ok to test

@bernhardmgruber
Copy link
Contributor Author

@miscco I would love to deprecate cub::Traits in favor of standard facilities in libcu++. As it currently stands, we would still need:

  • support for FP16, BF16 and FP8 types by cuda::std::is_floating_point
  • support for FP16, BF16 and FP8 types by cuda::std::numeric_limits (only min and lowest)

Do you think it's possible we can have this support soonish?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cub For all items related to CUB
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

Deprecate cub::Traits
1 participant