-
Notifications
You must be signed in to change notification settings - Fork 32
TensorRT Cuda Context Sharing #106
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is Windows only as per my understanding, CMakeLists.txt
doesn't reflect it and it cause a confusion.
Is it something that we consider to build by default then configuration change should reflect it accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Files in src
folder may be related not only to Windows but other platforms as well. We need to make sure it doesn't impact the container build.
@GuanLuo are we OK with it?
@saturley-hall have we tested it well in CI for other stacks are we confident with result?
This PR enabled cuda context sharing with CiG streams on Windows Gaming applications