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 2021/02/24 02:39:13 UTC

[GitHub] [tvm] slyubomirsky commented on a change in pull request #7503: [Rust] Make TVM Rust bindings installable via Cargo.

slyubomirsky commented on a change in pull request #7503:
URL: https://github.com/apache/tvm/pull/7503#discussion_r581574950



##########
File path: python/tvm/relay/frontend/tensorflow.py
##########
@@ -914,10 +914,11 @@ def _impl(inputs, attr, params, mod):
 
 
 def _sparse_tensor_dense_matmul():
-    # Sparse utility from scipy
-    from scipy.sparse import csr_matrix
-
     def _impl(inputs, attr, params, mod):
+        # Loading this by default causes TVM to not be loadable from other languages.

Review comment:
       Just curious, what kind of error does loading scipy cause for other bindings?




----------------------------------------------------------------
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.

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