Skip to content

180 기능 cli 실행 옵션을 UI에서 통합 관리하도록 개선 #187

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 11 commits into from
Oct 5, 2025
Merged

180 기능 cli 실행 옵션을 UI에서 통합 관리하도록 개선 #187

merged 11 commits into from
Oct 5, 2025

Conversation

ehddnr301
Copy link
Collaborator

#️⃣ Issue Number

📝 요약(Summary)

  • CLI 실행/구성 옵션을 Streamlit Settings로 통합해 env 세팅과정을 제거하려했습니다.

💬 To Reviewers (선택)

  • 동작에 오류는 없는지 체크 부탁드립니다.
  • 현재 모든 환경변수 값을 Json으로 저장해 사용합니다. (추후에 DB환경이 잘 지원되면 분리가능할 것으로 보입니다.)
    • 이번 PR에서 모두 적용하지 않은 이유는 Datahub처럼 자연스러운 Docker환경을 제공하지 않는데 DB를 추가하는것에 부담을... 느꼈습니다!
  • 작업을 하다보니 고봉밥이 되어버렸는데 죄송합니다..!!

설정 페이지 제공

image

사이드바 에서 설정 선택

image

PR Checklist

  • uv run lang2sql run-streamlit -p 8888 로 실행 후 동작 확인

reference) How to Code Review

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

@ehddnr301
설정 페이지 추가 및 데이터 소스 섹션 추가 & 기존 Cli 지원 제거
1ca5d5d
@ehddnr301
데이터 소스 설정을 Lang2SQL 페이지에서 선택해 사용가능하도록 수정
dd24db6
@ehddnr301
VectorDB 설정을 CLI에서 제거하고 UI에서 관리하도록 변경. 관련된 옵션 및 초기화 함수 수정.
af2d7b6
@ehddnr301
LLM 및 Embeddings 선택 기능 추가: Lang2SQL 페이지와 설정 페이지에 LLM 및 Embeddings 선택 …
b31de27 
…컴포넌트 통합
@ehddnr301
LLM/Embeddings/DB 프로파일(시크릿 포함) JSON 저장·사이드바 적용 + DB 섹션 CRUD 확장
a6e9cc1
@ehddnr301
설정 페이지에서 디바이스 탭 제거 및 관련 정보 메시지 삭제
1295296
@ehddnr301
설정 관련 모듈을 분리하여 재구성
656cbdd
@ehddnr301
불필요한 체크박스 제거
97a3307
@ehddnr301
sidebar_component로 디렉토리명 수정
02c5149
@ehddnr301 ehddnr301 linked an issue Oct 4, 2025 that may be closed by this pull request
[기능] CLI 실행 옵션을 UI에서 통합 관리하도록 개선 #180
Closed
ParkGyeongTae
ParkGyeongTae approved these changes Oct 4, 2025
View reviewed changes
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.

👍 완전 좋네여! 잘동작합니다~!

interface/core/config/__init__.py Outdated
DEFAULT_VECTORDB_TYPE,
DEFAULT_VECTORDB_LOCATION,
load_config,
_get_session_value,
Copy link
Contributor

Choose a reason for hiding this comment

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

💊 _ 접두사 함수의 경우 클래스 내에서만 선언된 경우 클래스 내에서만 private하게 사용, 파일에서 함수만 선언된 경우 해당 파일에서만 사용하는 것으로 추후에 개선할 수 있을 것 같아요~!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

👍 좋은 지적 감사합니다! 초기에 하나의 파일로 구현하고 쪼개다 보니 기존 함수명을 그대로 들고와서, 불필요한 import를 제거하고 함수명을 업데이트 했습니다. f25e897

@ehddnr301 ehddnr301 merged commit efa97ed into master Oct 5, 2025
1 check passed
@ehddnr301 ehddnr301 mentioned this pull request Oct 13, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ParkGyeongTae ParkGyeongTae ParkGyeongTae approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[기능] CLI 실행 옵션을 UI에서 통합 관리하도록 개선

2 participants

@ehddnr301 @ParkGyeongTae