Skip to content

Conversation

@HWALIMLEE
Copy link
Contributor

#️⃣ Issue Number

  • 194

📝 요약(Summary)

  • graph_builder 메뉴에서 preset 설정이 query_executor 로 전파되지 않는 문제

💬 To Reviewers (선택)

  • execute_query 실행할 때 graph_config 설정 값을 가져올 수 있도록 수정했습니다.

PR Checklist

  • 그래프 빌더 메뉴에서 확장으로 선택 후 langgraph 실행결과를 이미지로 저장 후 잘 동작하는 것을 확인

reference) How to Code Review

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

@ehddnr301
Copy link
Collaborator

👍 @HWALIMLEE 기여 감사합니다!

❗❓
image

engine/query_executor.py 에서 디버깅용 print문을 아래와 같이 추가하고 streamlit에서 실행했을때 수정때 사용하신 변수가 인식되지 않는것 같습니다. (변수명이 다릅니다.)

추가로 default값으로 넣게되면 함수 정의 시점에 평가되어서 session_state가 업데이트 되어도 반영되지 않을듯 합니다.

    print("--------------------------------")
    print(f"use_enriched_graph: {use_enriched_graph}")
    print(st.session_state.get("use_enriched", False))
    print("--------------------------------")
--------------------------------
use_enriched_graph: False
True
--------------------------------

💊 streamlit의interface/app_pages/lang2sql.py에서 run_lang2sql 을 호출하고 있는데 여기서 파라미터를 넘겨주면 어떨까 싶습니다.

❗Black 을 통해서 포매팅 부탁드립니다!

💬 리뷰를 위해 살피다 보니 코드가 복잡하기는 하네요...! 변수명도 그렇고 정비의 필요성을 여전히 느낍니다!

@HWALIMLEE
Copy link
Contributor Author

아 저걸 체크하고 돌리는 걸 깜빡했군요..!
다시 수정해서 pr 올리겠습니다~

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.

👍

@HWALIMLEE HWALIMLEE changed the title graph_config 변수 값 받아올 수 있도록 수정 run_lang2sql 단에서 파라미터 받아올 수 있도록 수정 Oct 22, 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