[FEATURE] gpt 결과물 & SSML 연결 작업 #23
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
수고하셨습니다 🤝 |
ja7811
approved these changes
Jul 28, 2024
Comment on lines
62
to
72
| byte[] audioBytes = Base64.getDecoder().decode(audioContent); | ||
| //S3 설정 후 변경 예정 | ||
| String rand= UUID.randomUUID().toString(); | ||
| String outputFilePath = "src/main/resources/speech/" + rand + ".mp3"; | ||
| try (OutputStream out = new FileOutputStream(outputFilePath)) { | ||
| out.write(audioBytes); | ||
| } catch (Exception e) { | ||
| e.printStackTrace(); | ||
| } | ||
| return "success"; | ||
| } |
There was a problem hiding this comment.
수고하셨습니다 잘 작동하네용 👍
requestSpeech() 함수 부분 읽으면서 든 생각 몇 가지 있는데 괜찮다 싶으면 적용해주세요
outputFilePath같은 건 함수 윗 부분이나 static 변수로 따로 빼는 게 어떨지"success"를 반환하기보단, 생성된 파일명 등을 return하는 게 어떨지 (이후 스트리밍 등에 사용할 테니까)- 파일 저장 중 예외 발생 시 콘솔 출력 후 끝나는데, 새로운 RuntimeException 등을 던지는 게 어떨지
- 지금은 에러만 출력한 후 똑같이
return "success";를 거쳐서 정상적인 흐름으로 넘어가는데,
파일 저장을 못하면 여기서 흐름을 끊어버리는 게 맞다고 생각해요 - 그래서 RuntimeException을 새로 던져서 상위의
CastService나ExceptionAdvice등에서 처리하도록 두는 게 더 나을 거 같습니다 - 교수님이 '처리 못할 예외는 다시 던져라`라고 강조하시는 걸 여러 번 들어서 적음..
- 지금은 에러만 출력한 후 똑같이
There was a problem hiding this comment.
- S3 설정 후 어짜피 바꿀 로직이여서 일단 그냥 두겠습니다!
- 반환하는 부분 파일명 반환으로 바꾸고 번역 기능 마무리 후에 exception 처리 하겠습니다아
Comment on lines
47
to
48
| Map<String, Object> voice = new HashMap<>(); | ||
| voice.put("name", ttsdto.getVoice()); |
There was a problem hiding this comment.
지금 당장 할 부분은 아니긴 한데, voice에도 디폴트 값을 넣거나 예외처리하는 로직이 있으면 좋을 것 같아요
voice가 비었거나 이상한 값 넣으니까 요런 응답을 내뿜더라고요
물론 급한 기능 아니니까 나아중에 해도 될 것 같습니다
There was a problem hiding this comment.
voice default 처리...
사실 enum 처리를 하는 방법도 있는데 나아중에 이야기 해봅시다ㅋㅋㅋ
Comment on lines
+59
to
+60
| ResponseEntity<Map> response = restTemplate.exchange(url, HttpMethod.POST, entity, Map.class); | ||
| String audioContent = (String) response.getBody().get("audioContent"); |
There was a problem hiding this comment.
그리고 궁금한 점 있는데
Google API 읽어보니까 response에서 timepoints로 문장별 시작시간 접근할 수 있는 거 맞죠??
요 timepoints들을 어디다 저장해야할지 모르겠네
There was a problem hiding this comment.
네네 따로 사진 보내드린 것처럼 나오는데 아무래도 sentence에 저장하는게 맞지 않을까요??
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
DONE
TODO