Skip to content

Using AutoConfig to load model config file.#416

Merged
JamesKunstle merged 3 commits into
instructlab:mainfrom
abhi1092:main
Feb 4, 2025
Merged

Using AutoConfig to load model config file.#416
JamesKunstle merged 3 commits into
instructlab:mainfrom
abhi1092:main

Conversation

@abhi1092

Copy link
Copy Markdown
Member
  • Right now loading from local file causes the code to fail on HF checkpoints.
  • This should fix that issue

Signed-off-by: Abhishek Bhandwaldar <abhi1092@gmail.com>
Signed-off-by: Abhishek Bhandwaldar <abhi1092@gmail.com>
@mergify mergify Bot added ci-failure and removed ci-failure labels Jan 27, 2025
Signed-off-by: Abhishek Bhandwaldar <abhi1092@gmail.com>
@mergify mergify Bot added ci-failure and removed ci-failure labels Jan 28, 2025

@Maxusmusti Maxusmusti left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@mergify mergify Bot added the one-approval label Jan 29, 2025
@abhi1092 abhi1092 requested a review from RobotSail January 30, 2025 21:08
@mergify mergify Bot removed the one-approval label Jan 30, 2025
@JamesKunstle JamesKunstle merged commit 89c5bd5 into instructlab:main Feb 4, 2025
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.

4 participants