-
Notifications
You must be signed in to change notification settings - Fork 350
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: Fix the CUDAGraphs C++ runtime implementation #3067
Conversation
37ea7c2
to
a6e3234
Compare
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.
8bd7f78
to
0c8f280
Compare
@@ -219,7 +219,7 @@ jobs: | |||
export USE_HOST_DEPS=1 | |||
pushd . | |||
cd tests/py/dynamo | |||
python -m pytest -n 4 --junitxml=${RUNNER_TEST_RESULTS_DIR}/tests_py_dynamo_core_runtime_test_results.xml runtime/ | |||
python -m pytest --junitxml=${RUNNER_TEST_RESULTS_DIR}/tests_py_dynamo_core_runtime_test_results.xml runtime/ |
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.
Update windows tests as well
43f446f
to
d7ee6e7
Compare
06241d2
to
96810f3
Compare
pushd . | ||
cd tests/py/dynamo | ||
python -m pytest -ra --junitxml=${RUNNER_TEST_RESULTS_DIR}/tests_py_dynamo_core_runtime_cudagraphs_cpp_test_results.xml runtime/test_002_cudagraphs_cpp.py | ||
#python -m pytest -ra --junitxml=${RUNNER_TEST_RESULTS_DIR}/tests_py_dynamo_core_runtime_cudagraphs_py_test_results.xml runtime/test_002_cudagraphs_py.py |
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.
Uncomment the py tests
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.
LGTM
export USE_HOST_DEPS=1 | ||
pushd . | ||
cd tests/py/dynamo | ||
nvidia-smi |
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.
need this for the logs ?
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.
Yeah
Signed-off-by: Naren Dasan <[email protected]> Signed-off-by: Naren Dasan <[email protected]>
96810f3
to
8971333
Compare
Description
Fixes the implementation intra-engine CUDAGraph support in the C++ runtime. Cleans up the implementation to distinctly handle creating and maintaining persistent buffers as well as adds explicit concurrency management between the calling stream and the engine execution stream.
Fixes #
Type of change
Please delete options that are not relevant and/or add your own.
Checklist: