Skip to content

Feature/105 add setup clickhouse prompt dir #106

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bongkyunSON
Copy link

#️⃣ Issue Number

📝 요약(Summary)

  • setup.py에 clickhouse-driver, prompt dir 추가

💬 To Reviewers (선택)

  • clickhouse-driver 버전이 맞는지 확인부탁드립니다.
  • prompt는 streamlit을 실행했을때 문제가 생기던데 저만 그런건지는 모르겠어요. 가상환경은 .venv입니다

PR Checklist

reference) How to Code Review

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

@bongkyunSON bongkyunSON added the bug Something isn't working label May 13, 2025
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.

.env.example, pyproject.toml 은 어떤 이유로 삭제된지 알 수 있을까요?

❗ black 으로 formatting이 필요한것으로 보입니다


with open("docs/README.md", "r", encoding="utf-8") as fh:
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗ docs/README.md 에 프로젝트 실행관련을 추가해두어서 해당 부분을 의도한 바이기는 합니다.

Copy link
Author

@bongkyunSON bongkyunSON May 15, 2025

Choose a reason for hiding this comment

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

.env.example, pyproject.toml 은 어떤 이유로 삭제된지 알 수 있을까요?

.env.example -> 습관적으로 삭제했네요...특별한 이유는 없습니다.
pyproject.toml 삭제하고 install하면서 다시 복구가 안된것같습니다..ㅎㅎ

❗ black 으로 formatting이 필요한것으로 보입니다

네 formatting 하겠습니다.

❗ docs/README.md 에 프로젝트 실행관련을 추가해두어서 해당 부분을 의도한 바이기는 합니다.

감사합니다.

setup.py Outdated
version="0.1.9", # 버전
author="ehddnr301",
author_email="[email protected]",
url="https://github.com/CausalInferenceLab/Lang2SQL",
description="Lang2SQL - Query Generator for Data Warehouse",
long_description=long_description,
long_description_content_type="text/markdown",
packages=find_packages(), # my_package를 자동으로 찾음
packages=find_packages() + ['prompt'], # prompt 패키지 직접 추가
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍 오 md 파일이 일반적으로는 포함이 안되는게 이 이유에서였군요!!

❓ 아래 추가로만으로도 빌드파일에 md파일이 포함되는것으로 보입니다. 혹시 data_files와 include_package_data 는 어떤 이유로 추가된건지 알수있을까요!?

    packages=find_packages() + ["prompt"],  # my_package를 자동으로 찾음
    package_data={
        "prompt": ["*.md", "*.py"],  # prompt 디렉토리의 모든 .md 파일 포함
    },

Copy link
Author

@bongkyunSON bongkyunSON May 15, 2025

Choose a reason for hiding this comment

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

👍 오 md 파일이 일반적으로는 포함이 안되는게 이 이유에서였군요!!
저만 안돼는게 아니였군요 ㅎㅎ

❓ 아래 추가로만으로도 빌드파일에 md파일이 포함되는것으로 보입니다. 혹시 data_files와 include_package_data 는 어떤 이유로 추가된건지 알수있을까요!?

    packages=find_packages() + ["prompt"],  # my_package를 자동으로 찾음
    package_data={
        "prompt": ["*.md", "*.py"],  # prompt 디렉토리의 모든 .md 파일 포함
    },

패키지를 사용하는 모든 사용자를 고려해 오픈소스에 적합하다고 생각되어 포함하였습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants