Skip to content
This repository was archived by the owner on Mar 12, 2024. It is now read-only.

Feature/Mixtral test#414

Closed
EstebanDalelR wants to merge 2 commits intodevfrom
feature/togetherAI-test
Closed

Feature/Mixtral test#414
EstebanDalelR wants to merge 2 commits intodevfrom
feature/togetherAI-test

Conversation

@EstebanDalelR
Copy link
Member

Description

Proposal to use Mixtral on Together AI

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Chore: cleanup/renaming, etc
  • RFC
  • Test

Notes

Acceptance

@vercel
Copy link

vercel bot commented Dec 14, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
watermelon ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 14, 2023 1:56am

@watermelon-copilot-for-code-review

Watermelon AI Summary

This pull request introduces a new feature that replaces OpenAI with Mixtral in the Together AI system, adding the necessary integration files and tests for the newly proposed functionality.

GitHub PRs

Click here to login to Jira
Click here to login to Confluence
No results found in Slack Threads :(

Notion Pages

No results found in Linear Tickets :(

Click here to login to Asana
watermelon is an open repo and Watermelon will serve it for free.
🍉🫶

.then((response) => response.json())
.then((result) => result.choices[0].message.content)
.catch((error) => console.log("error", error));
} catch (error) {

Choose a reason for hiding this comment

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

This PR contains console logs. Please review or remove them.

Copy link
Collaborator

@baristaGeek baristaGeek left a comment

Choose a reason for hiding this comment

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

Code looks fine and works fine. I'll approve. We can go ahead and add Mixtral to the other endpoints.

However, I do want to approve with this in mind. This is the Mixtral result for #412:
mixtralResult

And this is the one for Open AI:
openAIResult

Open AI's result has higher quality. Is it just a matter of time that Mixtral's quality gets there?

@EstebanDalelR
Copy link
Member Author

Code looks fine and works fine. I'll approve. We can go ahead and add Mixtral to the other endpoints.

However, I do want to approve with this in mind. This is the Mixtral result for #412: mixtralResult

And this is the one for Open AI: openAIResult

Open AI's result has higher quality. Is it just a matter of time that Mixtral's quality gets there?

I think it's 2 things:

  • Mixtral is a GPT3.5 level model
  • and the one we are using is not chat optimized, so we need a better prompt.

On the other hand, this PR proves that it works! We can now use Together to deploy pretty much any OSS model.

My proposal is to make GPT the default and add a random text suggesting you try others?

@EstebanDalelR
Copy link
Member Author

Not yet ready for production.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants