tkonolige commented on a change in pull request #7107:
URL: https://github.com/apache/tvm/pull/7107#discussion_r547540336
##########
File path: tests/scripts/task_ci_python_setup.sh
##########
@@ -31,3 +31,4 @@ set -o pipefail
echo "Addtiional setup in" ${CI_IMAGE_NAME}
python3 -m pip install --user tlcpack-sphinx-addon==0.1.3 synr==0.2.1
+python3 -m pip install --user tokenizers==0.9.4 transformers==4.0.1
Review comment:
Ok, I think I have this figured out. This tutorial depends on
transformers, but because the import was local to the download function (which
wasn't run), we didn't hit it. That means that we need to install transformers
in the ci_script (like you had done before). However, this model is pretty
large (500MB), so I don't think we want to be downloading it for every run on
CI. Do you have a small sparse model we could use instead? If not, I suggest we
leave the finally running of the script commented out and let the user run it
if they so choose.
----------------------------------------------------------------
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:
[email protected]
