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

[TensorRT EP] New CIs to test TRT+minimal CUDA build #23028

Merged
merged 8 commits into from
Dec 19, 2024

Conversation

yf711
Copy link
Contributor

@yf711 yf711 commented Dec 5, 2024

Description

New CI: Linux_TRT_Minimal_CUDA_Test_CI and Win_TRT_Minimal_CUDA_Test_CI

Setting config for new CI to monitor if there's no issue to build ORT-TRTEP with minimal CUDA

Motivation and Context

Monitor if user is able to build ORT-TRTEP-minimalCUDA without any blocker
(which takes ~30min to build)

@yf711 yf711 requested a review from a team as a code owner December 5, 2024 19:51
@yf711 yf711 requested review from snnn and jywu-msft December 5, 2024 20:00
@chilo-ms
Copy link
Contributor

chilo-ms commented Dec 9, 2024

Could you also help add a Windows CI for minimal CUDA build?
Since Nvidia provides prebuilt binaries for both Linux and Windows.
https://github.com/NVIDIA/onnxruntime/releases/tag/v1.0

@yf711 yf711 changed the title [TensorRT EP] New CI to test TRT+minimal CUDA build [TensorRT EP] New CIs to test TRT+minimal CUDA build Dec 9, 2024
@yf711 yf711 requested a review from chilo-ms December 9, 2024 18:40
@chilo-ms
Copy link
Contributor

chilo-ms commented Dec 10, 2024

The Windows CI seems having issue downloading CUDA SDK v12.2.

@yf711
Copy link
Contributor Author

yf711 commented Dec 11, 2024

The Windows CI seems having issue downloading CUDA SDK v12.2.

The CI is now fixed with correct auth config. Will init another test on this new CI

@yf711 yf711 merged commit a3bb3f1 into main Dec 19, 2024
96 checks passed
@yf711 yf711 deleted the yifanl/test_cuda_minimal_ci branch December 19, 2024 18:30
guschmue pushed a commit that referenced this pull request Dec 20, 2024
### Description
<!-- Describe your changes. -->
New CI:
[Linux_TRT_Minimal_CUDA_Test_CI](https://dev.azure.com/onnxruntime/onnxruntime/_build?definitionId=230&_a=summary)
and [Win_TRT_Minimal_CUDA_Test_CI
](https://dev.azure.com/onnxruntime/onnxruntime/_build?definitionId=231)
Setting config for new CI to monitor if there's no issue to build
ORT-TRTEP with minimal CUDA
* yaml content is following Linux TRT CI yaml, with different build
arg/cache name
* build arg is following [[TensorRT EP] Enable a minimal CUDA EP
compilation without
kernels](#19052 (comment))



### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Monitor if user is able to build ORT-TRTEP-minimalCUDA without any
blocker
(which takes ~30min to build)
tarekziade pushed a commit to tarekziade/onnxruntime that referenced this pull request Jan 10, 2025
### Description
<!-- Describe your changes. -->
New CI:
[Linux_TRT_Minimal_CUDA_Test_CI](https://dev.azure.com/onnxruntime/onnxruntime/_build?definitionId=230&_a=summary)
and [Win_TRT_Minimal_CUDA_Test_CI
](https://dev.azure.com/onnxruntime/onnxruntime/_build?definitionId=231)
Setting config for new CI to monitor if there's no issue to build
ORT-TRTEP with minimal CUDA
* yaml content is following Linux TRT CI yaml, with different build
arg/cache name
* build arg is following [[TensorRT EP] Enable a minimal CUDA EP
compilation without
kernels](microsoft#19052 (comment))



### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Monitor if user is able to build ORT-TRTEP-minimalCUDA without any
blocker
(which takes ~30min to build)
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.

3 participants