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

[onnxruntime] Fix CUDA build #137

Merged
merged 7 commits into from
Dec 10, 2023
Merged

[onnxruntime] Fix CUDA build #137

merged 7 commits into from
Dec 10, 2023

Conversation

luncliff
Copy link
Owner

@luncliff luncliff commented Dec 9, 2023

Changes

Fix #136

Tested with CUDA Toolkit 12.3.
For some reason, MSBuild projects can't detect CUDA toolkit paths and NVCC outputs.
Tried to provide CMAKE_VS_PLATFORM_TOOLSET_CUDA and some variables but they didn't either.

  • Use the default CMake generator option(Ninja) in configure vcpkg_cmake_configure.
  • Patch fix-cuda.patch to add path information for CUPTI library which is installed under "${CUDAToolkit_LIBRARY_ROOT}/extras/CUPTI/"

Print more messages in portfile.cmake

References

Triplet Support

  • x64-windows

Configuration

"vcpkg-configuration.json" changes for the release.

{
    "registries": [
        {
            "kind": "git",
            "repository": "https://github.com/luncliff/vcpkg-registry",
            "packages": [
                "onnxruntime"
            ],
            "baseline": "..."
        }
    ]
}

* create a new workflow
* update run-vcpkg and vcpkgJsonGlob
* provide option for CUPTI
* more script messages
@luncliff luncliff added platform: windows Windows, UWP ci: GitHub Actions GitHub Actions (CI) configuration changes labels Dec 9, 2023
@luncliff luncliff self-assigned this Dec 9, 2023
@luncliff luncliff merged commit 7f21dd8 into main Dec 10, 2023
18 of 20 checks passed
@luncliff luncliff deleted the port/onnxruntime branch December 10, 2023 01:17
@luncliff luncliff added the bug label Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug ci: GitHub Actions GitHub Actions (CI) configuration changes platform: windows Windows, UWP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[onnxruntime[cuda]]No CUDA toolset found.
1 participant