-
Notifications
You must be signed in to change notification settings - Fork 618
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
Disable JAX iterator tests in ASAN build #4920
Conversation
Signed-off-by: Albert Wolant <awolant@nvidia.com>
CI MESSAGE: [8711952]: BUILD STARTED |
CI MESSAGE: [8717952]: BUILD STARTED |
qa/TL0_FW_iterators/test_jax.sh
Outdated
# it takes very long time to run it with sanitizers on and provides little value so turn it off | ||
if [ -z "$DALI_ENABLE_SANITIZERS" ]; then | ||
# General tests for iterators | ||
${python_invoke_test} -m '(?:^|[\b_\./-])[Tt]est.*jax*' test_fw_iterators.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.
I think that works fine, the bellow never happens. I would keep it as a basic test for integration.
qa/TL0_FW_iterators/test_jax.sh
Outdated
# it takes very long time to run it with sanitizers on and provides little value so turn it off | ||
if [ -z "$DALI_ENABLE_SANITIZERS" ]; then | ||
# General tests for iterators | ||
${python_invoke_test} -m '(?:^|[\b_\./-])[Tt]est.*jax*' test_fw_iterators.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.
# it takes very long time to run it with sanitizers on and provides little value so turn it off | |
if [ -z "$DALI_ENABLE_SANITIZERS" ]; then | |
# General tests for iterators | |
${python_invoke_test} -m '(?:^|[\b_\./-])[Tt]est.*jax*' test_fw_iterators.py | |
# General tests for iterators | |
${python_invoke_test} -m '(?:^|[\b_\./-])[Tt]est.*jax*' test_fw_iterators.py | |
# it takes very long time to run it with sanitizers on and provides little value so turn it off | |
if [ -z "$DALI_ENABLE_SANITIZERS" ]; then |
CI MESSAGE: [8711952]: BUILD FAILED |
CI MESSAGE: [8717952]: BUILD FAILED |
Signed-off-by: Albert Wolant <awolant@nvidia.com>
!build |
CI MESSAGE: [8728335]: BUILD STARTED |
d68bcad
to
6fa06eb
Compare
CI MESSAGE: [8728335]: BUILD PASSED |
* Disable JAX iterator tests in ASAN build Signed-off-by: Albert Wolant <awolant@nvidia.com>
Category:
Bug fix
Description:
Running frameworks iterators tests with sanitizers enabled takes a long time, may timeout the job and does not provide that much value.
This PR disables JAX iterator tests when sanitizer is enabled.
Additional information:
Affected modules and functionalities:
JAX iterator tests.
Key points relevant for the review:
Did I run CI pipeline with ASAN?
Tests:
Checklist
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: N/A