yongwww commented on code in PR #18697:
URL: https://github.com/apache/tvm/pull/18697#discussion_r2742660652


##########
LICENSE:
##########
@@ -192,7 +192,7 @@
    you may not use this file except in compliance with the License.
    You may obtain a copy of the License at
 
-       http://www.apache.org/licenses/LICENSE-2.0
+       https://www.apache.org/licenses/LICENSE-2.0

Review Comment:
   why do we need to update license here?



##########
docs/how_to/tutorials/customize_opt.py:
##########
@@ -55,6 +55,12 @@
 from tvm import IRModule, relax
 from tvm.relax.frontend import nn
 
+# Note: This tutorial requires TVM to be built with CUDA support.
+# If you encounter 'No module named ''tvm_ffi''' error,

Review Comment:
   The statement might not be correct, tvm_ffi is now a independent repo (and 
3rdparty in tvm), I think we just need to pip install tvm-ffi if we find the 
`No module named tvm_ffi` error. 



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to