Skip to content

Commit ea0c193

Browse files
committed
Address PR comments: add -U to pip install, fix grammar
Signed-off-by: Meng Xin <mxin@nvidia.com>
1 parent 429b4f6 commit ea0c193

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

.claude/skills/ptq/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ Report the path and size to the user.
124124

125125
## Common Pitfalls
126126

127-
- **Transformers version**: New models may need a newer transformers than what's installed. Check `config.json` for `transformers_version`. In containers, beware of `PIP_CONSTRAINT` blocking upgrades — see `references/slurm-setup-ptq.md` for workarounds
127+
- **Transformers version**: New models may need a newer version of transformers than what's installed. Check `config.json` for `transformers_version`. In containers, beware of `PIP_CONSTRAINT` blocking upgrades — see `references/slurm-setup-ptq.md` for workarounds
128128
- **Gated datasets**: Some calibration datasets require HF authentication. Ensure `HF_TOKEN` is set in the job environment, or use `--dataset cnn_dailymail` as a non-gated alternative
129129
- **NFS root_squash + Docker**: See `skills/common/slurm-setup.md` section 5
130130

.claude/skills/ptq/references/slurm-setup-ptq.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,13 +46,13 @@ If `PYTHONPATH` doesn't work due to missing compiled extensions, fall back to `p
4646

4747
```bash
4848
unset PIP_CONSTRAINT
49-
pip install transformers # now resolves with new deps included
49+
pip install -U transformers # now upgrades and resolves with new deps included
5050
```
5151

5252
If that still conflicts, fall back to `--no-deps` (skips new deps — may need to add missing ones manually):
5353

5454
```bash
55-
pip install transformers --no-deps
55+
pip install -U transformers --no-deps
5656
```
5757

5858
---

0 commit comments

Comments
 (0)