Conversation
yushangdi
requested review from
fegin,
tianyu-l,
wconstab and
wwwjn
as code owners
meta-cla
Bot
added
the
CLA Signed
| cd torchtitan | ||
| pip install -r requirements.txt | ||
| pip install --pre torchdata --index-url https://download.pytorch.org/whl/nightly/cpu | ||
| pip install -e . --no-deps |
There was a problem hiding this comment.
curious why you'd need this?
There was a problem hiding this comment.
we're trying to land some runtimes estimators for the graph trainer in https://github.com/meta-pytorch/torchinsights/tree/main. For some components, the estimator wants to import from torchtitan. Without install it as a pacakage locally, import from torchtitan doesn't work.
The estimator #3828 (note that though it's currently a torchtitan PR, the plan is to land in torchinsights instead).
There was a problem hiding this comment.
btw the other two sections Nightly builds and Stable releases both install torchtitan as a package. But the From Source doesn't, it only installs the dependencies.
There was a problem hiding this comment.
btw the other two sections Nightly builds and Stable releases both install torchtitan as a package. But the From Source doesn't, it only installs the dependencies.
That's right, because the source doesn't require torchtitan install to use.
we're trying to land some runtimes estimators for the graph trainer in https://github.com/meta-pytorch/torchinsights/tree/main.
If it's another repo which depends on torchtitan, then I agree it's better to install.
I vaguely remember there's some caveat installing it even with -e, but I'll stamp since it's only readme.
| cd torchtitan | ||
| pip install -r requirements.txt | ||
| pip install --pre torchdata --index-url https://download.pytorch.org/whl/nightly/cpu | ||
| pip install -e . --no-deps |
There was a problem hiding this comment.
btw the other two sections Nightly builds and Stable releases both install torchtitan as a package. But the From Source doesn't, it only installs the dependencies.
That's right, because the source doesn't require torchtitan install to use.
we're trying to land some runtimes estimators for the graph trainer in https://github.com/meta-pytorch/torchinsights/tree/main.
If it's another repo which depends on torchtitan, then I agree it's better to install.
I vaguely remember there's some caveat installing it even with -e, but I'll stamp since it's only readme.
2 participants
as title
The
pip install -e . --no-depsactually installs torchtitan as a local importable package.