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

Fix Google Colab notebook 2024-05 #1662

Merged
merged 2 commits into from
May 28, 2024

Conversation

maciejgryka
Copy link

@maciejgryka maciejgryka commented May 28, 2024

A couple of fixes to make the Google Colab notebook run (tried on an L4 GPU):

Description

The accelerate launch command fails without installing mlflow, so I included !pip install mlflow=="2.13.0" with the other deps at the top.

The current config YAML included in the notebook results in _amp_foreach_non_finite_check_and_unscale_cuda" not implemented for 'BFloat16'. This doesn't happen when we instead use the current QLORA config for tinyllama. Possibly just updating to bf16: auto would be enough? But I didn't test that and using the entire new config seems better.

Motivation and Context

I tried to get started in Colab and couldn't because of these issues.

How has this been tested?

Ran in Colab using an L4 GPU.

Types of changes

Social Handles (Optional)

maciejgryka and others added 2 commits May 28, 2024 11:46
Without explicitly installing mlflow the `accelerate launch` command fails.
@maciejgryka maciejgryka changed the title Fix colab 2024 05 Fix Google Colab notebook 2024-05 May 28, 2024
@maciejgryka maciejgryka marked this pull request as ready for review May 28, 2024 10:50
@winglian winglian merged commit 5f91064 into axolotl-ai-cloud:main May 28, 2024
@maciejgryka maciejgryka deleted the fix-colab-2024-05 branch May 29, 2024 07:20
djsaunde pushed a commit that referenced this pull request Dec 17, 2024
* include mlflow installation in the colab notebook

Without explicitly installing mlflow the `accelerate launch` command fails.

* update the colab noteboko to use the latest tinyllama config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants