Skip to content

fix: Updates to change the Azure OpenAI model from gpt-35-turbo to gpt-4o #1665

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

Merged
merged 4 commits into from
Feb 13, 2025

Conversation

Roopan-Microsoft
Copy link
Contributor

Purpose

  • This pull request includes updates to change the Azure OpenAI model from gpt-35-turbo to gpt-4o across various configuration files, documentation, and test cases. The changes ensure that the new model is consistently referenced throughout the codebase and associated resources.

Configuration and Environment Updates:

  • .env.sample: Updated AZURE_OPENAI_MODEL and AZURE_OPENAI_MODEL_NAME to gpt-4o.
  • infra/main.bicep: Changed default values for azureOpenAIModel, azureOpenAIModelName, and azureOpenAIModelVersion to gpt-4o and 2024-05-13 respectively.
  • infra/main.json: Updated default values for azureOpenAIModel, azureOpenAIModelName, and azureOpenAIModelVersion to gpt-4o and 2024-05-13 respectively. [1] [2]

Documentation Updates:

  • README.md: Updated the default configuration to deploy gpt-4o with version 2024-05-13.
  • docs/LOCAL_DEPLOYMENT.md, docs/TEAMS_LOCAL_DEPLOYMENT.md, docs/model_configuration.md: Updated references to gpt-4o and the corresponding version 2024-05-13. [1] [2] [3]

Code Updates:

Test Updates:

  • code/tests/functional/tests/backend_api/default/test_conversation.py, code/tests/functional/tests/backend_api/integrated_vectorization_custom_conversation/test_iv_question_answer_tool.py: Updated mock responses to use gpt-4o. [1] [2]

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

Copy link

github-actions bot commented Feb 12, 2025

Coverage

Coverage Report •
FileStmtsMissCoverMissing
code/backend/batch/utilities/helpers
   env_helper.py2001194%104, 107–108, 160, 368–370, 392, 397–399
TOTAL381985777% 

Tests Skipped Failures Errors Time
395 0 💤 0 ❌ 0 🔥 53.645s ⏱️

@Roopan-Microsoft Roopan-Microsoft changed the base branch from main to dev February 13, 2025 05:37
@Roopan-Microsoft Roopan-Microsoft merged commit 8fca481 into dev Feb 13, 2025
6 checks passed
@Roopan-Microsoft Roopan-Microsoft deleted the psl-gpt-model-update branch April 3, 2025 10:12
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.

2 participants