Skip to content

프롬프트 관리 #11

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
Closed
3 tasks
nonegom opened this issue Mar 9, 2025 · 3 comments
Closed
3 tasks

프롬프트 관리 #11

nonegom opened this issue Mar 9, 2025 · 3 comments
Labels
enhancement New feature or request prompt The input provided by the user in natural language that serves as the basis for SQL generation.
Milestone
Mission: Version ...

Comments

@nonegom
Copy link
Contributor

nonegom commented Mar 9, 2025

Why ❓

현재 상황 및 문제점 👿:

프롬프트가 메인 코드에 함게 작성이 되어 있으며, 이로 인해 관리 및 코드 가독성에 이슈가 있을 것으로 판단됩니다.

How 🤔

해결 방안 ✅:

프롬프트를 yaml로 관리함으로써, 프롬프트 관리 편의성과 코드 가독성을 챙길 수 있을 것으로 기대됩니다.

What 🛠️

TODO

  • 프롬프트 파일 관리
  • 프롬프트 로드 클래스 생성
  • 기반 코드 수정
@ehddnr301
Copy link
Collaborator

너무 좋습니다 !_!

@ehddnr301
Copy link
Collaborator

첨언: DB에 따라 Fewshot을 다르게 넣는 부분도 커버가능할것 으로 보임

@ehddnr301 ehddnr301 added prompt The input provided by the user in natural language that serves as the basis for SQL generation. enhancement New feature or request labels Mar 11, 2025
@DShomin DShomin mentioned this issue Mar 18, 2025
Merged
@nonegom nonegom closed this as completed Mar 21, 2025
@nonegom nonegom reopened this Mar 21, 2025
@nonegom nonegom added this to the Mission: March Complete ✅ milestone Mar 21, 2025
ehddnr301 added a commit that referenced this issue Mar 25, 2025
@ehddnr301
Merge pull request #14 Resolve #6
8dd00ee 
#6 이슈 해결
#11 이슈에서 `쿼리 설명이 영어로 나오는` 부분 해결 기대
@ParkGyeongTae
Copy link
Contributor

가독성이 훨~씬 좋아질 것 같습니다.
.yaml 파일, .md 파일 둘 다 좋은데 .md가 좀 더 선호되기는 하는것 같아요!
.yaml의 경우 들여쓰기 문제로 from textwrap import dedent 와 같은 코드가 추가될 것 같네요..!

@nonegom nonegom linked a pull request Apr 7, 2025 that will close this issue
Feature/11 prompt config #44
Closed
@nonegom nonegom mentioned this issue Apr 20, 2025
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request prompt The input provided by the user in natural language that serves as the basis for SQL generation.
Projects
None yet
Milestone
Mission: Version 1.0.0 Complete ✅
Development

Successfully merging a pull request may close this issue.

Feature/11 prompt config CausalInferenceLab/Lang2SQL
3 participants
@ehddnr301 @ParkGyeongTae @nonegom