[GitHub] [tvm] areusch commented on issue #7751: Inconsistent python dependencies "tlcpack-sphinx-addon, synr" in CI and Docker setup scripts

[GitBox <gi...@apache.org>]

areusch commented on issue #7751:
URL: https://github.com/apache/tvm/issues/7751#issuecomment-808314918


   yeah I also hit this problem. I think we need to fix the CI image update process so it is not such a burden. My opinion is we should implement one method to include dependencies in TVM and support that method, and avoid making CI-time additions like these two. I hit this problem with updating ci-arm and ci-cpu containers as well.
   
   I think if we make the CI image update process less of a burden, it wouldn't be a big deal to ask us to update all ci-* containers at once, rather than having ci-cpu built 3 weeks ago and ci-gpu built 3 months ago.
   
   What's not clear to me in this particular case is why synr and tlcpack-sphinx-addon are pinned. @tqchen do you have more insight into that--is it just to ensure the CI is predictable (I.e. because these packages are installed every CI run rather than each container build), or was there another reason?


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