Skip to content

[Feat] 입력 제한 추가#1

Open
minseo9451 wants to merge 3 commits intomainfrom
minseo
Open

[Feat] 입력 제한 추가#1
minseo9451 wants to merge 3 commits intomainfrom
minseo

Conversation

@minseo9451
Copy link
Contributor

# 5개 입력
# 5개 미만인 경우
# 5개 - 입력 받는 갯수를 계산해서 가장 높은 tf-idf값 추출 단어 갯수 찾고
# 단어를 추출
# 모인 5개 단어를 번역
# 번역한 단어를 karlo에 전송
# 8개 이미지 받아서
# 클라에 전송

    # 5개 입력
    # 5개 미만인 경우
    # 5개 - 입력 받는 갯수를 계산해서 가장 높은 tf-idf값 추출 단어 갯수 찾고
    # 단어를 추출
    # 모인 5개 단어를 번역
    # 번역한 단어를 karlo에 전송
    # 8개 이미지 받아서
    # 클라에 전송
Copy link
Member

@namsh1125 namsh1125 left a comment

Choose a reason for hiding this comment

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

  1. URL: generate_image를 generate-image로 수정해주세요.
  2. 기존 return type으로 변경해주세요.
  3. 중간중간 요청 결과에 따른 로그 작성해주세요.
  4. 코드 보니 이미지 한 개만 return 하는 것 같아요. 저희는 생성된 이미지 전부 보내줘야 하므로 생성된 이미지를 전부 return 하도록 수정해주세요.

이외엔 특별한 것 없는 것 같아요. 고생하셨습니다.

1. URL: generate_image를 generate-image로 수정.
2. 기존 return type으로 변경
3. 요청 결과에 따른 로그 작성
4. 생성된 이미지를 전부 return
@minseo9451
Copy link
Contributor Author

minseo9451 commented Oct 19, 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.

2 participants