-
Notifications
You must be signed in to change notification settings - Fork 615
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 wheel bundling with cvcuda for debug builds #4959
Conversation
9090031
to
4f938ee
Compare
CI MESSAGE: [9175202]: BUILD STARTED |
4f938ee
to
a204701
Compare
CI MESSAGE: [9175202]: BUILD FAILED |
Signed-off-by: Rafal Banas <rbanas@nvidia.com>
a204701
to
f6c1066
Compare
CI MESSAGE: [9175793]: BUILD STARTED |
dali/python/bundle-wheel.sh
Outdated
"lib/libcvcuda_d.so.0" # debug build | ||
"lib/libnvcv_types_d.so.0" # debug build |
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.
May I ask for making this comment more descriptive?
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.
# cvcuda adds _d suffix to lib names for debug builds
"lib/libcvcuda_d.so.0"
"lib/libnvcv_types_d.so.0"
descriptive enough?
CI MESSAGE: [9175793]: BUILD PASSED |
Signed-off-by: Rafal Banas <rbanas@nvidia.com>
CI MESSAGE: [9189881]: BUILD STARTED |
CI MESSAGE: [9189881]: BUILD PASSED |
Signed-off-by: Rafal Banas <rbanas@nvidia.com>
It fixes wheel bundling for debug builds with cv-cuda.
Category:
Other
Description:
I fixes CI
Additional information:
Affected modules and functionalities:
Key points relevant for the review:
Tests:
Checklist
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: N/A