ai-pr-reviewer icon indicating copy to clipboard operation
ai-pr-reviewer copied to clipboard

Added support for gpt-4 128k

Open jimmystridh opened this issue 2 years ago β€’ 6 comments

With yesterday's release GPT-4 128k is available in the API. Updated limits (but left default) with the new values from https://help.openai.com/en/articles/8555510-gpt-4-turbo

Summary by CodeRabbit

  • New Feature: Added support for the new GPT-4 128k model in the TokenLimits class. This update allows users to utilize the increased token limit and response tokens of the GPT-4 model, enhancing the capabilities of their AI applications. Please note that the knowledgeCutOff value has also been updated for this model.

jimmystridh avatar Nov 07 '23 13:11 jimmystridh

Image description CodeRabbit

Walkthrough

This change introduces support for the new GPT-4 128k model in the TokenLimits class. It adds a case for the gpt-4-1106-preview model, setting appropriate token limits and updating the knowledge cut-off value. The default values remain unaffected.

Changes

File Summary
src/limits.ts Updated TokenLimits class to include support for the new GPT-4 128k model. Added a case for gpt-4-1106-preview, set maxTokens to 128000, responseTokens to 4000, and updated knowledgeCutOff. Default values remain unchanged.

πŸ‡πŸ’»

In the land of code, where the shadows lie,

A new model emerges, reaching for the sky.

With tokens aplenty, it's ready to fly,

GPT-4 is here, oh my, oh my! πŸš€πŸŽ‰


Uplevel your code reviews with CodeRabbit Pro

CodeRabbit Pro

If you like this project, please support us by purchasing the Pro version. The Pro version has advanced context, superior noise reduction and several proprietary improvements compared to the open source version. Moreover, CodeRabbit Pro is free for open source projects.

github-actions[bot] avatar Nov 07 '23 13:11 github-actions[bot]

@jimmystridh thanks for your PR. We tried the new model yesterday and found that it’s severely rate limited right now - 100 requests per day (100 RPD). We can merge this PR as soon as the limit is increased by OpenAI. Cheers!

harjotgill avatar Nov 07 '23 13:11 harjotgill

https://www.threads.net/@firtoz/post/Cze1ZLmorg3/?igshid=NTc4MTIwNjQ2YQ== seems like it's happening

jimmystridh avatar Nov 11 '23 02:11 jimmystridh

Limits raised to 10,000 RPD. I think it's time to merge.

@harjotgill

kuzaxak avatar Nov 17 '23 22:11 kuzaxak

@harjotgill When will this be merged?

mopemope avatar Nov 20 '23 21:11 mopemope

Let's make this happen together!

a6a18 avatar Jan 30 '24 09:01 a6a18