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

minor fixes to kempner docs #354

Merged
merged 1 commit into from
Nov 2, 2023
Merged

minor fixes to kempner docs #354

merged 1 commit into from
Nov 2, 2023

Conversation

epwalsh
Copy link
Member

@epwalsh epwalsh commented Nov 1, 2023

No description provided.

@epwalsh epwalsh requested a review from dirkgr November 2, 2023 16:59
6. `conda create -y -n LLM python=3.10 ipython`
7. `conda activate LLM`
8. `conda install -y pytorch torchvision torchaudio pytorch-cuda=11.8 -c pytorch -c nvidia`
9. `git clone https://github.com/allenai/LLM.git`
10. `cd LLM`
11. `pip install -e .`
12. Set up S3 key env vars (in `~/.bashrc`)
Copy link
Member

Choose a reason for hiding this comment

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

Why did you remove this?

Copy link
Member

Choose a reason for hiding this comment

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

It's not strictly necessary, I guess, but it's very annoying later if you want to switch to reading things from S3, or do some checkpoint management, etc.

Copy link
Member Author

Choose a reason for hiding this comment

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

Step 4 is to run aws configure, so this is redundant.

Copy link
Member

Choose a reason for hiding this comment

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

Oh, hmm. Fair enough.

@epwalsh epwalsh marked this pull request as ready for review November 2, 2023 18:51
@epwalsh epwalsh merged commit a952f44 into main Nov 2, 2023
10 checks passed
@epwalsh epwalsh deleted the epwalsh/kempner-docs branch November 2, 2023 18:52
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