search

tenstorrent / tt-metal

:metal: TT-NN operator library, and TT-Metalium low level kernel programming model.
Apache License 2.0
466 stars 73 forks source link

Add `python-is-python3` package to docker images #12225

Open asrinivasanTT opened 2 months ago

asrinivasanTT commented 2 months ago

The current docker images require the user to run python3 instead of just python, this is because the apt package python-is-python3 is missing. The package just adds a link, making /usr/bin/python to point to /usr/bin/python3. Please add this package to the docker images.

tt-rkim commented 2 months ago

Before we start blindly adding stuff, is there any reason that you couldn't do this in your image that you create after using ours as a base?

cc: @ttmchiou

asrinivasanTT commented 2 months ago

We do that already. We feel that it'll be useful to have it in because other users might be using this image as a base as well.

rreece commented 2 months ago

We should nudge the company to setting up python to be the python we use.

tt-rkim commented 2 months ago

Hmm I see - so you're saying industry-wide people prefer python as the executable name?

ttmchiou commented 2 months ago

I'm not opposed to this change since I believe tt-metal as a whole just uses python3 (someone correct me if I'm wrong). It can eliminate inconsistency since ubuntu 20.04 python defaults to python2.

I think this package also keeps track if python versions get updated too?