-
Notifications
You must be signed in to change notification settings - Fork 188
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
[FEA]: Replace thrust
type traits with libcu++
#487
Comments
I'm all for this and would love to see it prioritized. This is a blocker for addressing #403. Based on my study (cupy/cupy#7869), Thrust is extremely NVRTC unfriendly because thrust/detail/type_traits.h directly/indirectly includes a ton of host code that just paralyzes NVRTC. Fixing at least this header moves us one step closer. |
Hi, never contributed before, but this one looks like a good first issue. I will try to tackle this on my own time if that is ok :) |
Hi @ZelboK that is awesome! Please do not hesitate to ping us here if there are any questions arising. We are painfully aware that our documentation currently is not up to date |
Hey, our contributor guide docs are up to date! @ZelboK you can check out how to get started here: https://github.com/NVIDIA/cccl/blob/main/CONTRIBUTING.md To start, I'd suggest making sure you can set up the development containers and can successfully build the tests. |
Thank you folks! Is there some slack channel or discord server that I can join, or is this the preferred discussion channel? I currently use WSL2 and a 3080. Currently following the docs on building the tests. Hopefully I won't run into too many troubles with code generation on older architectures. I plan on doing this independently before I ask questions for the learning experience :) |
Not yet, I'm working on getting us a dedicated channel on the NVIDIA Discord. I'll let you know when I've got that done :)
This should work as there are folks on our team who use WSL2 with the Dev Containers. It's on my TODO list to add a section for any WSL-specific instructions to the dev container README. Let us know if you run into any problems! |
Hey @ZelboK, you can join the #cuda-cpp-core-libraries channel on the NVIDIA Discord here: https://discord.gg/nvXdfvRh |
Joined! Thank you that was quick :) My discord IGN is |
Is this a duplicate?
Area
Thrust
Is your feature request related to a problem? Please describe.
Thrust predates our investments into libcu++.
Consequently it has a lot of machinery implemented on its own. Mostly in https://github.com/NVIDIA/cccl/blob/main/thrust/thrust/detail/type_traits.h
We should remove all custom type traits in favor of
cuda::std::
implementationsDescribe the solution you'd like
We should simply include the
cuda/std/type_traits
header and use aliases whenever possible, e.g. thisWould become:
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: