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

config: exclude hkg1 from the deployment regions of the edge function #450

Closed
wants to merge 9 commits into from

Conversation

iRedScarf
Copy link
Contributor

Description

This commit updates the deployment configuration for our edge functions, specifically excluding the Hong Kong region (hkg1). The primary reason for this change is to address an issue where requests made to the OpenAI API were not being responded to when our application was deployed in the Hong Kong region. By excluding Hong Kong from our deployment regions, we aim to resolve this issue, ensuring stability and efficiency in application response. This adjustment is expected to enhance user experience and ensure smooth operation of our services globally.

Linked Issues

Additional context

Reviewers should focus on the updated list of deployment regions in the vercel.json file, ensuring that Hong Kong is correctly excluded. Consideration of the potential impact on performance and user experience in other regions due to this change is also important.

iRedScarf and others added 9 commits September 15, 2023 23:11
Set the default context to only send the latest 10 historical messages (the AI reply is also counted as 1) to avoid exceeding the tokens limit error after chatting for a period of time.
Change the maximum number of history messages by setting the environment variable `PUBLIC_MAX_HISTORY_MESSAGES` if need to.
It's not recommended to set it too low, such as 2, as it can easily lead to AI making random mistakes, even if it is gpt-4.
config: exclude hkg1 from the deployment regions of the edge function
Copy link

vercel bot commented Nov 17, 2023

@iRedScarf is attempting to deploy a commit to the ddiu8081's Team Team on Vercel.

A member of the Team first needs to authorize it.

@iRedScarf iRedScarf closed this Nov 20, 2023
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.

1 participant