-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Enable unit tests for TVM ops for all cuda compute capabilities #16824
Conversation
This affects 1.6, right? I encountered similar errors ( |
@ptrendx This does not affect 1.6. We plan not to release TVM powered operators in 1.6. If you see the invalid ptx error, it's probably you invoked an operator powered by TVM, for example, |
Hmmm, so it looks like a slightly different error with similar result - I did not do anything with fp16 - this is my test:
(note: I encountered it during fixing a bug in the pointwise fusion, that is why the test has
(link to the CI that failed: http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-gpu/detail/PR-16796/6/pipeline) |
@ptrendx It fails because of the following line where the F.np.ones((10,)) < valid_length You can skip the test by decorating the test function as @unittest.skipUnless(is_op_runnable(), 'skip cuda compute capability < 53 for TVM ops')
def test_fusion_boolean_inputs_debug_tvm():
... |
Ok, if it is expected onmaster and does not affect 1.6 release then that's fine. |
9dfb643
to
e29ba42
Compare
e29ba42
to
1faf352
Compare
Description
As a result of #16777.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
Comments