Skip to content

Conversation

@eserscor
Copy link
Contributor

Description

Motivation and Context

Comment on lines 2 to 3
cuda12_trt_version: '10.9.0.34'
cuda13_trt_version: '10.13.3.9'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update these as well.

Please search these version numbers in files. Just in case you missed other files as well.

Copy link
Contributor Author

@eserscor eserscor Dec 22, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup, thanks! I figured I'd update Linux first then then will update windows. Once both are finished, I will publish the PR. Separating out the 'variables' portion here was intentional so I could work on a single platform at a time.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm, it looks like 12.8 is pretty pervasive. It might make more sense to update 12.8 -> 12.9 then do the TRT update.

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