Skip to content

model_ckpt_conversion: use python module, enable gcloud checkpoint downloading#258

Closed
karan wants to merge 1 commit into
AI-Hypercomputer:mainfrom
karan:model_ckpt_conversion
Closed

model_ckpt_conversion: use python module, enable gcloud checkpoint downloading#258
karan wants to merge 1 commit into
AI-Hypercomputer:mainfrom
karan:model_ckpt_conversion

Conversation

@karan
Copy link
Copy Markdown
Collaborator

@karan karan commented Apr 24, 2025

  1. cb86ae2 seems to have missed llama_or_mistral_ckpt.

  2. GCS copying was commented out in 082c0ac. I'm guessing this was unintended.

@karan karan requested a review from vipannalla as a code owner April 24, 2025 21:17
@vipannalla vipannalla requested a review from Lumosis April 28, 2025 16:33
@karan
Copy link
Copy Markdown
Collaborator Author

karan commented Apr 28, 2025

I'm guessing merging from forks doesn't work in this repo either. Could someone help get this one, and in the future I'll send PRs from branches.

@vipannalla vipannalla added the pull ready This label is needed if we want the copybara service to auto sync it to g3. label Apr 28, 2025
@jyj0w0
Copy link
Copy Markdown
Collaborator

jyj0w0 commented Apr 28, 2025

Yes, forking is not working for this repo either, and we might not merge manually -- could you restart with branch instead? Thank you!

@karan
Copy link
Copy Markdown
Collaborator Author

karan commented Apr 28, 2025

Ack will do!

@karan
Copy link
Copy Markdown
Collaborator Author

karan commented Apr 28, 2025

@jyj0w0 How can I get access to push to the repo?

@jyj0w0
Copy link
Copy Markdown
Collaborator

jyj0w0 commented Apr 28, 2025

@jyj0w0 How can I get access to push to the repo?

I just invite you to be an writer to this repo

@karan
Copy link
Copy Markdown
Collaborator Author

karan commented Apr 28, 2025

Closing in favor of #261

@karan karan closed this Apr 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pull ready This label is needed if we want the copybara service to auto sync it to g3.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants