Feedback #1

github-classroom · 2024-11-11T01:17:38Z

👋! GitHub Classroom created this pull request as a place for your teacher to leave feedback on your work. It will update automatically. Don’t close or merge this pull request, unless you’re instructed to do so by your teacher.
In this pull request, your teacher can leave comments and feedback on your code. Click the Subscribe button to be notified if that happens.
Click the Files changed or Commits tab to see all of the changes pushed to the default branch since the assignment started. Your teacher can see this too.

Notes for teachers

Use this PR to leave feedback. Here are some tips:

Click the Files changed tab to see all of the changes pushed to the default branch since the assignment started. To leave comments on specific lines of code, put your cursor over a line of code and click the blue + (plus sign). To learn more about comments, read “Commenting on a pull request”.
Click the Commits tab to see the commits pushed to the default branch. Click a commit to see specific changes.
If you turned on autograding, then click the Checks tab to see the results.
This page is an overview. It shows commits, line comments, and general comments. You can leave a general comment below.
For more information about this pull request, read “Leaving assignment feedback in GitHub”.

Subscribed: @Hwan7919 @aaiss0927 @wlgudths @hanbyeol1001 @chungSungMin

- commit message를 수정하여 ISSUE 제목과 통일되게 맞추었습니다. Resolves: #4

Resolves: #2

[DOCS] update gitmessage.txt

DOCS: 나머지 issue 템플릿 작성완료

- 커밋 메시지 수정했습니다. - commit templates 설정을 전역 설정으로 바꾸었습니다. Resolves: #7

[DOCS] update commit templates

Resolves: #3

[FEAT] PR 템플릿을 추가하였습니다.

- customDataset을 구축하였습니다. config 파일로 관리가능하도록 구현하여 config 내에서 이미지 경로와 증강 기법을 활용하여 이에 맞는 이미지, 라벨을 반환하도록 설계하였습니다. Fixes : #12

Fixes: #12

- augmentation, dataloader, dataset, train_val split을 모두 모듈화 하여 작성하였습니다. 기본적으로 각각 로딩 및 증강 적용 실험은 완료하였으나, 추가적으로 모든 실험을 통해 에러를 확인 할 계 Fixes: #12

Fixes: #12

split 데이터 코드 상 val과 중복되어 뽑는 경우 존재하여 이를 수정하였고, 추가적으로 dataset.py에서 test 이미지 또한 split 하는 경우가 존재하여 이를 수정하였습니다. Fixes: #12

Fixes: #12

Fixes: #13

Feat 13/make train.py

Resloves: #12 #13

Develop 중간 train, dataset merge

[CHORE] train.py 안에 do_train 함수 내 config 변수이름 수정

Develop

- configs : torchvision segmentation model의 설정파일 - utils : 모델에 맞게 num_classes 수정 함수 - torchvision/model_loader.py : 모델 불러오는 함수 Resolves: #14

[FEAT] baseline code model 추가

Fixes: #14

Related to: #17

Develop

[FEAT] 사용하지 않는 라이브러리 제거

mybirth0407

전체적으로 굉장히 잘 작성된 코드입니다. 2.의 이유로 렉이 너무 심해 적당히 잘 작성된 경우 의견을 남기지 않았습니다.
다만, 리뷰를 받을 준비가 전혀 되어있지 않은 PR입니다. 불필요한 파일이 너무 많습니다. 수많은 설정 파일과 리드미 파일은 제가 리뷰할 필요가 없으므로, 리뷰 단계에선 올리시면 안 됩니다.

코드만 봐도 고생 많았다는 게 느껴집니다. 수고 많았습니다.

mybirth0407 · 2024-12-08T20:46:56Z

HYDRA/config/config.yaml

다양한 옵션들을 포함하고 있어, 실험에 용이해 보입니다.

보통 데이터 경로는 파일 형식의 config 관리라면 절대경로로 표현하는 게 좋습니다.

mybirth0407 · 2024-12-08T20:52:51Z

HYDRA/data_utils.py

가장 먼저 보인 .py 파일이라 여기에 적습니다.

최근에는 type hint를 작성해주는 것이 대세이므로 함수의 입출력에 대해 type hint를 추가해주시면 좋겠습니다.

val이나 valid 하나로 통일되면 좋겠습니다.

mybirth0407 · 2024-12-08T20:54:18Z

HYDRA/data_utils.py

+    print(f"Training set: {len(train_filenames)}, Validation set: {len(val_filenames)}")
+    return train_filenames, train_labelnames, val_filenames, val_labelnames
+
+def create_datasets(train_files, train_labels, val_files, val_labels, config, preprocessing_fn, feature_extractor=None, normalize=True):


한 줄이 너무 길어지니 한 매개변수마다 개행하는 게 좋겠습니다.

mybirth0407 · 2024-12-08T20:55:47Z

HYDRA/dataset.py

os.path는 기니 osp로 줄여 씁니다.

try-except 구문이 꼭 필요한지 잘 모르겠습니다.

mybirth0407 · 2024-12-08T21:00:13Z

HYDRA/inferencer.py

인퍼런스 할 때는 안전하게 model.eval()과 no_grad()를 같이 해주는 게 좋습니다.

try-except 구문이 왜 필요한지 모르겠습니다.

mybirth0407 · 2024-12-08T21:14:28Z

src/Model/utils/model_utils.py

어떤 파일은 라이브러리 임포트 부분과 코드 내용간 개행이 2줄이고, 어떤 파일은 1줄인데, 2줄로 통일해주세요.

한 줄의 최대 글자 수는 79자로 통일해주세요.

mybirth0407 · 2024-12-08T21:22:10Z

src/Train/loss/custom_loss.py

직접 함수 구현하신거 너무 좋습니다. 훌륭합니다.

mybirth0407 · 2024-12-08T21:24:48Z

src/Utils/combine_csv_predictions.py

csv1 <-> file_1 어색합니다.

mybirth0407 · 2024-12-08T21:26:06Z

src/Utils/set_seed.py

utils 내의 함수들과 파일들을 잘 정리할 필요가 있겠어요.

set_seed는 파일 하나로 뺄 정도는 아닙니다.

mybirth0407 · 2024-12-08T21:27:24Z

src/train.py

default 인자에 경로를 넣을 때는 절대경로로 넣는 것이 좋습니다.

mybirth0407 · 2024-12-08T21:30:29Z

고생 많았습니다.

- 개행문자 통일 - os.path -> osp 변경 - train을 제외한 디렉토리 이름과 파일명 소문자로 변경 - model_utils.py 코드 개행, 최대 글자 수 79자 - model의 config 예시 제외 삭제 Resolves: #107

Fix 107/update feedback

Fixes: #107

[DOCS] 피드백 반영 및 추가 수정 사항 적용용

Develop 갱신 갱갱갱

github-classroom bot and others added 30 commits November 11, 2024 01:17

Setting up GitHub Classroom Feedback

a00991e

Feat: commit template 추가

f35de29

[DOCS] update gitmessage.txt

e5e998c

- commit message를 수정하여 ISSUE 제목과 통일되게 맞추었습니다. Resolves: #4

DOCS: bug_request.md 작성

9651365

Resolves: #2

DOCS: 나머지 issue 템플릿 작성완료

6052b2c

Resolves: #2

Merge pull request #5 from boostcampaitech7/docs-4/commit_templates

477eec7

[DOCS] update gitmessage.txt

Merge pull request #6 from boostcampaitech7/docs-#3/issue_template

1a2c4a2

DOCS: 나머지 issue 템플릿 작성완료

[DOCS] update commit templates

b360374

- 커밋 메시지 수정했습니다. - commit templates 설정을 전역 설정으로 바꾸었습니다. Resolves: #7

Merge pull request #8 from boostcampaitech7/docs-7/commit_templates

a8648bc

[DOCS] update commit templates

[FEAT] PR 템플릿을 추가하였습니다.

f17b83b

Resolves: #3

Merge pull request #9 from boostcampaitech7/docs-3/PR_temp

700202d

[FEAT] PR 템플릿을 추가하였습니다.

[FEAT] Dataset.py 구현완료

6423625

- customDataset을 구축하였습니다. config 파일로 관리가능하도록 구현하여 config 내에서 이미지 경로와 증강 기법을 활용하여 이에 맞는 이미지, 라벨을 반환하도록 설계하였습니다. Fixes : #12

[CHORE] type hint 적용

683b26c

Fixes: #12

[FEAT] config val -> valid

575788b

Fixes: #12

[FEAT] split.py and dataset.py and dataloader 수정정

4c2be4c

split 데이터 코드 상 val과 중복되어 뽑는 경우 존재하여 이를 수정하였고, 추가적으로 dataset.py에서 test 이미지 또한 split 하는 경우가 존재하여 이를 수정하였습니다. Fixes: #12

[FEAT] 파이썬 코드 엔터, 띄어쓰기 관

8952849

Fixes: #12

[RENAME] 구조 재구성

37ba138

[FEAT] train.py + dataset + dataloader Merge version

e0160f6

Fixes: #13

Merge pull request #15 from boostcampaitech7/feat-13/make_train.py

4d9579e

Feat 13/make train.py

[FEAT] FIX

840d4f8

Resloves: #12 #13

[CHORE] 불필요한 code 폴더 삭제

e92bcfa

Merge pull request #18 from boostcampaitech7/develop

32baa4b

Develop 중간 train, dataset merge

[CHORE] train.py 안에 do_train 함수 내 config 변수이름 수정

7287fc8

Merge pull request #20 from boostcampaitech7/feat-17/develop_train

9ab0f98

[CHORE] train.py 안에 do_train 함수 내 config 변수이름 수정

Merge pull request #21 from boostcampaitech7/develop

4720020

Develop

[FEAT] baseline code model 추가

8b68031

- configs : torchvision segmentation model의 설정파일 - utils : 모델에 맞게 num_classes 수정 함수 - torchvision/model_loader.py : 모델 불러오는 함수 Resolves: #14

Merge pull request #22 from boostcampaitech7/feat-14/make_model.py

d429f8b

[FEAT] baseline code model 추가

[FEAT] develop + model 모듈화 코드 Merge

b87ee29

Fixes: #14

[FEAT] config 수정, val주기에 체크포인트 저장

badd491

Related to: #17

aaiss0927 and others added 18 commits December 6, 2024 18:53

저장되는 config 파일명 수정

0e05598

[Docs] hydra 수정

5da15bf

Update README.md

1cf8d64

Merge branch 'main' into develop

24a02bb

[FIX] devlop과 main 충돌 해결

43e950b

Merge pull request #104 from boostcampaitech7/develop

8812946

Develop

Rename model.py to model.py

00627ba

Rename model.py to model.py

5198fb8

Rename model to model_temp

85065fd

Rename model_temp to Model

b0d9815

Rename train to tain_temp

0a98bf7

Rename train_temp to train

046ff03

Rename train to utils

03aae0a

Rename train_temp to utils

3a4cb80

Merge branch 'main' into develop

ec5693a

Merge pull request #105 from boostcampaitech7/develop

802fe24

Develop

[FEAT] 사용하지 않는 라이브러리 제거

d7c2ed8

Merge pull request #106 from boostcampaitech7/develop

86f6aa9

[FEAT] 사용하지 않는 라이브러리 제거

mybirth0407 reviewed Dec 8, 2024

View reviewed changes

wlgudths and others added 10 commits December 9, 2024 19:24

[DOCS] 피드백 반영 수정

8cefd79

- 개행문자 통일 - os.path -> osp 변경 - train을 제외한 디렉토리 이름과 파일명 소문자로 변경 - model_utils.py 코드 개행, 최대 글자 수 79자 - model의 config 예시 제외 삭제 Resolves: #107

Merge pull request #108 from boostcampaitech7/fix-107/update_feedback

9ac74bd

Fix 107/update feedback

[DOCS] 피드백 반영 및 추가 수정 사항 적용용

855c0c2

Fixes: #107

[FIX] config 변수명 수정

9081227

[FIX] config 변수명 수정

4e2a0ef

Merge pull request #109 from boostcampaitech7/fix-107/update_feedback

1aa58e6

[DOCS] 피드백 반영 및 추가 수정 사항 적용용

Merge pull request #110 from boostcampaitech7/develop

429cac4

Develop 갱신 갱갱갱

[FIX] config 타입 변경

bfb2b8d

[FIX] config 타입 변경

e410393

Update README.md

18edd10

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feedback #1

Feedback #1

github-classroom bot commented Nov 11, 2024 •

edited

Loading

mybirth0407 left a comment

mybirth0407 Dec 8, 2024

mybirth0407 Dec 8, 2024

mybirth0407 Dec 8, 2024

mybirth0407 Dec 8, 2024

mybirth0407 Dec 8, 2024

mybirth0407 Dec 8, 2024

mybirth0407 Dec 8, 2024

mybirth0407 Dec 8, 2024

mybirth0407 Dec 8, 2024

mybirth0407 Dec 8, 2024

mybirth0407 commented Dec 8, 2024

Feedback #1

Are you sure you want to change the base?

Feedback #1

Conversation

github-classroom bot commented Nov 11, 2024 • edited Loading

mybirth0407 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

mybirth0407 commented Dec 8, 2024

github-classroom bot commented Nov 11, 2024 •

edited

Loading