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

Fix: Chat-300-BE-이미지-처리-수정 #54

Merged
merged 5 commits into from
Feb 16, 2024

Conversation

Kjiw0n
Copy link
Collaborator

@Kjiw0n Kjiw0n commented Feb 15, 2024

요약 (Summary)

  • 최근 채팅 조회 로직의 정렬 순서를 사용자의 최신 채팅을 우선적으로 가져오도록 수정

변경 사항 (Changes)

  • 사용자의 가장 최근 채팅부터 가져오도록 채팅 조회 쿼리의 정렬 순서를 오름차순(Asc)에서 내림차순(Desc)으로 변경했습니다.
    • findTop10ByMember_UserIdOrderByChatIdAsc 메소드를 findTop10ByMember_UserIdOrderByChatIdDesc 로 수정하여 구현.
  • 채팅 처리 중 에러 발생 시 보다 더 원활한 디버깅을 위해 로그를 추가하였습니다.
    • 가져오는 최근 채팅의 chatId를 gpt 모델에 전달되는 순으로 반환
    • IMG 처리에서 “Invalid Image” 라는 오류가 자주 발생하므로, 해당하는 채팅의 chatId 반환

리뷰 요구사항

  • 채팅 상호작용 중 문제없이 수행되는 지 확인해주세요.
  • 최신 채팅부터 정확히 조회되는지 검토해주세요.
  • 로그를 통해 채팅 처리 로직이 기능 요구사항에 부합하는지 재확인 부탁드립니다.

확인 방법 (선택)

요청 경로

ws://localhost:8080/chatwebsocket

요청 예시

{
    "userId": 1,
    "content": "안녕",
    "selectedModel": 1,
    "chatType" : "CHAT"
}

포스트맨 응답 예시

image

Copy link
Collaborator

@dainshon dainshon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

정렬이 내림차순이라는게 recentChats[-1] = 가장 최근 채팅 이 되는거죠?!
맞다면 잘 동작합니다!!:>

코드 실행할때 기존 채팅 중 이미지가 있으면

2024-02-16T00:16:31.791+09:00 ERROR 47387 --- [onPool-worker-1] c.kuit.chatdiary.service.OpenAIService   : Error calling OpenAI API: 

org.springframework.web.client.HttpClientErrorException$BadRequest: 400 Bad Request: "{<EOL>  "error": {<EOL>    "message": "Invalid image.",<EOL>    "type": "invalid_request_error",<EOL>    "param": null,<EOL>    "code": null<EOL>  }<EOL>}"

이런 오류가 뜨는게 이 부분은 이 브랜치에서의 작업이 아니라 무시해도 되는건지 궁금합니다!

@Kjiw0n
Copy link
Collaborator Author

Kjiw0n commented Feb 15, 2024

정렬이 내림차순이라는게 recentChats[-1] = 가장 최근 채팅 이 되는거죠?! 맞다면 잘 동작합니다!!:>

코드 실행할때 기존 채팅 중 이미지가 있으면

2024-02-16T00:16:31.791+09:00 ERROR 47387 --- [onPool-worker-1] c.kuit.chatdiary.service.OpenAIService   : Error calling OpenAI API: 

org.springframework.web.client.HttpClientErrorException$BadRequest: 400 Bad Request: "{<EOL>  "error": {<EOL>    "message": "Invalid image.",<EOL>    "type": "invalid_request_error",<EOL>    "param": null,<EOL>    "code": null<EOL>  }<EOL>}"

이런 오류가 뜨는게 이 부분은 이 브랜치에서의 작업이 아니라 무시해도 되는건지 궁금합니다!

넵 맞습니다.
에러 로그의 경우 해당 부분을 수정한 것이기 때문에 다른 브랜치는 아직 적용되지 않았을거에요!

dainshon and others added 3 commits February 16, 2024 21:49
recentchats 에 이미지는 무시.
단, 가장 마지막 채팅이 이미지인경우(지금 보낸게 이미지인경우)는 읽음
Copy link
Collaborator

@dainshon dainshon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨어욥:)

@dainshon dainshon merged commit f54d717 into develop Feb 16, 2024
1 check failed
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