Skip to content

#4 가사생성 파이프라인 구축#70

Merged
crosstar1228 merged 20 commits intodevfrom
feat/lyric_gen
Aug 10, 2025
Merged

#4 가사생성 파이프라인 구축#70
crosstar1228 merged 20 commits intodevfrom
feat/lyric_gen

Conversation

@crosstar1228
Copy link
Copy Markdown
Collaborator

📝 Summary

✅ Checklist

  • 관련 이슈가 명시되어 있습니다.
  • 테스트가 완료되었습니다.
  • 문서 업데이트가 포함되었습니다.
  • 코드 리뷰를 위한 사전 검토를 완료했습니다.

📄 Description

💡 Notice (Optional)

🔗 Related Issue(s)

crosstar1228 and others added 16 commits August 2, 2025 12:40
- Music Team 에 사용될 모델을 초기화하는 get_openai_model 함수 추가
- 가사 생성을 위한 "get_lyric_template" 함수 추가
- LyricGenerationNode 클래스 추가
- start-lyric_generation-end 과정으로 기능 추가함
- 날씨 기반 가사 생성을 지원하는 WeatherGenerationNode를 추가
- 기존 그래프를 병렬 노드로 구현
- 파일 상단에 dotenv 로딩 코드 추가
- DiaryGenerationNode에서 프롬프트 키를 ‘diary_query’로 통합하여 입력 방식 일관화 및 가독성 향상
- 날씨·영상 분석 컨텍스트 기반 가사 생성 기능을 제공하는 LyricGenerationNode 도입
- 외부 API로 실시간 날씨 정보를 조회하는 WeatherGenerationNode 도입
- 코드 클린업을 위해 불필요하게 주석 처리된 코드를 제거
- lyric 템플릿에 오늘의 날씨와 유투브 분석 내용을 포함
- 입력 변수 목록에 weather_info와 video_analysis 추가
- MusicState 클래스에 diary_query와 lyric_query 필드를 추가 및 state 값 통합
- 날씨 정보를 가져오고 음악 생성에 활용할 수 있도록 변환하는 WeatherService 클래스 구현
- 날씨 데이터 파싱 및 포맷팅 기능 포함
- 기본 X, Y 좌표를 서울 용산구로 설정
- 실수로 누락된 코드 다시 추가가
Copy link
Copy Markdown
Collaborator

@Kimhansav Kimhansav left a comment

Choose a reason for hiding this comment

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

제 부분은 누락 없이 잘 된 것 같습니다!

@wlsrb9564
Copy link
Copy Markdown
Collaborator

image 현재 전체적으로 잘 작동되며 빠진 코드 부분도 없는 것을 확인했습니다. 추후 LangGraph Studio에서는 입력값으로 Diary Query와 Lyric Query만 받으면 되는데, 현재 Weather Info, Youtube Query 등 하위 노드의 값들도 입력할 수 있게 되어 있어 헷갈릴 수 있을 것 같습니다. 이 부분은 state.py에서 weather_info: NotRequired[str]로 수정하면 LangGraph Studio에서 해당 입력란이 사라질 것으로 보입니다. 추후 제가 수정하도록 하겠습니다.

wlsrb9564
wlsrb9564 previously approved these changes Aug 5, 2025
@crosstar1228
Copy link
Copy Markdown
Collaborator Author

@qjrm1430 @gsgh3016 이거 merge block된 이유가 뭔가요? 혹시 특정인원의 approve가 필요한가요?

qjrm1430
qjrm1430 previously approved these changes Aug 5, 2025
@qjrm1430 qjrm1430 requested review from gsgh3016 and susani-01 August 5, 2025 05:32
@qjrm1430
Copy link
Copy Markdown
Member

qjrm1430 commented Aug 5, 2025

@qjrm1430 @gsgh3016 이거 merge block된 이유가 뭔가요? 혹시 특정인원의 approve가 필요한가요?

github ruleset에 변경사항이 적용되지 않아서 발생헀던 문제였습니다..! 현재 해당 문제는 해결되었습니다.

Comment thread .env.example Outdated
Comment thread agents.svg
@crosstar1228 crosstar1228 dismissed stale reviews from qjrm1430 and wlsrb9564 via 2ebd38c August 7, 2025 14:00
@crosstar1228
Copy link
Copy Markdown
Collaborator Author

@qjrm1430 말씀주신거 반영해서 approve 부탁드립니다! 그리고 github write 권한(approve권한) 있으신 분이 중헌님 말고 또 있나요?

@crosstar1228 crosstar1228 merged commit b18a5e0 into dev Aug 10, 2025
2 checks passed
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.

4 participants