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

Add notebook for setting up LLM training environment #78

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

meoflynn
Copy link
Contributor

This PR adds a new LLM specific notebook. This is similar to the pytorch notebook, but with a script containing installation steps that are required for the training environment.

Using the pytorch notebook as a base for the llm notebook.
Created an empty .sh file for adding installation script
Docerfile includes steps for copying and running the script.
Script renamed to make it clear what dependencies are being installed
@meoflynn meoflynn added the enhancement New feature or request label Nov 20, 2024
RUN chown ${NB_UID} /home/${NB_USER}/install-sciml-deps.sh
RUN chmod +rx /home/${NB_USER}/install-sciml-deps.sh

USER $NB_UID

Choose a reason for hiding this comment

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

Why is this line $NB_UID while other lines use ${NB_UID}? Will that make a difference?

Copy link
Contributor

Choose a reason for hiding this comment

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

While technically you can do that - for simplicity you don't need to - makes it easier to read

Copy link
Contributor Author

@meoflynn meoflynn Nov 20, 2024

Choose a reason for hiding this comment

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

The curly braces protect the variable, it's something that is optional to put around bash variables

The original directory was being hidden on JupyterHub by Longhorn,
moving it to a different location to download the model
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants