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

【5章】日本リージョン対応, davinci → gpt-35-turbo 対応 #74

Merged
merged 24 commits into from
Sep 21, 2023

Conversation

kazuki-komori
Copy link
Member

@kazuki-komori kazuki-komori commented Sep 13, 2023

Purpose

以下の変更を行いました。

  • Bicep, アプリケーションコード, README/図において、davinciからgpt-35-turboに移行
  • model version を 0301→0630 へ変更
  • openai.Completion → openai.ChatCompletionへの移行
  • "davinci"等古い変数名や値をアプリケーションコード内から削除
  • アプリケーションUI上で"davinci"等の古い名称ではなく正確な名称が表示されるように修正
  • 日本リージョンへのデプロイが可能になります

Does this introduce a breaking change?

[x] Yes
[ ] No

text-davinci のモデル変更に伴い、モデルをgpt-3.5-turboに切り替えています。
したがって、これ以降のバージョンではtext-davinciモデルはデプロイされなくなります。

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • 再度 azure 上にアプリケーションの環境を作り直して、文章検索およびチャットが動作するかご確認ください。
$ azd up

What to Check

  • davinci に付随するコードが残ってないか?
  • リソースのデプロイが正常に行われるか?
  • アプリケーションの動作 (チャットと文章検索) が正しく行われるか?
  • アプリケーション UI 上で davinci が存在していないか?

Other Information

@haruos
Copy link
Contributor

haruos commented Sep 13, 2023

本版では Cognitive Search のベクトル検索に未対応なことから、クライアント(react) から chatreadretrieveread.py の has_text には値が渡ってこないので処理を削除してOKです。

ここと
has_text = overrides.get("retrievalMode") in ["text", "hybrid", None]

ここですね。
if not has_text:
query_text = None

@haruos haruos marked this pull request as ready for review September 13, 2023 14:04
@kazuki-komori kazuki-komori changed the title 【5章】davinci → gpt-35-turbo 対応 【5章】日本リージョン対応, davinci → gpt-35-turbo 対応 Sep 20, 2023
@kazuki-komori kazuki-komori merged commit a337fb5 into Azure-Samples:main Sep 21, 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.

4 participants