Skip to content

Conversation

@jhongy1994
Copy link
Contributor

#️⃣ Issue Number

📝 요약(Summary)

  • db_type을 선택적으로 받도록 함
  • 쿼리 실행이 필요한 체크박스를 선택안하면 DB에 연결하지 않음

💬 To Reviewers (선택)

  • show tableshow chart 체크박스 모두 선택 시, 두번 반복해서 쿼리를 실행하도록 되어있어서 이를 공용으로 사용하도록 변경했습니다. 문제는 없을지 참고해주세요!

PR Checklist

  • .env 파일에서 db_type 이나 정보를 주석처리 하고 실행
  • show tableshow chart를 체크하지 않았을때, 쿼리 실행이 안되는지 확인

reference) How to Code Review

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

Copy link
Collaborator

@ehddnr301 ehddnr301 left a comment

Choose a reason for hiding this comment

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

👍 Database 관련 오류가 나지않고 잘 작동하는것을 확인했습니다!! (black formatting만 재확인 부탁드립니다!)

💬 관련해서 확인하다보니 lang2sql.py 의 내용도 정리가 필요해보이네요ㅎㅎ...!!

💬 추가로 권한관련해서 Member로 초대 보내드렸는데 확인부탁드립니다! (fork한 pr은 github action이 자동으로 돌지를 않네요ㅎㅎ;;)

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.

👍 Black Formatting만 수정하면 될것같아요~!

)
st.plotly_chart(fig)
else:
st.error("SQL 원본이 문자열이 아닙니다.")
Copy link
Contributor

Choose a reason for hiding this comment

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

💊 제 기억이 확실하지 않지만..... 리마인드 차원으로 기록만 해두겠습니다~
이 else가 필요했던 이유가 우리는 크게보면 "한국어로 질문" -> "LLM에서 쿼리를 리턴" -> "리턴받은 쿼리를 실행" 인데,
간헐적으로 질문에 따라 LLM에게 리턴받은 쿼리가.. 쿼리 형태가 아닌 경우가 있어서 이런식으로 예외처리를 했던 기억이 있습니다.
추후에 다양한 질문으로 테스트해볼텐데, 요 부분에서 자꾸 걸리는 현상이 있으면 다시 살펴보면 좋을 것 같습니다~

@jhongy1994 jhongy1994 merged commit 430fdd9 into CausalInferenceLab:master Sep 29, 2025
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.

[기능] 쿼리 생성과 실행을 분리

3 participants