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

fix requirements.txt path #22946

Merged
merged 4 commits into from
Dec 4, 2024
Merged

fix requirements.txt path #22946

merged 4 commits into from
Dec 4, 2024

Conversation

fs-eire
Copy link
Contributor

@fs-eire fs-eire commented Nov 26, 2024

Description

#22380 removes the file tools/ci_build/github/linux/docker/inference/x86_64/python/cpu/scripts/requirements.txt but it is still used in dockerfiles/Dockerfile.cuda.

This change updates the file path of the requirements.txt

fixes #22945.

@fs-eire fs-eire merged commit 7b0fa40 into main Dec 4, 2024
95 checks passed
@fs-eire fs-eire deleted the fs-eire/fix-requirements-txt-path branch December 4, 2024 21:08
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
### Description

microsoft#22380 removes the file
`tools/ci_build/github/linux/docker/inference/x86_64/python/cpu/scripts/requirements.txt`
but it is still used in `dockerfiles/Dockerfile.cuda`.

This change updates the file path of the requirements.txt

fixes microsoft#22945.
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
### Description

microsoft#22380 removes the file
`tools/ci_build/github/linux/docker/inference/x86_64/python/cpu/scripts/requirements.txt`
but it is still used in `dockerfiles/Dockerfile.cuda`.

This change updates the file path of the requirements.txt

fixes microsoft#22945.
tarekziade pushed a commit to tarekziade/onnxruntime that referenced this pull request Jan 10, 2025
### Description

microsoft#22380 removes the file
`tools/ci_build/github/linux/docker/inference/x86_64/python/cpu/scripts/requirements.txt`
but it is still used in `dockerfiles/Dockerfile.cuda`.

This change updates the file path of the requirements.txt

fixes microsoft#22945.
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 this pull request may close these issues.

[Build] Error when trying to build docker image of ONNXRuntime with CUDA support
2 participants