Skip to content

123 cli output format 지원 #125

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

Merged
merged 2 commits into from
Jul 3, 2025
Merged

Conversation

ehddnr301
Copy link
Collaborator

#️⃣ Issue Number

📝 요약(Summary)

  • main.py 파일 삭제
  • basic_graph 및 enriched_graph 그래프 구성 분리
  • query cli 추가
  • cli output foramt 추가
  • query_executor 를 cli에서도 streamlit에서도 사용

💬 To Reviewers (선택)

  • 테스트 부탁드립니다!
  • 예시: lang2sql query "browser 정보"
  • cli로 쿼리 생성말고도 해석, 설명, 테이블 목록 등도 얻을 수 있도록 고려중인데 의견주세요!
  • 리팩토링 코드가 적절한지 검토해주세요!

PR Checklist

  • lang2sql query "browser 정보"

reference) How to Code Review

  • 따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
  • 느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
  • 물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
  • 알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.

- main.py 파일 삭제
- basic_graph 및 enriched_graph 그래프 구성 분리
- query cli 추가
- cli output foramt 추가
- llm_utils/query_executor.py 파일 추가: CLI와 Streamlit에서 공통으로 사용할 수 있는 쿼리 실행 함수 구현
- query_command 함수에서 그래프 실행 로직을 execute_query 함수로 변경
- lang2sql.py에서 그래프 실행 로직을 execute_query 함수로 통합하여 코드 간소화
@ehddnr301 ehddnr301 linked an issue Jun 29, 2025 that may be closed by this pull request
Copy link
Contributor

@ParkGyeongTae ParkGyeongTae left a comment

Choose a reason for hiding this comment

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

👍

@ehddnr301 ehddnr301 merged commit 6a5451c into master Jul 3, 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.

cli output format 지원
2 participants