Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update README for updated docker images #328

Merged
merged 2 commits into from
Jul 28, 2023
Merged

Conversation

winglian
Copy link
Collaborator

No description provided.

README.md Outdated
@@ -107,7 +106,7 @@ accelerate launch scripts/finetune.py examples/openllama-3b/lora.yml \

3. Install torch
```bash
pip3 install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cu118
pip3 install torch==2.0.1+cu118 --index-url https://download.pytorch.org/whl/cu118
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
pip3 install torch==2.0.1+cu118 --index-url https://download.pytorch.org/whl/cu118
pip3 install -U torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cu118

I think you do not need to specify the version of torch as latest is 2.0.1 one. Setting that index url also sets the cuda version. The thing I forgot to add was the -U

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, I've updated it with the -U and removed the explicit version. I'm going to keep the removal of torchvision and torchaudio since those aren't needed I don't think

- `winglian/axolotl-runpod:main-py3.9-cu118-2.0.0-gptq`: for gptq
- `winglian/axolotl:dev`: dev branch (not usually up to date)
- `winglian/axolotl-runpod:main-py3.10-cu118-2.0.1`: for runpod
- `winglian/axolotl-runpod:main-py3.9-cu118-2.0.1-gptq`: for gptq
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason gptq is 3.9 vs the run pod 3.10?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't touched gptq in a while, so I'd rather leave it at 3.9 where it is believed to still work, rather than trying to upgrade and introduce another potential issue

@winglian winglian merged commit 41a4d15 into main Jul 28, 2023
6 checks passed
@winglian winglian deleted the update-readme-new-dockers branch July 31, 2023 13:32
mkeoliya pushed a commit to mkeoliya/axolotl that referenced this pull request Dec 15, 2023
* update README for updated docker images

* update readme from pr feedback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants