-
Notifications
You must be signed in to change notification settings - Fork 197
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] Allow RMM to Handle Using Conda Compilers #539
Comments
What are the breakages? |
It used to be breaking exactly like #315 but as I just tested it, the |
Looks like I used an incorrect version of conda's gcc. I'll offload this debugging to one of the members on ops I did a knowledge transfer with and they'll take it from here. |
This issue has been marked rotten due to no recent activity in the past 90d. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed. |
This issue has been marked stale due to no recent activity in the past 30d. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed. This issue will be marked rotten if there is no activity in the next 60d. |
@kkraus14 @aleksficek @mike-wendt is this still relevant? |
Yes, still relevant |
This issue has been labeled |
The current plan is to adopt conda compilers for the RAPIDS libraries in 0.20 when CUDA 10.X support is dropped |
Thanks for the update Mike! 😄 |
This issue has been labeled |
@beckernick I saw you closed the cuDF equivalent of this issue. Is this issue still relevant or can it be closed? |
cuDF can build with the conda compilers, but I don't know if RMM can currently. We may want to leave this open until we verify. |
I can confirm that building RMM CUDA / C++ tests and building the RMM Cython extensions works as expected with the conda compilers with version 9.3.0. |
This is also how I've built RMM in the past (though haven't done this recently). Given the C++ side of RMM is header only, suspect this is a non-issue from that side. |
Thanks @kkraus14 and @jakirkham . Closing. |
Is your feature request related to a problem? Please describe.
Currently using conda compilers with RMM results in breakages. RAPIDS is posed to take advantage of anaconda compiler versions of
gcc
andg++
(https://docs.conda.io/projects/conda-build/en/latest/resources/compiler-tools.html) in the future. This is part of an overarching effort in project scout where developers will be able to one day install the scout conda package which includesgcc
,g++
andnvcc
conda compilers at specific versions with ccache hooked up for fast compilation.Describe the solution you'd like
This is similar to the issue and fix as part of #315 but based on my own testing in RMM a similar failure has since resurfaced.
Steps to simulate failure
Conda install conda compilers
gcc_linux-64
andgxx_linux-64
to conda environment. Then build the repo and see failures. If this does not simulate the error then the error can be reproduced by first pulling thegpuci/rapidsai:0.15-cuda10.2-devel-ubuntu16.04-py3.7
docker image, cloning the rmm repo, installing conda compilers and then building it.Additional context
After this is solved, the general plan of attack will be to integrate conda compilers into cuDF and then distribute the requests outwards to each other repo.
The text was updated successfully, but these errors were encountered: