Skip to content

Ianmacleod/add mistral#307

Merged
ian-scale merged 6 commits intomainfrom
ianmacleod/add_mistral
Oct 4, 2023
Merged

Ianmacleod/add mistral#307
ian-scale merged 6 commits intomainfrom
ianmacleod/add_mistral

Conversation

@ian-scale
Copy link
Copy Markdown
Contributor

@ian-scale ian-scale commented Oct 3, 2023

adding mistral endpoint support for completions

"falcon-7b-instruct": "tiiuae/falcon-7b-instruct",
"falcon-40b": "tiiuae/falcon-40b",
"falcon-40b-instruct": "tiiuae/falcon-40b-instruct",
"mistral-7b": "mistralai/Mistral-7B-v0.1",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we need to add additional validation/tests? Also, think we need to update the EGP APIs?

Also we may want to update our Model Zoo docs to have some more granularity, to reflect the fact that not all models are supported for all frameworks.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oh yes, the EGP APIs will also need to change. And I will add the model-zoo comment and add a supported frameworks column to the docs.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

as far as testing, we can add synthetic monitors for completions for the new models. I will take out a new pr for this.

"falcon-7b-instruct": "tiiuae/falcon-7b-instruct",
"falcon-40b": "tiiuae/falcon-40b",
"falcon-40b-instruct": "tiiuae/falcon-40b-instruct",
"mistral-7b": "mistralai/Mistral-7B-v0.1",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

update L494 about context window?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added this in, also added some logic to make the output dir "mistral_files" and added the max_model_len param

@ian-scale ian-scale merged commit acdd1ea into main Oct 4, 2023
@ian-scale ian-scale deleted the ianmacleod/add_mistral branch October 4, 2023 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants