tacit/nanochat
1
0
Fork 0
mirror of https://github.com/karpathy/nanochat.git synced 2025-12-29 15:52:48 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix AMD Triton re-installation issue in speedrun.sh

Browse Source 
On AMD ROCm environments, `uv run` was detecting that the manually uninstalled `triton` package was missing (since it's a transitive dependency of `torch`) and reinstalling it during the tokenizer build step. This caused `ImportError: cannot import name 'Config' from 'triton'` due to conflict with `pytorch-triton-rocm`.

This change adds `--no-sync` to the `uv run` command for building the tokenizer, preventing `uv` from undoing the manual uninstallation of `triton`.
This commit is contained in:
google-labs-jules[bot] 2025-11-23 04:26:32 +00:00
parent 054394c708
commit d291a62ad8
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
speedrun.sh
View File

@ -74,7 +74,8 @@ curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y
source "$HOME/.cargo/env" source "$HOME/.cargo/env"
# Build the rustbpe Tokenizer # Build the rustbpe Tokenizer
uv run --extra $EXTRAS maturin develop --release --manifest-path rustbpe/Cargo.toml # use --no-sync to avoid re-installing triton on AMD, which we just uninstalled
uv run --no-sync --extra $EXTRAS maturin develop --release --manifest-path rustbpe/Cargo.toml
# Download the first ~2B characters of pretraining dataset # Download the first ~2B characters of pretraining dataset
# look at dev/repackage_data_reference.py for details on how this data was prepared # look at dev/repackage_data_reference.py for details on how this data was prepared