-
Notifications
You must be signed in to change notification settings - Fork 18
[Playground] Refactoring OpenAIApiClient
#163
#314
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
[Playground] Refactoring OpenAIApiClient
#163
#314
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 ChatWndowComponent.razor
가 들어있는 Home.razor
파일을 로컬에서 테스트 돌리면 잘 돌아가는지만 확인해 보시지요. 통합테스트 부분은 나중에 따로 처리해야 할 듯.
src/AzureOpenAIProxy.PlaygroundApp/Components/UI/ChatWindowComponent.razor
Outdated
Show resolved
Hide resolved
@5jisoo 컨플릭 먼저 해결해 보시구요, 이제부터는 플레이그라운드 페이지에 직접 적용을 시켜볼까요? |
작업 내용
다음과 같은 에러가 발생하였고, 아마 Program.cs에서 설정해준 Endpoint 값(
저는 Aspire에서 ApiApp 프로젝트를 등록하면서 |
src/AzureOpenAIProxy.PlaygroundApp/Components/UI/ChatWindowComponent.razor
Outdated
Show resolved
Hide resolved
일단 수정 사항은 전부 고쳤습니다! |
정 안되면 HttpClient 인스턴스를 인젝션하고 거기서 BaseUri 값을 받아오는 식의 꼼수를 써야 할 수도 있을 듯 해요. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
대략의 의견을 남겨봤어요
.... 이렇게 바꾸어도 동일한 에러가 발생합니다 |
지수님 리포로 PR 하나 날렸어요: 5jisoo#1 한 번 확인해 보시고, 적용시켜 보세요. |
src/AzureOpenAIProxy.PlaygroundApp/Components/UI/OldChatWindowComponent.razor
Show resolved
Hide resolved
…ient Add app settings to resolve service discovery
지금 OpenAI SDK 가장 최신 버전이 2.1.0-beta.1 입니다. 그거 기준으로 가시지요. |
제가 main 브랜치 업데이트해서 컨플릭 날 거예요 @5jisoo 확인해 보시고, 반영시켜 주세요 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
일단 테스트 코드 없이 가 봅시다
작업중인 이슈
OpenAIApiClient
#163작업 내용
궁금한 점
https+http://apiapp
를 HttpClient의 BaseAddress로 지정해주었던 것과 동일한 방식으로 AzureOpenAIClient의 endpoint를 설정하였는데, 이게 올바른 방향이 맞는지 궁금합니다.