Skip to content

changing default model to instructlab model #921

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
Merged
merged 1 commit into from
Apr 19, 2024
Merged

changing default model to instructlab model #921

merged 1 commit into from
Apr 19, 2024

Conversation

mairin
Copy link
Member

@mairin mairin commented Apr 19, 2024

Using instructlab/merlinite-7b-lab

I am making this change so we are using an InstructLab community model by default with the instructlab cli.

@github-actions github-actions bot added documentation Improvements or additions to documentation testing Relates to testing labels Apr 19, 2024
@mairin
Copy link
Member Author

mairin commented Apr 19, 2024

i still need to test this.

xukai92
xukai92 reviewed Apr 19, 2024
View reviewed changes
@mairin
changing default model to instructlab model
6916cce 
Using instructlab/merlinite-7b-lab

Signed-off-by: Máirín Duffy <duffy@redhat.com>
@mairin mairin force-pushed the merlinite-update branch from f38deae to 6916cce Compare April 19, 2024 04:30
@mairin
Copy link
Member Author

mairin commented Apr 19, 2024

Tested with ilab download and it's working fine

xukai92
xukai92 approved these changes Apr 19, 2024
View reviewed changes
Copy link
Member

@xukai92 xukai92 left a comment

Choose a reason for hiding this comment

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

also test ilab train, ilab test and ilab convert followed by ilab serve of the new model. all good.

@xukai92 xukai92 merged commit 6fcceed into main Apr 19, 2024
@xukai92 xukai92 deleted the merlinite-update branch April 19, 2024 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xukai92 xukai92 xukai92 approved these changes

@jeremyeder jeremyeder Awaiting requested review from jeremyeder

@abhi1092 abhi1092 Awaiting requested review from abhi1092

@markstur markstur Awaiting requested review from markstur

@hickeyma hickeyma Awaiting requested review from hickeyma

@spzala spzala Awaiting requested review from spzala

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation testing Relates to testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mairin @xukai92