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

Remove legacy GPT-3.5 Turbo 16k via OpenAI API #839

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

PeterDaveHello
Copy link
Contributor

Groq removed it and replaced with Llama 3 since 25 April.

Reference:

Groq removed it and replaced with Llama 3 since 25 April.

Reference:
- https://twitter.com/GroqInc/status/1783235427916251464
Copy link

vercel bot commented May 14, 2024

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

Name Status Preview Comments Updated (UTC)
chatall ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 14, 2024 4:28pm

@sunner
Copy link
Owner

sunner commented May 15, 2024

Directly remove code will break app since legacy data can't be parsed right.

Could you just kindly add it to the disabled list? Someday later, we can remove it in a whole.

Disabled list:

ChatALL/src/bots/index.js

Lines 171 to 181 in 9b73005

const disabled = [
"ClaudeBot",
"AlpacaBot",
"HuggingChatBot",
"Falcon180bBot",
"ChatGLM6bBot",
"ChatGLM36bBot",
"CodeLlamaBot",
"Vicuna7bBot",
"Wizardlm13bBot",
];

@PeterDaveHello
Copy link
Contributor Author

Maybe the solution is to fix the parse method?

@sunner
Copy link
Owner

sunner commented May 23, 2024

Maybe the solution is to fix the parse method?

Do you mean fix the parse method in the chat view? If so, there are two solutions:

  1. Filter out the messages from removed bots
  2. Depends other code to show the messages correctly

Solution 1 looks like data loss. Solution 2 will lead to duplicate code. IMO, disable the removed bot in the bots menu is the best solution.

What do you think?

@PeterDaveHello
Copy link
Contributor Author

I prefer not to keep legacy and deprecated codes or naming in the project. It'll make the maintenance complexity much higher and also make it harder for new contributors to join. That's also why I want #780.

@sunner
Copy link
Owner

sunner commented Jun 1, 2024

I understand your concern. How about moving the legacy codes to a legacy folder and totally remove them several months or years later?

@PeterDaveHello
Copy link
Contributor Author

Yeah, I also got your point, the legacy folder could be a workaround, though I doubt how many users will keep all those old bots conversations, considering the AI evolutions are currently so fast, most users will use the newer models, and this currently broken bots didn't cause obvious bug reports, maybe we can just remove it after a month? Anyway, I can close this PR if the workaround is something we both think is possible.

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.

None yet

2 participants