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

Updates for Llama 3.1 compatibility, import error workarounds & more #75

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AgenP
Copy link

@AgenP AgenP commented Aug 6, 2024

Train.py

  • Axolotl base image SHA updated, and pinned, to a Llama 3.1 compatible version
  • Allow background commits flag removed since deprecated
  • Regression test: No errors when running this file using llama-3.yml

Inference.py

Common.py

Have been unable to get past a ModuleNotFoundError importing common.py

So I commented out the import and directly pasted the relevant code into both train.py & inference.py

Apologies if I’ve missed something obvious for the import as this doesn’t seem like the most elegant fix

But submitting it, cause as recently been said, fear is the mind-killer

(On a real note, more than happy to iterate if I’ve missed something here)

Requirements.txt (local usage)

  • Modal
  • PyYaml (used in inference.py)
  • Pandas (Commented out, but used for CI in check_loss.py)

Llama 3.1 yaml file added

CI files

  • check_loss.py ran fine
  • check_inference unable to get past ModuleNotFoundError (However inference.py worked w/ Llama 3.1, Llama 3 & Open-Llama-7b-v2)

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.

1 participant