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

CUDA 12 Migration includes Unsupported Windows #1698

Closed
carterbox opened this issue May 31, 2023 · 4 comments · Fixed by conda-forge/conda-forge-pinning-feedstock#4526
Closed

Comments

@carterbox
Copy link

carterbox commented May 31, 2023

conda-forge/libmagma-feedstock#4

The CUDA 12 Migration bot doesn't work for Windows. It fails on the conda-forge build setup step with the following error message:

2023-05-31T20:19:59.2050124Z ##[section]Starting: conda-forge build setup
2023-05-31T20:19:59.2216625Z ==============================================================================
2023-05-31T20:19:59.2217082Z Task         : Command line
2023-05-31T20:19:59.2217230Z Description  : Run a command line script using Bash on Linux and macOS and cmd.exe on Windows
2023-05-31T20:19:59.2217374Z Version      : 2.212.0
2023-05-31T20:19:59.2217493Z Author       : Microsoft Corporation
2023-05-31T20:19:59.2217671Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/command-line
2023-05-31T20:19:59.2217818Z ==============================================================================
2023-05-31T20:20:00.3475222Z Generating script.
2023-05-31T20:20:00.3880761Z ========================== Starting Command Output ===========================
2023-05-31T20:20:00.4170133Z ##[command]"C:\Windows\system32\cmd.exe" /D /E:ON /V:OFF /S /C "CALL "D:\a\_temp\08a183b4-2c66-4613-a3be-6c3bfa7cc469.cmd""
2023-05-31T20:20:12.0113688Z Could Not Find C:\Windows\System32\msmpi.dll
2023-05-31T20:20:12.0120362Z Could Not Find C:\Windows\System32\msmpires.dll
2023-05-31T20:20:12.1608678Z CUDA '12.0' is not supported
2023-05-31T20:20:12.1613229Z Could not install CUDA
2023-05-31T20:20:12.2546635Z ##[error]Cmd.exe exited with code '1'.
2023-05-31T20:20:12.2948615Z ##[section]Finishing: conda-forge build setup

IMHO, the migration should be halted until all the Windows bits are in place (otherwise, packages will need to be migrated twice), and/or the bot should automatically add a skip statement like:

skip: True  # [win and cuda_compiler_version == "12.0"]
@beckermr
Copy link
Contributor

Cc @jakirkham @h-vetinari

@jakirkham
Copy link
Contributor

Thank for the ping! 🙏

Yep will take a look

@jakirkham
Copy link
Contributor

jakirkham commented May 31, 2023

Submitted PR ( conda-forge/conda-forge-pinning-feedstock#4526 ), which should do the trick

Tried re-rendering PR ( conda-forge/libmagma-feedstock#4 ) with those changes locally and it removed the Windows CUDA 12 variant (and related bits)

@jakirkham
Copy link
Contributor

After merging that PR, tried re-rendering the libmagma feedstock and it appears to be doing the right thing now ( conda-forge/libmagma-feedstock#4 (comment) )

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 a pull request may close this issue.

3 participants