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

Lit-GPT model integration #1783

Open
aniketmaurya opened this issue Aug 15, 2023 · 3 comments
Open

Lit-GPT model integration #1783

aniketmaurya opened this issue Aug 15, 2023 · 3 comments
Labels
competition Support for the NeurIPS Large Language Model Efficiency Challenge user question

Comments

@aniketmaurya
Copy link
Contributor

Hello HELM team, would you be interested in accepting a PR to integrate Lightning AI's Lit-GPT? If yes, I would love to contribute here.

@yifanmai
Copy link
Collaborator

Hi @aniketmaurya, this is something that we'd definitely like to have. HELM is also supporting the NeurIPS efficiency challenge, so it would be nice if users could run HELM on the Lit-GPT checkpoints directly.

Here's some related documentation:

Would you like to take a shot at an initial PR? Otherwise I can take a look sometime next week and draft a PR.

Also, if this would be faster with sync communication, feel free to email me and set up a time for a video call. I can be reached at my email address in the git commit log.

One blocker is the Lit-GPT has a lot of configuration parameters; I'm currently working on configuration files for models over the next couple of weeks, which would solve this, but in the meantime, we might have to read configuration from environment variables or something.

@yifanmai yifanmai added the competition Support for the NeurIPS Large Language Model Efficiency Challenge label Aug 16, 2023
@aniketmaurya
Copy link
Contributor Author

hi @yifanmai, thank you for your reply! I have a initial PR here.

Yeah, I realized that for configuration I need to depend on environment variable. Would be supercool to have a configuration file. I will drop you an email to setup a time for call.

@yifanmai
Copy link
Collaborator

yifanmai commented Aug 5, 2024

I think we should migrate to use the new LitGPT API. We could also consider using supporting LitServe.

@yifanmai yifanmai mentioned this issue Aug 5, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
competition Support for the NeurIPS Large Language Model Efficiency Challenge user question
Projects
None yet
Development

No branches or pull requests

2 participants