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

Feat: transformQuestion 그림 변환 구현 #147

Merged
merged 2 commits into from
Aug 6, 2024
Merged

Conversation

RinRinPARK
Copy link
Contributor

@RinRinPARK RinRinPARK commented Aug 3, 2024

PR

✨ 작업 내용

  • transformQuestion의 그림변환 부분을 추가했습니다.
  • AWS의 Lambda와 API Gateway를 이용해서 따로 파이썬 코드 서버(?, 서버리스인디..)를 구축했습니다. 혼자서 나름 챌린지했습니다;;;
  • 왜 lambda를 이용했는지.. 기타 자세한 내용은 위키에.. 위키 첨 써봣는데 어렵네요
  • 프로퍼티파일 노션에 업로드 해놨습니다. 편리한 테스트를 위해! 테스트케이스 두가지도 노션 api문서 내에 첨부했습니당 참조해주세여

✨ 참고 사항

  • 이미지 좌표를 시계방향(좌상, 우상, 우하, 좌하 순서로) 4개의 점으로 주어야 합니다. 그렇지 않으면 이상하게 잘릴 수 있습니다.
    저도 아무 순서로 찍어도 처리되도록 하고 싶었는데, 뭐 그럴 가능성은 적지만 예를 들어 직사각형이 아닌 마름모 형태로 자르고 싶어한다던가 그러면
    정확히 어디가 위고 아래이며 이걸 어떻게 다시 직사각형 변환할지 처리가 애매해서 무조건 시계방향 순서로 배열이 와야 헙니다. 그래야 첫번째, 두번째 점을 위로보고 세번째, 네번째 점을 아래로 두어 직사각형 형태의 그림으로 반환할 수 있으니까요!(뭔말인지 이해가 어려우려나..)
    클라에서 유저에게 가이드할 때 해당 사항을 화면에 안내해주어야 할 듯 합니다.

  • 현재 그림변환 이후 gpt에 넘겨서 문제변환을 해서 이부분을 동시에 처리할 수 있도록 비동기처리 하면 될 듯 합니다!!! 챌린징하자 챌린징


⏰ 현재 버그


✏ Git Close #134

@RinRinPARK RinRinPARK added ✨ Feature 기능 개발 ☘️ surin surin labels Aug 3, 2024
@RinRinPARK RinRinPARK requested a review from adorableco August 3, 2024 13:40
@RinRinPARK RinRinPARK self-assigned this Aug 3, 2024
Copy link

github-actions bot commented Aug 3, 2024

Unit Test Results

16 tests   16 ✔️  2s ⏱️
  6 suites    0 💤
  6 files      0

Results for commit 9bf55fa.

♻️ This comment has been updated with latest results.

@adorableco
Copy link
Contributor

adorableco commented Aug 3, 2024

오옹.. Lambda를 아주 적재적소에 사용한 것 같으네요 위키 설명도 상세해서 잘 이해했숩니다 어렵긴 하네요..
수고하셨읍니다!!

슬랙에 보냈던 500 error 원인을 찾고나서 테스트 후 approve 남기겠습니다!

@adorableco
Copy link
Contributor

이건 런타임 환경을 파이썬이 아닌 다른 언어로 설정해도 정상작동한다는 뜻인가요??

스크린샷 2024-08-03 오후 11 19 28

): BaseResponse<TransformQuestionResponse> {
val imageUrl = image?.let { s3Service.uploadChatGptImage(it) }
val response: TransformQuestionResponse = questionService.transformQuestion(imageUrl!!)
val response: TransformQuestionResponse = questionService.transformQuestion(imageUrl!!, imageCoordinates)
Copy link
Contributor

Choose a reason for hiding this comment

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

엇 이 부분은 제가 조만간 image 변수에서 nullable 제거하겠습니다.. .^^

@adorableco
Copy link
Contributor

adorableco commented Aug 3, 2024

Lambda 함수에서 63번 라인의 버킷 네임은 뒤에 "/question" 을 붙이는걸로 바꿔서, 문제 변환에 사용되는 사진과 마찬가지로 주기적으로 삭제되도록 하는게 좋을 것 같습니다!

(스크린샷 첨부했다가 버킷 네임은 프로퍼티로 관리하니 보안 상의 이유로 삭제합니당)

@RinRinPARK
Copy link
Contributor Author

오옹.. Lambda를 아주 적재적소에 사용한 것 같으네요 위키 설명도 상세해서 잘 이해했숩니다 어렵긴 하네요.. 수고하셨읍니다!!

슬랙에 보냈던 500 error 원인을 찾고나서 테스트 후 approve 남기겠습니다!

간헐적으로 람다함수에서 500이 뜨길래 람다함수 내에서 테스트 해보니까 첨에 s3연결할 때 timeout이 나는 것 같드라구요!!
제일 처음에 연결할 때 timeout이 나고 그 뒤로 연결된 뒤에는 쑥쑥 3초내로 잘 작동하길래(기존 timeout시간 제한이 3초입니다)
timeout시간을 10초로 늘렸습니다!! 아마 해결됐을겁니다 확인부탁드리와용

@RinRinPARK
Copy link
Contributor Author

RinRinPARK commented Aug 3, 2024

이건 런타임 환경을 파이썬이 아닌 다른 언어로 설정해도 정상작동한다는 뜻인가요??

스크린샷 2024-08-03 오후 11 19 28

런타임환경을 다른 언어로 설정하고 코드도 다른 언어로 작성하면 됩니다

@RinRinPARK
Copy link
Contributor Author

Lambda 함수에서 63번 라인의 버킷 네임은 뒤에 "/question" 을 붙이는걸로 바꿔서, 문제 변환에 사용되는 사진과 마찬가지로 주기적으로 삭제되도록 하는게 좋을 것 같습니다!

(스크린샷 첨부했다가 버킷 네임은 프로퍼티로 관리하니 보안 상의 이유로 삭제합니당)

이 사진은 실제로 저장되어서 문제 삭제가 되기 전까지 계속계속 사용되어야하는 사진이라 주기적으로 삭제되면 안 될 것 같습니다!!!

@adorableco
Copy link
Contributor

Lambda 함수에서 63번 라인의 버킷 네임은 뒤에 "/question" 을 붙이는걸로 바꿔서, 문제 변환에 사용되는 사진과 마찬가지로 주기적으로 삭제되도록 하는게 좋을 것 같습니다!
(스크린샷 첨부했다가 버킷 네임은 프로퍼티로 관리하니 보안 상의 이유로 삭제합니당)

이 사진은 실제로 저장되어서 문제 삭제가 되기 전까지 계속계속 사용되어야하는 사진이라 주기적으로 삭제되면 안 될 것 같습니다!!!

그러면 createQuestion 요청 시에 이미지 파일 대신 s3 이미지 주소를 String으로 받는게 좋을 것 같네요!!

@adorableco
Copy link
Contributor

이건 런타임 환경을 파이썬이 아닌 다른 언어로 설정해도 정상작동한다는 뜻인가요??
스크린샷 2024-08-03 오후 11 19 28

런타임환경을 다른 언어로 설정하고 코드도 다른 언어로 작성하면 됩니다

오호 그렇군요!!

@adorableco
Copy link
Contributor

확인 완료입니다오~

@adorableco adorableco self-requested a review August 5, 2024 06:50
@RinRinPARK RinRinPARK merged commit ad9e881 into develop Aug 6, 2024
2 checks passed
@RinRinPARK RinRinPARK deleted the rin/swm-124 branch August 6, 2024 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 ☘️ surin surin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ Feat: transformQuestion 그림 변환 구현
2 participants