You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2022/06/29 01:19:05 UTC

[GitHub] [tvm] shingjan commented on a diff in pull request #11761: [TOPI][ONNX] Fix for trilu and set_matrix_diag ops

shingjan commented on code in PR #11761:
URL: https://github.com/apache/tvm/pull/11761#discussion_r909116854


##########
python/tvm/topi/transform.py:
##########
@@ -17,9 +17,12 @@
 # pylint: disable=invalid-name,consider-using-enumerate,redefined-outer-name
 """Injective transformation operators"""
 from __future__ import absolute_import as _abs
+import numpy as np
+from tables import Expr

Review Comment:
   are these two lines necessary? i am getting a `tables` not found on my end and it seems like those two libs aren't referenced anyway



-- 
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: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org