kookmin-sw / capstone-2023-05

capstone-2023-05 created by GitHub Classroom
https://capstone-2023-05.vercel.app
0 stars 0 forks source link

Feat: Opinion filtering #79

Closed Binsk-dev closed 1 year ago

Binsk-dev commented 1 year ago

본 PR은 의견 필터링 기능의 추가를 위해 작성되었습니다. 본 필터링의 대상은 다음과 같습니다.

사전 준비

테스트 방법

기존에 했던 것처럼 sendOpinion 액션을 이용한 의견 보내기 테스트를 진행합니다. 테스트를 하면서 부적절한 의견인데 DB에 반영되는 상황이 있는지도 확인해주시면 좋을 것 같습니다.

API 스펙

주의사항(필독)

  1. 노션에 적힌 API key가 유효하지 않을 수 있습니다. 개발과 테스트의 경우 개인 api key를 통해 진행했습니다. 이와 관련하여 문제가 생기면 담당자가 조치해주세요.
  2. sendOpinion의 액션의 경우 round 관련해서 여전히 클라이언트에게 정보를 제공받는 코드로 되어있었습니다. 해당 부분은 제가 임의로 고쳐서 현재 라운드 정보를 백엔드에서 기입하는 걸로 바꿨습니다.
  3. Internal server error가 발생할 수도 있습니다. 그렇다면 serverless.yml 파일을 수정해 send-handler의 timeout 값을 2초 정도 늘려주시면 될 것 같습니다.
vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
capstone-2023-05 ✅ Ready (Inspect) Visit Preview May 24, 2023 11:35am
Jaewook-Lee commented 1 year ago

테스트를 하려 하는데 아래의 에러를 얻었어요.

openai.error.RateLimitError: You exceeded your current quota, please check your plan and billing details.

...돈 내야 하는건가요?

seungholee-dev commented 1 year ago

@Jaewook-Lee @Binsk-dev 아마도 용준 openAI 계정에서 Daily Quota 설정햇나봐용. 좀만 늘려줘요 용준님 ㅠ

Binsk-dev commented 1 year ago

위에서 언급했던 주의사항이 여러분이 말씀하시는 문제를 이야기 한겁니다. 현재 노션에 있는 API 키는 제가 관리하는 것이 아닙니다. 노션에 있는 키는 저희 팀이 관리하는 키이기 때문에 @Jaewook-Lee @ChangHoon-Sung 두 분 중에 계정 관리하시는 분이 openai 계정 설정을 확인하셔야 합니다.

Binsk-dev commented 1 year ago

테스트를 하려 하는데 아래의 에러를 얻었어요.

openai.error.RateLimitError: You exceeded your current quota, please check your plan and billing details.

...돈 내야 하는건가요?

기본적으로 api 호출을 통한 chatgpt 서비스는 기본이 유료입니다.

seungholee-dev commented 1 year ago

오 신기하네요 정상작동 확인했습니다.

Screenshot from 2023-05-25 15-13-14

시간관계 상 Merge 할게용

seungholee-dev commented 1 year ago

고생하셨어요