-
Notifications
You must be signed in to change notification settings - Fork 6.8k
[CI] run operator tests with naive engine #18252
Conversation
Hey @szha , Thanks for submitting the PR
CI supported jobs: [centos-gpu, miscellaneous, windows-cpu, website, sanity, windows-gpu, clang, centos-cpu, unix-cpu, edge, unix-gpu] Note: |
f721ca4
to
e585f73
Compare
a009067
to
41fd8b1
Compare
What's our strategy to test multi threaded engine? |
@marcoabreu all but the operator unit tests still run on threaded engine. |
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
hey @szha , this commit breaks the nightly CD pipeline. tests/python/unittest/test_extensions.py::test_subgraph should be SKIPPED, but after this commit it is not getting skipped. Basically, this change is not working as expected: 10b6b48#diff-460d06cb3af2af996c5ec259b086712fR100 Can you please fix this? |
* run operator tests with naive engine * fix take tests * update skip mark * fix cuda error reset * adjust tests * disable parallel testing and naive engine for mkl/mkldnn apache#18244
I realize this PR is over a year old, but does anyone remember the original problem being solved, i.e. why can't the operator unittests run with the threaded_engine_per_device? Seems like if we're only running the unittests once, it should be with the default engine. |
Description
run operator tests with naive engine
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes