ashutosh-arm commented on a change in pull request #10802:
URL: https://github.com/apache/tvm/pull/10802#discussion_r836452538



##########
File path: tests/python/contrib/test_vitis_ai/infrastructure.py
##########
@@ -59,7 +59,7 @@ def visit_call(self, call):
 
 def skip_test():

Review comment:
       Overall LGTM! I see that tvm has more or less a standard way of defining 
require_*() functions in this file and using those functions to skip the tests. 
Maybe you could define a similar function for vitis_ai.
   https://github.com/apache/tvm/blob/main/python/tvm/testing/utils.py




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to