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.

Sign up for GitHub

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

Jump to bottom

Update JetStream instructions #132

Merged
merged 5 commits into from
Aug 23, 2024
Merged

Update JetStream instructions #132

merged 5 commits into from
Aug 23, 2024

Conversation

yeandy
Copy link
Contributor

@yeandy yeandy commented Aug 19, 2024

  • General cleanup of instructions
  • Generalizing the script to convert llama checkpoints s.t. it supports custom GCS buckets for the maxtext checkpoints.
  • Adding quantization instructions

@yeandy yeandy requested a review from vipannalla as a code owner August 19, 2024 19:45
FanhaiLu1
FanhaiLu1 approved these changes Aug 20, 2024
View reviewed changes
Copy link
Contributor

@FanhaiLu1 FanhaiLu1 left a comment

Choose a reason for hiding this comment

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

Thanks for the changes!

@yeandy yeandy merged commit 647ab24 into main Aug 23, 2024
3 checks passed
@yeandy yeandy deleted the yeandy_update_instructions branch August 23, 2024 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FanhaiLu1 FanhaiLu1 FanhaiLu1 approved these changes

@vipannalla vipannalla Awaiting requested review from vipannalla vipannalla is a code owner

@morgandu morgandu Awaiting requested review from morgandu

@JoeZijunZhou JoeZijunZhou Awaiting requested review from JoeZijunZhou

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yeandy @FanhaiLu1