[GitHub] [tvm] leandron commented on issue #9703: [Bug][CI] Python 3.6 used in TVM CI is reaching EOL

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

leandron commented on issue #9703:
URL: https://github.com/apache/tvm/issues/9703#issuecomment-1006450571


   > 1. What's the advantage of `deadsnakes` vs the default system repos for `python3.8` or a more complete Python packager like [miniconda](https://docs.conda.io/en/latest/miniconda.html)?
   
   I'm not very familiar with *conda distributions. I feel that `deadsnakes` is very quick into adding support for new versions of Python in a Ubuntu-friendly packaging, and also simple to install and to adapt into existing Docker images we have. That is why I suggested that.
   
   > 2. I would suggest we stick to the lowest supported version of Python available (meaning we should use Python 3.7 in this case), even if that means we have to upgrade more often. If we go to Python 3.8 it would be pretty easy for someone to accidentally break 3.7 support (e.g. using a `:=` operator) without anyone noticing until a user reports an error
   
   I'm OK with Python 3.7 as minimum (that is the minimum for e.g. _TensorFlow_ from now). My suggestions was mostly trying to avoid going with moving python again in months time.
   
   > * What else needs to be done other than update the docker images + fix any issues that come up with new dependency versions pip might pick up?
   
   I think that summarises all that needs doing: adjust the docker images, and in case the python version to be used is not the default version (the one called when invoking `python3`) in the system, we probably need to have a way to point to what the python binary file is, e.g. `python3.7` for example.
   
   > 3. * tlcpack in general looks pretty anemic, maybe now would be a good time to increase support for more python/system versions (e.g. conda only has 3.6-3.8, no pip wheels are available for Linux), though I don't have any of the historic context on these things
   
   In `tlcpack`, things are pretty much implement on an as-needed basis. Certainly needs expanding to cover more platforms.
   
   As @mshawcroft mentioned, perhaps this discussion is more about moving Ubuntu versions, rather than Python versions, but I didn't do any investigation in that direction.
   
   


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