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

Reflecting the updated models released on April 9, 2024 #18

Merged
merged 17 commits into from
Apr 16, 2024

Conversation

toshiakit
Copy link
Collaborator

The list of supported models updates to reflect the changes in released on April 9, 2024.

  • Added new models
  • Removed preview models superseded by the new models
  • Removed models that were not publicly released (i.e., gpt-4-32k)
  • Updated the README and Examples to reflect the updated models
  • Updated the test to reflect the changes
  • modified how Stop Sequences and Max Num Tokens are passed to the API in order to deal with the issues with GPT-4 Vision.

openAIChat.m Show resolved Hide resolved
Copy link
Member

@ccreutzi ccreutzi left a comment

Choose a reason for hiding this comment

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

I'm fine with the code changes and just assume that you (and maybe one of the other reviewers) know enough about the currently offered models to assess which model/parameter combinations make sense to support.

Too bad GitHub doesn't show a diff of the mlx file.

Copy link
Member

@vpapanasta vpapanasta left a comment

Choose a reason for hiding this comment

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

Thanks for these changes Toshi. Please see my comments and feel free to let me know if you have any questions.

@toshiakit toshiakit merged commit d3e7389 into main Apr 16, 2024
1 check passed
@toshiakit toshiakit deleted the dev-update-040924models branch April 16, 2024 10:35
Copy link
Collaborator

@MiriamScharnke MiriamScharnke left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you for renaming all the example files!

@@ -49,8 +49,10 @@
catalog("llms:mustBeMessagesOrTxt") = "Messages must be text with one or more characters or an openAIMessages objects.";
Copy link
Collaborator

Choose a reason for hiding this comment

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

Messages must be nonempty char, string, or openAIMessages object.

image_path = "peppers.png";
messages = openAIMessages;
messages = addUserMessageWithImages(messages,"What is in the image?",image_path);
[txt,response] = generate(chat,messages);
[txt,response] = generate(chat,messages,MaxNumTokens=4096);
% Should output the description of the image
Copy link
Collaborator

Choose a reason for hiding this comment

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

Returns image description

Copy link
Collaborator

Choose a reason for hiding this comment

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

"in" should be upper case, i.e. AnalyzeSentimentInTextUsingChatGPTInJSONMode.mlx

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