Skip to content

Conversation

@tianyu-l
Copy link
Contributor

Running python or torchrun with -m option helps resolve package structure without the need to do pip install -e.

This is suggested by @yzhangcs in #897 (comment)

It avoids the potential pitfalls of using pip install -e ., which can sometimes lead to inconvenient and hard-to-debug issues, especially when we're making frequent changes to the codebase.

@tianyu-l tianyu-l merged commit f3943dd into main Mar 24, 2025
6 checks passed
@tianyu-l tianyu-l deleted the entry branch March 24, 2025 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Moving train.py to torchtitan submodule makes run_train.sh failed with "Can not find module"

4 participants