-
Notifications
You must be signed in to change notification settings - Fork 6
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
[Feat] lint 설정, 가상환경 설정 및 baseline 코드 추가 #4
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
flake8, black, isort 등 설치법 추가
- 주석문 일부 삭제 - 불필요한 EDA 파트 삭제 - 일단 모듈화 및 함수화 없이 단일 파일 내에 일괄 작성
cuda index로 인해 pip freeze 시 오류 발생하여 직접 작성
chell9999
approved these changes
Nov 11, 2024
Usunwoo
approved these changes
Nov 11, 2024
gsgh3016
approved these changes
Nov 11, 2024
jagaldol
added
Priority: Medium
적절한 시기에 처리해야 할 작업
Priority: High
우선적으로 처리해야 할 중요한 작업
and removed
Priority: Medium
적절한 시기에 처리해야 할 작업
labels
Nov 12, 2024
Usunwoo
approved these changes
Nov 12, 2024
canolayoo78
approved these changes
Nov 12, 2024
eyeol
reviewed
Nov 12, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코멘트 확인 한번만 부탁드립니다 👀
eyeol
approved these changes
Nov 12, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정 사항 모두 확인했습니다. 고생하셨습니다!
canolayoo78
approved these changes
Nov 12, 2024
Usunwoo
reviewed
Nov 12, 2024
jagaldol
changed the title
[Feature] lint 설정, 가상환경 설정 및 baseline 코드 추가
[Feat] lint 설정, 가상환경 설정 및 baseline 코드 추가
Nov 12, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 Summary
프로젝트 협업을 위한 초기 세팅 작업을 하였습니다.
✅ Checklist
[ ] 관련 이슈가 명시되어 있습니다.📄 Description
프로젝트 환경설정
가상환경
venv
venv
를 사용하여 가상환경을 만들고 필요한 종속성을 설치할 수 있도록 구성해두었습니다.$ python -m venv .venv $ source .venv/bin/activate (.venv) $ pip install -r requirements.txt (.venv) $ python main.py
main.py
베이스 라인 코드로부터
main.py
를 생성하였습니다.lint
를 위해 코드 스타일 일부 수정 및 EDA 파트를 삭제하였습니다.학습 및 테스트 데이터
데이터는
data/
폴더에 저장하면 됩니다.baseline_code.ipynb
main.py에서 빠진 EDA는
baseline_code.ipynb
를 다운받아 확인해주세요!Flake8
,Black
,isort
Lint각 설정은 다음의 2가지 파일에서 관리됩니다:
-
.flake8
: flake8 설정 파일-
pyproject.toml
: black, isort 설정 파일cli 상에서는 아래의 명령어로 확인 가능합니다.
Flake8
flake8
은 오류를 검출하며 수정을 해주지는 않습니다. 직접 로직 등을 고쳐야합니다.Black Formatter
black
과isort
는 자동으로 수정을 해줍니다.isort
CI Github Action
위의 linting을
github action
을 통해CI
검사를 할 수 있도록 작성하였습니다.3가지의
job
이 각각 위 명령어들을 수행하여 lint를 검사합니다.PR이 업로드 되면 이와 같이 검사를 수행하게 됩니다.
💡 Notice (Optional)
README
환경 설정과 데이터 다운로드에 관련된 내용은 리드미에도 작성해두었습니다.
리드미를 통해서도 다시 한번 확인 부탁드립니다!
Extensions
코드 상에서 자동 포맷팅 및 오류를 확인하기 위해 다음의 extension을 설치하여야합니다:
Black Formatter
Flake8
isort
해당 extension들을 활용하여 vscode 상에서 auto formatting이 가능하도록 .vscode/settings.json을 함께 PR에 포함시켰습니다.